Bug 73214 - [test case] css picks wrong selector on hp.com (regression)
Summary: [test case] css picks wrong selector on hp.com (regression)
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
: 75821 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-01-22 08:48 UTC by Hasso Tepper
Modified: 2004-02-22 12:53 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
test case (515 bytes, text/html)
2004-01-22 14:29 UTC, Stephan Kulow 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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. ***