Bug 90799 - Web Inspector: [Extensions API] simplify panel size check in extensions-panel.html
Summary: Web Inspector: [Extensions API] simplify panel size check in extensions-panel...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andrey Kosyakov
URL:
Keywords:
Depends on:
Blocks: 89333 89790
  Show dependency tree / graph
 
Reported: 2012-07-09 10:37 PDT by Andrey Kosyakov
Modified: 2012-07-10 05:25 PDT (History)
8 users (show)

See Also:

Attachments
Patch (8.53 KB, patch)
2012-07-09 10:46 PDT, Andrey Kosyakov 		vsevik: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Kosyakov 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.
Comment 1 Andrey Kosyakov 2012-07-09 10:46:39 PDT 
Created attachment 151274 [details]
Patch
Comment 2 Andrey Adaikin 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
Comment 3 Andrey Kosyakov 2012-07-10 05:25:06 PDT 
Committed r122216: <http://trac.webkit.org/changeset/122216>