Bug 64074 – Web Inspector: CSS inspector gets confused about specificity of !important properties

• Component: Web Inspector (Deprecated)
• Status: RESOLVED FIXED

[Pavel Feldman]

<Upstreaming from http://code.google.com/p/chromium/issues/detail?id=88345>

Chrome Version       : 14.0.803.0 (Official Build 90483) dev 
URLs (if applicable) : See attached test case.
Other browsers tested:
  Firefox 3.6.18: OK

What steps will reproduce the problem?
1. Open attached HTML document.
2. Inspect the "I should be green italic" span.
3. Specifically look at which properties have strikethrough.

What is the expected result?

The more specific rule (".outer span.middle.modifier > span") should take precedence over the less specific one. Indeed, WebKit renders this correctly. When inspecting the green span, both properties from the less specific rule should be stricken through.

What happens instead?

The inspector is confused, and shows a strikethrough "color: green !important".

This happens only with !important properties. The font-style property is only there to illustrate this. If you change it to !important (in both rules), it'll exhibit the same wrong behavior.

[Alexander Pavlov (apavlov)]

Created attachment 100114 [details]
[PATCH] Suggested fix

[Pavel Feldman]

Comment on attachment 100114 [details]
[PATCH] Suggested fix

View in context: https://bugs.webkit.org/attachment.cgi?id=100114&action=review

> Source/WebCore/inspector/front-end/StylesSidebarPane.js:457
> +            // FIXME: Will same-specificity rules be reported in reverse order (the active one first), too?

Could you check this?

[Alexander Pavlov (apavlov)]

Created attachment 100115 [details]
[PATCH] Unnecessary comment removed

[Alexander Pavlov (apavlov)]

(In reply to comment #2)
> (From update of attachment 100114 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=100114&action=review
> 
> > Source/WebCore/inspector/front-end/StylesSidebarPane.js:457
> > +            // FIXME: Will same-specificity rules be reported in reverse order (the active one first), too?
> 
> Could you check this?

Yes, the code works correctly in this case, too. Sorry about the confusion.

[WebKit Review Bot]

Comment on attachment 100115 [details]
[PATCH] Unnecessary comment removed

Clearing flags on attachment: 100115

Committed r90630: <http://trac.webkit.org/changeset/90630>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Alexander Pavlov (apavlov)]

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