Closed
Bug 1399648
Opened 7 years ago
Closed 7 years ago
Get browser_aboutHome.js passing on both about:homes with activity stream or not
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
RESOLVED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
Attachments
(1 file)
+++ This bug was initially created as a clone of Bug #1396274 +++
Some stuff will probably need to be skipped and others fixed
Assignee | ||
Comment 1•7 years ago
|
||
I split out the imitation test which actually runs fine with activity stream, and it doesn't leak. But now it's leaking about:addons: https://treeherder.mozilla.org/#/jobs?repo=try&revision=25d9b7491a405eff873fabf0b7acb89aca5d9e2e&selectedJob=130866368
Potentially there's something leaky when enabling and disabling about:home activity stream? But only intermittently? 2 of 20 win32 debug non-e10s.
I also split out every single search test and search_suggestions is still timing out, so trying request longer for just that one test:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=25d9b7491a405eff873fabf0b7acb89aca5d9e2e&selectedJob=130865769
Trying again by skipping that intermittent leak, longer timeout, as well as skipping bug 1395602:
https://hg.mozilla.org/try/rev/644648fa9a9efc99a548cca1ce64a9c218d0fc1e
Assignee: nobody → edilee
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Comment 2•7 years ago
|
||
search_suggestion was still timing out (1 in 20) after various increases in requested longer timeouts on linux32/64 opt so just skipping
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8eed62ff63c7d028eda52b89cb8d4867914738aa&selectedJob=130892363
search_composing was also timing out (1 in 80) on linux32 opt only so far.. so just skipping
https://treeherder.mozilla.org/#/jobs?repo=try&revision=60d70b73cfb10e14168c2566af3093b0d171869a&selectedJob=130898682
Latest set of test splitting and skipping:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=99402ed4f6b78e1810b466c487ef26cfa0630c49
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•7 years ago
|
||
Still waiting on mac but my try and mozreview try look good:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=99402ed4f6b78e1810b466c487ef26cfa0630c49
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fb30c23c858a
Feel free to autoland if it's good. (And if export bug 1399607 is good too)
Assignee | ||
Comment 5•7 years ago
|
||
Looks like your try from yesterday with the combined aboutHome_search tests with request(2) are timing out 1 in 16:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f6fdd141314a86c3d5e40aac8032c83629f6a11c&filter-searchStr=mac%20(bc
In particular they seem to timeout at "composing"
Not sure what that will mean for the split up tests in this patch. Maybe will need to skip _composing for osx too. Will check this in the morning: https://treeherder.mozilla.org/#/jobs?repo=try&revision=fb30c23c858a&filter-searchStr=mac%20(bc
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8908016 [details]
Bug 1399648 - Get browser_aboutHome.js passing on both about:homes with activity stream or not.
https://reviewboard.mozilla.org/r/179702/#review185112
Patch looks good, let's just wait on the finished try run to see if the search test still timeouts on Mac before we land it. Thanks!
Attachment #8908016 -
Flags: review?(usarracini) → review+
Assignee | ||
Comment 7•7 years ago
|
||
With 7 of 20 mac opt, 1 of 20 mac debug, 1 of 20 mac opt no stylo, 3 of 20 mac debug no stylo timing out for browser_aboutHome_search_suggestion.js. I guess we'll just skip the test for now. Could possibly requestLongerTimeout(4) as it did before, but there's contentSearch tests that are running.
Comment hidden (mozreview-request) |
Pushed by edilee@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f36eefa1beee
Get browser_aboutHome.js passing on both about:homes with activity stream or not. r=ursula
Comment 10•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•