Closed Bug 1380021 Opened 2 years ago Closed 2 years ago
Tests for showing the Page Action device sub-menu have a 5 second waits in them
59 bytes, text/x-review-board-request
In moving some code in bug 1380009, we discovered that bug 1369095 added a 5 second timeout to parts of tests that are waiting for the device sub-menu to be displayed: http://searchfox.org/mozilla-central/rev/5dadcbe55b4ddd1e448c06c77390ff6483aa009b/browser/base/content/test/urlbar/browser_page_action_menu.js#436 (as of bug 1380009 landing, that is moving to the head.js file). If I run the tests manually (MacBook Pro mid-2015), I can see that occasionally the device sub-menu is opened and then after a second or so it changes size, before the test then continues. This seems like it is could be a glitch that users will see, hence flagging as possible photon work.
I think the glitching is basically bug 1374509. The 5 seconds isn't when the relayout happens that you noticed here: (In reply to Mark Banner (:standard8) from comment #0) > If I run the tests manually (MacBook Pro mid-2015), I can see that > occasionally the device sub-menu is opened and then after a second or so it > changes size, before the test then continues. the timeout is related to when the item populates because of delayed sync initialization on startup and ensuring that's finished (for which apparently there isn't a useful event).
Depends on: 1374509
Whiteboard: [photon-structure][triage] → [reserve-photon-structure]
Bug 1384464 fixed browser_page_action_menu.js (let's hope it sticks and actually works), but browser_PageActions.js still has a 5000ms timeout: https://dxr.mozilla.org/mozilla-central/rev/ffe6cc09ccf38cca6f0e727837bbc6cb722d1e71/browser/modules/test/browser/browser_PageActions.js#1031 I think that's the only other one that needs fixing.
Assignee: nobody → adw
Status: NEW → ASSIGNED
A bunch of en-US failures... not sure what that's about. Bad tree I guess. But other than that, try looks OK.
Attachment #8910052 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8910052 [details] Bug 1380021 - Remove 5000ms timeout from browser_PageActions.js test. https://reviewboard.mozilla.org/r/181526/#review187024
Attachment #8910052 - Flags: review?(gijskruitbosch+bugs) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/c613a3d09cc1 Remove 5000ms timeout from browser_PageActions.js test. r=Gijs
You need to log in before you can comment on or make changes to this bug.