Bug 73214

Summary:	[test case] css picks wrong selector on hp.com (regression)
Product:	[Applications] konqueror	Reporter:	Hasso Tepper <hasso>
Component:	khtml renderer	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mihnea.balta
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	test case

Description Hasso Tepper 2004-01-22 08:48:56 UTC

Version:           3.2.0 (using KDE 3.2.0, compiled sources)
Compiler:          gcc version 3.3.3 20040110 (prerelease) (Debian)
OS:          Linux (i686) release 2.6.1-rc1-mm2

In HP drivers download page text is rendered in white to the white background. For example - http://h18007.www1.hp.com/support/files/EvoNotebook/us/locate/20_5557.html
It's OK with Mozilla and Opera.

Comment 1 Stephan Kulow 2004-01-22 14:29:37 UTC

Created attachment 4290 [details]
test case

I wonder there is no css test case for that, but it's surely mean

Comment 2 Dirk Mueller 2004-02-15 16:28:51 UTC

CVS commit by mueller: 

CCMAIL: 73214-done@bugs.kde.org


  M +5 -0      ChangeLog   1.205
  M +1 -1      css/css_base.cpp   1.14


--- kdelibs/khtml/ChangeLog  #1.204:1.205
@@ -1,2 +1,7 @@
+2004-02-15  Dirk Mueller  <mueller@kde.org>
+
+        * css/css_base.cpp (specificity): fix detection of the *
+        selector (#73214).
+
 2004-02-13  Zack Rusin  <zack@kde.org>
 

--- kdelibs/khtml/css/css_base.cpp  #1.13:1.14
@@ -135,5 +135,5 @@ unsigned int CSSSelector::specificity() 
         return 0;
 
-    int s = ((tag == 0xffff) ? 0 : 1);
+    int s = ((( tag & NodeImpl_IdLocalMask ) == 0xffff) ? 0 : 1);
     switch(match)
     {

Comment 3 Stephan Kulow 2004-02-22 12:53:42 UTC

*** Bug 75821 has been marked as a duplicate of this bug. ***