[Webkit-unassigned] [Bug 106811] Web Inspector: Add iframe option to inspectedWindow.eval() extension API
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Jan 16 16:32:35 PST 2013
https://bugs.webkit.org/show_bug.cgi?id=106811
johnjbarton <johnjbarton at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |106926
--- Comment #8 from johnjbarton <johnjbarton at chromium.org> 2013-01-16 16:34:20 PST ---
(In reply to comment #6)
>
> > Source/WebCore/inspector/front-end/ExtensionServer.js:816
> > + callback(errorMessage);
>
> this should not be necessary, if you return an error status, the caller will invoke callback.
>
That turned out to be a good hint...
>
> > LayoutTests/inspector/extensions/extensions-eval.html:49
> > +function extension_testZEvalInIFrame(nextTest) {
>
> Z?
These two issues were related. ExtensionServer.js was not correctly propagating errors it detected before sending the request to the debug server because the caller for evaluate() in this case, _onEvaluateOnInspectedPage(), was not invoking the callback. That allowed the test driver to launch nextTest() on both the main-line and async callback line, which resulted in duplicate test entries that I was trying to hackaround with Z (so the test case that issued dupes was last and there were no more tests to dupe).
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list