Bug 73214

Summary: [test case] css picks wrong selector on hp.com (regression)
Product: [Applications] konqueror Reporter: Hasso Tepper <hasso>
Component: khtml rendererAssignee: Konqueror Developers <konq-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: mihnea.balta
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
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. ***