Closed
Bug 1209521
Opened 9 years ago
Closed 9 years ago
Re-enable browser_cookies_exceptions.js on Linux
Categories
(Firefox :: Settings UI, defect)
Firefox
Settings UI
Tracking
()
RESOLVED
FIXED
Firefox 44
Tracking | Status | |
---|---|---|
firefox42 | --- | unaffected |
firefox43 | --- | fixed |
firefox44 | --- | fixed |
People
(Reporter: jaws, Assigned: jaws)
References
Details
Attachments
(1 file)
1.74 KB,
patch
|
Gijs
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
browser_cookies_exceptions.js was disabled on Linux for taking too long. We could split up the test, use requestLongerTimeout, or find what is taking it too long. It's disabled in the interim.
Assignee | ||
Comment 1•9 years ago
|
||
Try push with requestLongerTimeout(2),
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1f7549052d4a
Assignee | ||
Comment 2•9 years ago
|
||
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Attachment #8667641 -
Flags: review?(gijskruitbosch+bugs)
Comment 3•9 years ago
|
||
Comment on attachment 8667641 [details] [diff] [review]
Patch
Review of attachment 8667641 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks!
Attachment #8667641 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Comment 5•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8667641 [details] [diff] [review]
Patch
Approval Request Comment
[Feature/regressing bug #]: follow-up to bug 1203253
[User impact if declined]: none, tests only
[Describe test coverage new/current, TreeHerder]: only automated tests, landed on m-c
[Risks and why]: none
[String/UUID change made/needed]: none
Attachment #8667641 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
status-firefox43:
--- → affected
Updated•9 years ago
|
status-firefox42:
--- → unaffected
Comment 7•9 years ago
|
||
Comment on attachment 8667641 [details] [diff] [review]
Patch
Test-only change; ok to uplift to aurora
Attachment #8667641 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 8•9 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•