RESOLVED FIXED 90799
Web Inspector: [Extensions API] simplify panel size check in extensions-panel.html 
https://bugs.webkit.org/show_bug.cgi?id=90799
Summary Web Inspector: [Extensions API] simplify panel size check in extensions-panel...
Andrey Kosyakov
Reported 2012-07-09 10:37:54 PDT
We used to have an ugly hack to compare inner extension window size to outer panel size. Now that we have window argument to panel.onShown(), the check can be simplified. This should also remove flakiness due to additional test output coming from front-end context async wrt extension test output.
Attachments
Patch (8.53 KB, patch)
2012-07-09 10:46 PDT, Andrey Kosyakov 		vsevik: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andrey Kosyakov
Comment 1 2012-07-09 10:46:39 PDT
Created attachment 151274 [details] Patch
Andrey Adaikin
Comment 2 2012-07-10 04:23:49 PDT
Comment on attachment 151274 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=151274&action=review > LayoutTests/inspector/extensions/extensions-panel.html:72 > + output("panel width mismatch, outer: " + result.width + "inner:" + window.innerWidth); "inner:" -> ", inner:" > LayoutTests/inspector/extensions/extensions-panel.html:74 > + output("panel height mismatch, outer: " + result.height + "inner:" + window.innerHeight); ditto
Andrey Kosyakov
Comment 3 2012-07-10 05:25:06 PDT
Committed r122216: <http://trac.webkit.org/changeset/122216>
Note You need to log in before you can comment on or make changes to this bug.