Intermittent tier2 browser/components/urlbar/tests/legacy/browser_urlbarDecode.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. -
Categories
(Firefox :: Address Bar, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: mak)
References
Details
(Keywords: intermittent-failure, Whiteboard: [retriggered] [stockwell needswork])
Attachments
(1 file)
Filed by: nerli [at] mozilla.com
https://treeherder.mozilla.org/logviewer.html#?job_id=221368032&repo=mozilla-central
19:56:07 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | Visible action - "Visit" == "Visit" -
19:56:07 INFO - Leaving test bound injectJSON
19:56:07 INFO - Entering test bound losslessDecode
19:56:07 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | The string displayed in the textbox should not be escaped - "example.com/アイウエオ" == "example.com/アイウエオ" -
19:56:07 INFO - Leaving test bound losslessDecode
19:56:07 INFO - Entering test bound actionURILosslessDecode
19:56:07 INFO - Disable popup animation. Change dontAnimate value from true to true.
19:56:07 INFO - Buffered messages logged at 19:56:07
19:56:07 INFO - Restore popup dontAnimate value to true
19:56:07 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | visiturl action URI should be in the urlbar - "visiturl" == "visiturl" -
19:56:07 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | The string displayed in the textbox should not be escaped - "example.com/アイウエオ" == "example.com/アイウエオ" -
19:56:07 INFO - Leaving test bound actionURILosslessDecode
19:56:07 INFO - Buffered messages finished
19:56:07 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/urlbar/browser_urlbarDecode.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. -
19:56:07 INFO - GECKO(4068) | MEMORY STAT | vsize 2352MB | vsizeMaxContiguous 131589872MB | residentFast 457MB | heapAllocated 100MB
19:56:07 INFO - TEST-OK | browser/base/content/test/urlbar/browser_urlbarDecode.js | took 194085ms
19:56:07 INFO - GECKO(4068) | ++DOCSHELL 000001996F077000 == 1 [pid = 3608] [id = {b9128b05-d003-4cf3-a3da-7fb416ed33fc}]
19:56:07 INFO - GECKO(4068) | ++DOMWINDOW == 1 (000001996F0A7000) [pid = 3608] [serial = 121] [outer = 0000000000000000]
19:56:07 INFO - GECKO(4068) | ++DOMWINDOW == 2 (000001996F0D6000) [pid = 3608] [serial = 122] [outer = 000001996F0A7000]
19:56:07 INFO - GECKO(4068) | ++DOMWINDOW == 3 (00000199732B3C00) [pid = 3608] [serial = 123] [outer = 000001996F0A7000]
19:56:07 INFO - checking window state
Comment 1•5 years ago
|
||
There are 16 total failures in the last 7 days on windows10-64-ccov debug.
Recent failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=221632021&repo=mozilla-central&lineNumber=9577
23:33:53 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | Visible action - "Visit" == "Visit" -
23:33:53 INFO - Leaving test bound injectJSON
23:33:53 INFO - Entering test bound losslessDecode
23:33:53 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | The string displayed in the textbox should not be escaped - "example.com/アイウエオ" == "example.com/アイウエオ" -
23:33:53 INFO - Leaving test bound losslessDecode
23:33:53 INFO - Entering test bound actionURILosslessDecode
23:33:53 INFO - Buffered messages logged at 23:33:51
23:33:53 INFO - Disable popup animation. Change dontAnimate value from true to true.
23:33:53 INFO - Buffered messages logged at 23:33:53
23:33:53 INFO - Restore popup dontAnimate value to true
23:33:53 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | visiturl action URI should be in the urlbar - "visiturl" == "visiturl" -
23:33:53 INFO - TEST-PASS | browser/base/content/test/urlbar/browser_urlbarDecode.js | The string displayed in the textbox should not be escaped - "example.com/アイウエオ" == "example.com/アイウエオ" -
23:33:53 INFO - Leaving test bound actionURILosslessDecode
23:33:53 INFO - Buffered messages finished
23:33:53 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/urlbar/browser_urlbarDecode.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. -
23:33:53 INFO - GECKO(5692) | MEMORY STAT | vsize 2393MB | vsizeMaxContiguous 130603721MB | residentFast 490MB | heapAllocated 117MB
23:33:53 INFO - TEST-OK | browser/base/content/test/urlbar/browser_urlbarDecode.js | took 189447ms
23:33:53 INFO - GECKO(5692) | ++DOCSHELL 000001C424475800 == 1 [pid = 8324] [id = {9c8f7d44-78fd-425a-8272-bded8bda2b94}]
23:33:53 INFO - GECKO(5692) | ++DOMWINDOW == 1 (000001C4244C9400) [pid = 8324] [serial = 121] [outer = 0000000000000000]
23:33:53 INFO - GECKO(5692) | ++DOMWINDOW == 2 (000001C4244CD000) [pid = 8324] [serial = 122] [outer = 000001C4244C9400]
23:33:53 INFO - checking window state
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 4•5 years ago
|
||
There are 38 failures in the last 7 days. All of them on windows10-64-ccov -> debug.
As I can see, it could be caused by the landing of Bug 1513241 or Bug 1514664.
Marco, could you please take a look?
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment 8•5 years ago
|
||
There are 44 failures in the last 7 days.
They occur on windows10-64-ccov debug.
Recent failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=224904031&repo=mozilla-central&lineNumber=13045
mak: Hi, Do you have any updates on this? Thank you!
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 10•5 years ago
|
||
From the log
17:29:00 INFO - TEST-START | browser/base/content/test/urlbar/browser_urlbarDecode.js
17:32:18 INFO - Entering test bound injectJSON
^ This is the first test, we spent 3 minutes just to START executing the first test.
17:32:18 INFO - Leaving test bound actionURILosslessDecode
^ Here we exit from the LAST test.
Globally the test took less than 1 second, so apparently this is not a problem with the test, it's the test harness that for some reason takes a long time to setup, or something else is happening in the meanwhile.
In that amount of time there isn't much logged, I see some telemetry probes, a vacuum manager call trying to vacuum Places, something in dom quota, some GC.
Off-hand it looks like idle-daily hit us, based on the VacuumManager failure... That could mean something broke our special setting of "idle.lastDailyNotification" that we do here:
https://searchfox.org/mozilla-central/rev/9eb30227b21e0aa40d51d9f9b08bb0b113c5fadb/testing/mochitest/runtests.py#1882
Maybe we started running tests differently, or we run tests differently for ccov? I don't know, maybe Marco Castelluccio knows if we take a different path for these tests?
I don't have much to suggest here, we should figure out why idle-daily is hitting us
Assignee | ||
Comment 11•5 years ago
•
|
||
Maybe we should move those prefs to testing/profiles/perf/user.js or testing/profiles/unittest-required/user.js ? Oh, there is also testing/profiles/common/user.js.
Assignee | ||
Comment 12•5 years ago
|
||
Wes, what's the right location today for setting a pref for xpcshell and mochitests?
Assignee | ||
Comment 13•5 years ago
|
||
what's the difference between common and unittest-required?
Assignee | ||
Comment 14•5 years ago
|
||
Standard8 made me notice the only documentation we have is also completely out of date: https://developer.mozilla.org/en-US/docs/Mozilla/QA/Automated_testing#Need_to_set_preferences_for_test-suites
Comment 15•5 years ago
|
||
(In reply to Marco Bonardo [::mak] from comment #13)
what's the difference between common and unittest-required?
You can see which profiles apply to which test suites by inspecting testing/profiles/profiles.json. The current structure isn't very well organized, and is mostly the way it is to get all the test harnesses using it. We should really go through and audit all the prefs and try to group them into more logical profile by function (e.g one profile to disable network connections, one profile to turn off background tasks, etc).
Anyway, if you want to set these prefs for mochitest+xpcshell and only mochitest+xpcshell, I'd recommend adding them to both 'unittest-required' and 'xpcshell' for now.
Assignee | ||
Comment 16•5 years ago
|
||
Unfortunately the idle daily pref only accepts numbers between 0 and now, so it's hard to disable it from user.js, because we can't set a time in the future. It could even make sense, if the user's time is bogus we don't want to skip this work.
runtests.py sets the time to now (in seconds), I wonder if it's possible due to timers resolution that python time ends up being > then PR_NOW().
So there are 2 possibile solutions:
- allow to use a negative value to disable idle-daily
- Change python to subtract a few seconds (or even 1 hour for possible DST problems) to the time, to exclude timers.
Since it looks like the will is to move more prefs to specific user.js files, solution 1 sounds good. The likelihook we or the user sets a negative value here is quite low. and anyway I can add the idle. prefs to about:support
Assignee | ||
Comment 17•5 years ago
|
||
Comment 18•5 years ago
|
||
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/bac51381060d Disable idle-daily notifications in tests through user.js files. r=Standard8
Comment 19•5 years ago
|
||
Backed out changeset bac51381060d (Bug 1519538) for Linting failure in config.py
Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=bac51381060dd384e65e454e2411eb2731721b13
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=227209241&repo=autoland&lineNumber=272
Backout: https://hg.mozilla.org/integration/autoland/rev/42b90a18e239081b3e1a9871ab4ab1882d73a709
Assignee | ||
Comment 20•5 years ago
|
||
Apparently nor pushing nor mozreview run pylint...
Comment 21•5 years ago
|
||
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/58b16c9dabfd Disable idle-daily notifications in tests through user.js files. r=Standard8
Comment 22•5 years ago
|
||
Backed out changeset bac51381060d (bug 1519538) for causing multiple failures on Windows CLOSED TREE
Backout revision https://hg.mozilla.org/integration/autoland/rev/8a05a3f36cfe5c577d906f4e6fcf1f47c8650d8f
Log failure https://treeherder.mozilla.org/logviewer.html#?job_id=227219543&repo=autoland
https://treeherder.mozilla.org/logviewer.html#?job_id=227219523&repo=autoland
:mak Could you please take a look?
Assignee | ||
Comment 23•5 years ago
|
||
Comment 24•5 years ago
|
||
Backout by csabou@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3d8811e611d9 Backed out 2 changesets for linting failures. CLOSED TREE
Comment 25•5 years ago
|
||
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/b9fad54bae14 Disable idle-daily notifications in tests through user.js files. r=Standard8
Comment 26•5 years ago
|
||
bugherder |
Description
•