Last Comment Bug 705156 - embedding/test/test_window_open_units.html is flaky
: embedding/test/test_window_open_units.html is flaky
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla11
Assigned To: Mounir Lamouri (:mounir)
: Andrew Overholt [:overholt]
Depends on:
Blocks: 594140
  Show dependency treegraph
Reported: 2011-11-24 08:57 PST by Mounir Lamouri (:mounir)
Modified: 2011-11-25 02:02 PST (History)
1 user (show)
mounir: in‑testsuite+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch v1 (1.72 KB, patch)
2011-11-24 16:34 PST, Mounir Lamouri (:mounir)
roc: review+
Details | Diff | Splinter Review

Description User image Mounir Lamouri (:mounir) 2011-11-24 08:57:00 PST
When I push my WebSMS patch queue to try I got this perma-orange on Windows 7 Opt:
40 ERROR TEST-UNEXPECTED-FAIL | /tests/embedding/test/test_window_open_units.html | wrong width - got 398, expected 400
41 ERROR TEST-UNEXPECTED-FAIL | /tests/embedding/test/test_window_open_units.html | wrong height - got 198, expected 200 

I did look in bugzilla to see if there was known failures related to that tests and it happen that other people (bug 626064 and bug 661877). Both got the same perma-orange.

I believe the test is flaky and for some reasons those bugs are triggering the issue. According to bug 661877 it might be related to directories and like Jan, I'm adding a new test directory inside of dom/.

Is it correct to assume that the window size will be exactly what we are asking for? Should we look for a size "around" the one we did request (with delta=2 for example)? Otherwise, roc, what would you propose to fix that test?

Note: this is currently blocking me from pushing a part of WebSMS implementation to mozilla-central.
Comment 1 User image Mounir Lamouri (:mounir) 2011-11-24 16:34:35 PST
Created attachment 576839 [details] [diff] [review]
Patch v1
Comment 2 User image Mounir Lamouri (:mounir) 2011-11-25 02:02:32 PST

Note You need to log in before you can comment on or make changes to this bug.