[webkit-reviews] review denied: [Bug 11888] REGRESSION (r18320): Web Inspector panes broken : [Attachment 11943] Patch

bugzilla-request-daemon at macosforge.org bugzilla-request-daemon at macosforge.org
Wed Dec 20 21:35:42 PST 2006

Alexey Proskuryakov <ap at webkit.org> has denied Alexey Proskuryakov
<ap at webkit.org>'s request for review:
Bug 11888: REGRESSION (r18320): Web Inspector panes broken

Attachment 11943: Patch

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
Could we wait just a bit more before reverting a behavior that's (a)
standard-compliant AND (b) matches WinIE? So far, there were no reports of Web
compatibility problems (except for www.apple.com/getamac, where we share IE
behavior now).

To correctly remove a property, one should just set it to empty:
"myElement.style.display = ''".

I believe this Inspector JS bug needs to be fixed even if we decide to disable
exceptions in this case, so r-.

More information about the webkit-reviews mailing list