[webkit-reviews] review denied: [Bug 36828] Web Inspector: get rid of "Unsafe JavaScript attempt..." message when inspecting frames whose content has changed : [Attachment 52042] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 30 08:54:56 PDT 2010


Pavel Feldman <pfeldman at chromium.org> has denied Yury Semikhatsky
<yurys at chromium.org>'s request for review:
Bug 36828: Web Inspector: get rid of "Unsafe JavaScript attempt..." message
when inspecting frames whose content has changed
https://bugs.webkit.org/show_bug.cgi?id=36828

Attachment 52042: patch
https://bugs.webkit.org/attachment.cgi?id=52042&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
I don't think we should add such a complexity because of a corner case
scenario. I'd much rather not get into InjectedScript bindings from native code
if we know that inspected window is no longer available.


More information about the webkit-reviews mailing list