Created attachment 329064 [details] [Image] Bug Everything between of /* */ should be green.
Created attachment 329461 [details] Patch
I addressed this in the patch for Bug 180676 - Web Inspector: Styles Redesign: tabbing on commented out property throws exception, which still needs to be reviewed. I'm sorry I didn't mention it in this bug.
(In reply to Nikita Vasilyev from comment #2) > I addressed this in the patch for Bug 180676 - Web Inspector: Styles > Redesign: tabbing on commented out property throws exception, which still > needs to be reviewed. I'm sorry I didn't mention it in this bug. I haven't had a chance to test <https://webkit.org/b/180676> fully yet, but from what I can see I think the approach taken here is safer/cleaner. I would either recommend landing it separately (since we have two bugzilla anyways) or changing it to match what is here.
Comment on attachment 329461 [details] Patch I'm fine with lending these two separately. The patch looks good. `.property:not(.disabled)` is consistent with what we already use for .token-link, .token-string, and .token-comment. I'd r+, but I'm not a reviewer.
Comment on attachment 329461 [details] Patch Attachment 329461 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/5670870 New failing tests: webgl/1.0.2/conformance/context/context-release-with-workers.html
Created attachment 329476 [details] Archive of layout-test-results from ews117 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews117 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Comment on attachment 329461 [details] Patch The test failure is unrelated (caused by <https://webkit.org/b/180770>).
Comment on attachment 329461 [details] Patch Clearing flags on attachment: 329461 Committed r226075: <https://trac.webkit.org/changeset/226075>
All reviewed patches have been landed. Closing bug.
<rdar://problem/36114984>