Closed Bug 1310092 Opened 4 years ago Closed 4 years ago
_favicon _user Context Id .js | Test timed out -
58 bytes, text/x-review-board-request
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=37564350&repo=mozilla-inbound https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-win64-pgo/1476365478/mozilla-inbound_win8_64_test_pgo-mochitest-e10s-browser-chrome-6-bm111-tests1-windows-build93.txt.gz
This test was added in bug 1277803 and started failing almost right away.
Tim already got the root cause and has a solution in mind. :)
Assignee: nobody → tihuang
Priority: -- → P2
There are two possibilities that could trigger this time out problem: 1. The delayed completion of the second favicon load of the first test might leave an entry in the cache after the second test started. So, the event 'http-on-modify-request' will not be sent, then the test will be blocked by waiting for this event. To resolve this, I will use different sites for these two tests, which means they will not share the same entry during the test. 2. The Places favicon load finished too quick to be caught by the waitOnFaviconLoaded(). To address this, I will start the observer earlier before the tab has been created.
Comment on attachment 8807075 [details] Bug 1310092 - Fixing the timeout problem of the favicon test for userContextIds. https://reviewboard.mozilla.org/r/90354/#review90032
Attachment #8807075 - Flags: review?(amarchesini) → review+
Try looks good.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/da35d5511e92 Fixing the timeout problem of the favicon test for userContextIds. r=baku
You need to log in before you can comment on or make changes to this bug.