Closed Bug 1297518 Opened 5 years ago Closed 5 years ago
_bug1254766 .html | Should not import bad css - didn't expect "hidden", but got it
58 bytes, text/x-review-board-request
Working on this bug right now
Assignee: nobody → dlee
Status: NEW → ASSIGNED
The root cause is that in certain platform(Android in this case), http request, response may take longer than expect. In test_gethash.html and test_bug1254766.html we will use |gethash.sjs| to simulate gethash server, but if the response takes long than the timeout threshold, then the testcase will fail. I think we don't have to care about gethash timeout for testcases, so the solution here is just increase the timeout value.  https://dxr.mozilla.org/mozilla-central/source/toolkit/components/url-classifier/nsUrlClassifierHashCompleter.js#358
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Comment on attachment 8787131 [details] Bug 1297518 - Increase gethash timeout for testcases. Brilliant finding! Thanks!
Attachment #8787131 - Flags: review?(hchang) → feedback+
Comment on attachment 8787131 [details] Bug 1297518 - Increase gethash timeout for testcases. https://reviewboard.mozilla.org/r/75990/#review74874
Attachment #8787131 - Flags: review?(gpascutto) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/223b9a23f373 Increase gethash timeout for testcases. r=gcp
You need to log in before you can comment on or make changes to this bug.