Closed
Bug 1373139
Opened 7 years ago
Closed 7 years ago
Intermittent browser/components/resistfingerprinting/test/browser/browser_roundedWindow_open_mid.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N)
Categories
(Core :: Security, defect, P3)
Core
Security
Tracking
()
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: cfu)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=106913324&repo=autoland https://queue.taskcluster.net/v1/task/fotLLTXnSCqagt-0r5TRXw/runs/0/artifacts/public/logs/live_backing.log
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 7•7 years ago
|
||
Ethan: can someone on your team look into making this test more reliable?
Flags: needinfo?(ettseng)
Priority: -- → P3
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Assignee: nobody → tihuang
Flags: needinfo?(ettseng)
Comment 9•7 years ago
|
||
Tim knows the root cause. He just needs to figure out an ideal solution.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•7 years ago
|
Assignee: tihuang → cfu
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8918712 -
Flags: review?(tihuang)
Attachment #8918712 -
Flags: review?(bugs)
Assignee | ||
Comment 13•7 years ago
|
||
Number of opened windows: 8 -> 5, saving up to 37.5% of execution time.
Comment 14•7 years ago
|
||
mozreview-review |
Comment on attachment 8918712 [details] Bug 1373139 - Split browser/components/resistfingerprinting/test/browser/browser_roundedWindow_open_mid.js. https://reviewboard.mozilla.org/r/189522/#review195060 (MozReview is so broken when reviewing patches which remove files. I guess you could have renamed the file and add the boolean param, and then copy the file and change the boolean param. oh well, this is testing only, so fine)
Attachment #8918712 -
Flags: review?(bugs) → review+
Comment 15•7 years ago
|
||
mozreview-review |
Comment on attachment 8918712 [details] Bug 1373139 - Split browser/components/resistfingerprinting/test/browser/browser_roundedWindow_open_mid.js. https://reviewboard.mozilla.org/r/189522/#review195218
Attachment #8918712 -
Flags: review?(tihuang) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 16•7 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again. hg error in cmd: hg rebase -s 4b99359e45ab -d 9edd3c8a941d: rebasing 427008:4b99359e45ab "Bug 1373139 - Split browser/components/resistfingerprinting/test/browser/browser_roundedWindow_open_mid.js. r=smaug,timhuang" (tip) merging browser/components/resistfingerprinting/test/browser/browser.ini warning: conflicts while merging browser/components/resistfingerprinting/test/browser/browser.ini! (edit, then use 'hg resolve --mark') unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment 17•7 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again. hg error in cmd: hg rebase -s 4b99359e45ab -d 334fad658d2c: rebasing 427008:4b99359e45ab "Bug 1373139 - Split browser/components/resistfingerprinting/test/browser/browser_roundedWindow_open_mid.js. r=smaug,timhuang" merging browser/components/resistfingerprinting/test/browser/browser.ini warning: conflicts while merging browser/components/resistfingerprinting/test/browser/browser.ini! (edit, then use 'hg resolve --mark') unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(cfu)
Keywords: checkin-needed
Comment 20•7 years ago
|
||
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/43e31e001721 Split browser/components/resistfingerprinting/test/browser/browser_roundedWindow_open_mid.js. r=smaug,timhuang
Keywords: checkin-needed
Comment 21•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/43e31e001721
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in
before you can comment on or make changes to this bug.
Description
•