Bug 39057 – Web Inspector: A disabled style property cannot be deleted

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

[Alexander Pavlov (apavlov)]

1. In some CSS rule (with 2 or more properties) in the Styles sidebar pane, disable a property
2. Delete the disabled property by clearing its edit-box, as usual
3. Disable another property in the same rule

The property deleted in step 2 re-appears in a disabled state.
The requirement of 2 or more properties in step 1 is not mandatory, it's there merely to manifest the issue in an easy way.

[Pavel Feldman]

Created attachment 55991 [details]
[PATCH] Proposed fix.

[Yury Semikhatsky]

Comment on attachment 55991 [details]
[PATCH] Proposed fix.

LayoutTests/inspector/styles-disable-then-delete.html:37
 +      testController.runAfterPendingDispatches(frontend_deleteContainerStyle.bind(this, testController));
Can you pass __result as a parameter as well instead of creating a global __result?
Also it's not clear what 'bind(this,' means in this context, should it be .bind(null, ?

[Pavel Feldman]

Created attachment 55996 [details]
[PATCH] Comments addressed.

[Pavel Feldman]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	C	LayoutTests/inspector/styles-disable-then-enable.html => LayoutTests/inspector/styles-disable-then-delete.html
	M	LayoutTests/ChangeLog
	M	LayoutTests/inspector/elements-tests.js
	M	LayoutTests/inspector/styles-disable-inherited.html
	A	LayoutTests/inspector/styles-disable-then-delete-expected.txt
	M	LayoutTests/inspector/styles-disable-then-enable.html
	M	WebCore/ChangeLog
	M	WebCore/inspector/InspectorDOMAgent.cpp
	M	WebCore/inspector/front-end/TestController.js
Committed r59381