Closed Bug 1402018 Opened 7 years ago Closed 6 years ago

Intermittent TEST-UNEXPECTED-TIMEOUT | toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js | Test timed out

Categories

(WebExtensions :: General, defect)

Product:
WebExtensions
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox60 fixed, firefox61 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla61
Tracking Flags:
Tracking Status
firefox60 --- fixed
firefox61 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: kmag)

References

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell unknown])

Attachments

(1 file)

backport the patch
3.52 KB, patch
kmag
: review+
RyanVM
: approval-mozilla-beta+
Details | Diff | Splinter Review 
Filed by: archaeopteryx [at] coole-files.de

https://treeherder.mozilla.org/logviewer.html#?job_id=132488835&repo=autoland

https://queue.taskcluster.net/v1/task/CHo-undUSnuFy5wV7QTU0g/runs/0/artifacts/public/logs/live_backing.log

09:01:26     INFO -  TEST-START | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js
09:06:26  WARNING -  TEST-UNEXPECTED-TIMEOUT | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js | Test timed out
09:06:26     INFO -  TEST-INFO took 300001ms
09:06:26     INFO -  >>>>>>>
09:06:26     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
09:06:26     INFO -  (xpcshell/head.js) | test run_next_test 0 pending (2)
09:06:26     INFO -  (xpcshell/head.js) | test MAIN run_test finished (2)
09:06:26     INFO -  running event loop
09:06:26     INFO -  xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js | Starting check_remote
09:06:26     INFO -  (xpcshell/head.js) | test check_remote pending (2)
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js | check_remote - [check_remote : 241] useRemoteWebExtensions matches - true == true
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js | check_remote - [check_remote : 241] testing from extension process - false == false
09:06:26     INFO -  (xpcshell/head.js) | test run_next_test 0 finished (2)
09:06:26     INFO -  (xpcshell/head.js) | test run_next_test 1 pending (2)
09:06:26     INFO -  (xpcshell/head.js) | test check_remote finished (2)
09:06:26     INFO -  xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js | Starting
09:06:26     INFO -  (xpcshell/head.js) | test pending (2)
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/addons/XPIProvider.jsm, line 1576: ReferenceError: reference to undefined property "app-temporary"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/TelemetryUtils.jsm, line 23: ReferenceError: reference to undefined property "processType"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/RemotePageManager.jsm, line 534: ReferenceError: reference to undefined property "processType"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/RemotePageManager.jsm, line 492: ReferenceError: reference to undefined property "processType"
09:06:26     INFO -  (xpcshell/head.js) | test run_next_test 1 finished (2)
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "app-temporary"" {file: "resource://gre/modules/addons/XPIProvider.jsm" line: 1576}]"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "processType"" {file: "resource://gre/modules/TelemetryUtils.jsm" line: 23}]"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "processType"" {file: "resource://gre/modules/RemotePageManager.jsm" line: 534}]"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "processType"" {file: "resource://gre/modules/RemotePageManager.jsm" line: 492}]"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/Schemas.jsm, line 40: ReferenceError: reference to undefined property "processType"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "processType"" {file: "resource://gre/modules/Schemas.jsm" line: 40}]"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/ExtensionCommon.jsm, line 1481: ReferenceError: reference to undefined property "placeholders"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/Schemas.jsm, line 2707: ReferenceError: reference to undefined property "processType"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "placeholders"" {file: "resource://gre/modules/ExtensionCommon.jsm" line: 1481}]"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "processType"" {file: "resource://gre/modules/Schemas.jsm" line: 2707}]"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/addons/XPIInstall.jsm, line 412: ReferenceError: reference to undefined property "description"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "description"" {file: "resource://gre/modules/addons/XPIInstall.jsm" line: 412}]"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/addons/XPIProvider.jsm, line 5129: ReferenceError: reference to undefined property "applySoftBlock"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "applySoftBlock"" {file: "resource://gre/modules/addons/XPIProvider.jsm" line: 5129}]"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/addons/XPIProvider.jsm, line 6301: ReferenceError: reference to undefined property "action"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "action"" {file: "resource://gre/modules/addons/XPIProvider.jsm" line: 6301}]"
09:06:26     INFO -  "Extension attached"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Expected extension version - "1.0" == "1.0"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Got expected manifest name - "en-US 1.0" == "en-US 1.0"
09:06:26     INFO -  "Restart and re-check"
09:06:26     INFO -  PID 5476 | JavaScript strict warning: resource://gre/modules/RemotePageManager.jsm, line 508: ReferenceError: reference to undefined property "processType"
09:06:26     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "processType"" {file: "resource://gre/modules/RemotePageManager.jsm" line: 508}]"
09:06:26     INFO -  "Extension attached"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Expected extension version - "1.0" == "1.0"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Got expected manifest name - "en-US 1.0" == "en-US 1.0"
09:06:26     INFO -  "Change locale to 'fr' and restart"
09:06:26     INFO -  "Extension attached"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Expected extension version - "1.0" == "1.0"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Got expected manifest name - "fr 1.0" == "fr 1.0"
09:06:26     INFO -  "Update to version 1.1"
09:06:26     INFO -  "Extension attached"
09:06:26     INFO -  TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_startup_cache.js |  - Expected extension version - "1.1" == "1.1"
09:06:26     INFO -  <<<<<<<
This appears to have stopped on trunk around 12-March, but still affects Beta pretty regularly. Do you have any idea what might have fixed this, Kris?
Flags: needinfo?(kmaglione+bmo)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #30)
> This appears to have stopped on trunk around 12-March, but still affects
> Beta pretty regularly. Do you have any idea what might have fixed this, Kris?

Probably https://hg.mozilla.org/mozilla-central/rev/2cbcfa64364bdf2f74e552f0d3e0cfafd7756ae6
Flags: needinfo?(kmaglione+bmo)
Status: NEW → RESOLVED
Closed: 6 years ago
Depends on: 1444758
Resolution: --- → FIXED
Kris, I've verified that the changeset in question backports cleanly to Beta. Is there any reason to be concerned about doing so since the code in question isn't test-only?
Attachment #8966364 - Flags: review?(kmaglione+bmo)
Assignee: nobody → kmaglione+bmo
status-firefox60: --- → affected
status-firefox61: --- → fixed
Target Milestone: --- → mozilla61
Comment on attachment 8966364 [details] [diff] [review]
backport the patch

Review of attachment 8966364 [details] [diff] [review]:
-----------------------------------------------------------------

"This was originally landed on mozilla-central as a follow-up patcht to bug 1444758."

*patch to

(In reply to Ryan VanderMeulen [:RyanVM] from comment #32)
> Kris, I've verified that the changeset in question backports cleanly to
> Beta. Is there any reason to be concerned about doing so since the code in
> question isn't test-only?

No. The changes are pretty trivial, and mostly only affect tests anyway. We almost never hit this corner case in real world usage, so even if it did somehow cause problems (which is extremely unlikely), they wouldn't show up often enough to be a concern.
Attachment #8966364 - Flags: review?(kmaglione+bmo) → review+
Comment on attachment 8966364 [details] [diff] [review]
backport the patch

Low-risk fix for some oranges. Let's get this on Beta so we're not stuck with them on ESR60 for the next year.
Attachment #8966364 - Flags: approval-mozilla-beta+
Flags: qe-verify-
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: