Bug 147441 – Web Inspector: inherited CSS rules disappear from Styles sidebar while editing

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Nikita Vasilyev]

Created attachment 257818 [details]
Animated GIF of the problem

<table>
    <tr style="font-weight: 300">
        <td>font-weight: 300</td>
    </tr>
</table>

Steps:
1. Inspect <td>
2. In the styles sidebar, change "font-weight: 300" to "font-weight: "

Expected:
"font-weight: " stays where it was, even though it's invalid CSS.

Actual:
"font-weight: " disappears.

[Radar WebKit Bug Importer]

<rdar://problem/22066483>

[Nikita Vasilyev]

It doesn't look like a regression, it was broken in a 7 months old build (r177585).

The problem only exists with inline styles, the following works as expected:

<style>
  span {color: red}
</style>
<div>
  <span>Test</span>
</div>

This seem like a low priority bug, setting it to P4.

[Devin Rousso]

Created attachment 257899 [details]
Patch

[Nikita Vasilyev]

(In reply to comment #3)
> Created attachment 257899 [details]
> Patch

I can confirm that this fixes the issue for the provided test case.

[Timothy Hatcher]

Comment on attachment 257899 [details]
Patch

Do sections still update overrides when a duplicate property is added to the active section?

[Devin Rousso]

(In reply to comment #5)
> Do sections still update overrides when a duplicate property is added to the
> active section?

From what I can tell, it still does (but obviously only if the property is in a section above the existing property or an important is added).

[WebKit Commit Bot]

Comment on attachment 257899 [details]
Patch

Clearing flags on attachment: 257899

Committed r187714: <http://trac.webkit.org/changeset/187714>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.