Closed
Bug 1305993
Opened 8 years ago
Closed 8 years ago
Intermittent dom/security/test/hsts/browser_hsts-priming_main.js | Test timed out -
Categories
(Core :: DOM: Security, defect, P3)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox50 | --- | unaffected |
firefox51 | --- | fixed |
firefox52 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: kmckinley)
References
Details
(Keywords: intermittent-failure, Whiteboard: [domsecurity-intermittent])
Attachments
(1 file)
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Blocks: 1246540
status-firefox50:
--- → unaffected
status-firefox51:
--- → affected
status-firefox52:
--- → affected
Component: DOM → DOM: Security
Flags: needinfo?(kmckinley)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•8 years ago
|
||
Breaking up the tests to prevent the timeout occurring.
Assignee: nobody → kmckinley
Status: NEW → ASSIGNED
Flags: needinfo?(kmckinley)
Comment 4•8 years ago
|
||
mozreview-review |
Comment on attachment 8798762 [details]
Bug 1305993 - Break tests up to avoid timeouts
https://reviewboard.mozilla.org/r/84174/#review82790
::: dom/security/test/hsts/browser_hsts-priming_allow_active.js:14
(Diff revision 1)
> +add_task(function*() {
> + //jscs:enable
> + Services.obs.addObserver(Observer, "console-api-log-event", false);
> + Services.obs.addObserver(Observer, "http-on-examine-response", false);
> + registerCleanupFunction(do_cleanup);
> + requestLongerTimeout(4);
Do you still ned the requestLongerTimeOut(4) in every *.js file?
Comment 5•8 years ago
|
||
Comment on attachment 8798762 [details]
Bug 1305993 - Break tests up to avoid timeouts
It seems I still can't operate mozreview - anyway, r=me
Attachment #8798762 -
Flags: review?(ckerschb) → review+
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Priority: -- → P3
Whiteboard: [domsecurity-intermittent]
Comment hidden (mozreview-request) |
Comment 8•8 years ago
|
||
Comment on attachment 8798762 [details]
Bug 1305993 - Break tests up to avoid timeouts
Kate, I think I already r+ed those changes, right? I don't see anything changed. I assume mozreview just flagged me again, right? Anyway, If you need a review let me know.
Attachment #8798762 -
Flags: review?(ckerschb) → review+
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 10•8 years ago
|
||
mozreview-review |
Comment on attachment 8798762 [details]
Bug 1305993 - Break tests up to avoid timeouts
https://reviewboard.mozilla.org/r/84174/#review85444
Attachment #8798762 -
Flags: review+
Comment 11•8 years ago
|
||
Pushed by philringnalda@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/47b58740f054
Break tests up to avoid timeouts r=philor
Keywords: checkin-needed
Comment 12•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Updated•8 years ago
|
Whiteboard: [domsecurity-intermittent] → [domsecurity-intermittent][checkin-needed-aurora]
Comment 13•8 years ago
|
||
bugherder uplift |
Whiteboard: [domsecurity-intermittent][checkin-needed-aurora] → [domsecurity-intermittent]
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•