[webkit-reviews] review denied: [Bug 194607] Web Inspector: CSS Changes: resetting value still shows in the diff : [Attachment 365897] Patch

[bugzilla-daemon at webkit.org]

Nikita Vasilyev <nvasilyev at apple.com> has denied Nikita Vasilyev
<nvasilyev at apple.com>'s request for review:
Bug 194607: Web Inspector: CSS Changes: resetting value still shows in the diff
https://bugs.webkit.org/show_bug.cgi?id=194607

Attachment 365897: Patch

https://bugs.webkit.org/attachment.cgi?id=365897&action=review




--- Comment #12 from Nikita Vasilyev <nvasilyev at apple.com> ---
Comment on attachment 365897
  --> https://bugs.webkit.org/attachment.cgi?id=365897
Patch

There are a couple more CSS Changes bugs:
- Bug 195264 - Web Inspector: CSS Changes: modifications aren't shared for
rules that match multiple elements
- Bug 196215 - Web Inspector: Green highlight disappears from edited properties
when adding a new property

To fix them, I'm going to try a different approach:

- Check property equality based on their content. Unfortunately, CSSProperty
instances sometimes get re-created with the same content and strict equality
checks of models doesn't work.
- Copy all properties of a style declaration on the first edit. Updating
`this._initialState.properties` on each edit turned out to be unnecessary
confusing and error-prone.

This different approach would make this patch above obsolete.