Closed
Bug 1184123
Opened 9 years ago
Closed 5 years ago
Intermittent browser_identity_UI.js | uncaught exception - Error: operation not possible on dead CPOW at chrome://global/content/browser-child.js:106 | Test timed out
Categories
(Firefox :: Security, defect, P5)
Tracking
()
People
(Reporter: RyanVM, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell unknown])
Attachments
(1 file)
1.11 KB,
patch
|
mconley
:
review-
|
Details | Diff | Splinter Review |
21:05:22 INFO - 429 INFO TEST-PASS | browser/base/content/test/general/browser_identity_UI.js | mode matched
21:05:22 INFO - 430 INFO TEST-PASS | browser/base/content/test/general/browser_identity_UI.js | location matches for test #7 (IP address)
21:05:22 INFO - 431 INFO TEST-PASS | browser/base/content/test/general/browser_identity_UI.js | effectiveHost matches for test #7 (IP address)
21:05:22 INFO - 432 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_identity_UI.js | uncaught exception - Error: operation not possible on dead CPOW at chrome://global/content/browser-child.js:106
21:05:22 INFO - Stack trace:
21:05:22 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:simpletestOnerror:1518
21:05:22 INFO - null:null:0
21:05:22 INFO - JavaScript error: chrome://global/content/browser-child.js, line 106: Error: operation not possible on dead CPOW
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•9 years ago
|
Blocks: e10s-tests
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Intermittent Failures Robot) |
Reporter | ||
Comment 38•8 years ago
|
||
Looks like this is dying on line 119.
https://dxr.mozilla.org/mozilla-central/source/browser/base/content/test/general/browser_identity_UI.js#119
> content.location.reload(true);
Maybe another good mentored bug for Kevin?
status-firefox49:
--- → wontfix
status-firefox50:
--- → affected
status-firefox51:
--- → affected
Flags: needinfo?(kevchan85)
Comment 39•8 years ago
|
||
Hello,
Would I change this to contentDocument.location.reload(true)?
Comment 40•8 years ago
|
||
I changed line 119... is this right?
Flags: needinfo?(kevchan85)
Attachment #8791023 -
Flags: review?(ryanvm)
Reporter | ||
Comment 41•8 years ago
|
||
Comment on attachment 8791023 [details] [diff] [review]
bug1184123_changed_line119.diff
I'd redirect this review request to Paolo, but BMO says he's away. Matt, can you please take a look?
Attachment #8791023 -
Flags: review?(ryanvm) → review?(MattN+bmo)
Comment 42•8 years ago
|
||
Comment on attachment 8791023 [details] [diff] [review]
bug1184123_changed_line119.diff
Review of attachment 8791023 [details] [diff] [review]:
-----------------------------------------------------------------
Hey kevin! Thanks - I think there's a little tweak we need to make to ensure the same behaviour. See below.
::: browser/base/content/test/general/browser_identity_UI.js
@@ +115,5 @@
> gCheckETLD = false;
> gTestDesc = "#" + gCurrentTestIndex + " (" + gCurrentTest.name + " without eTLD in identity icon label)";
> if (!gForward)
> gTestDesc += " (second time)";
> + gBrowser.reload();
The "true" in reload(true) tells the browser to skip the network and history cache. We should make sure we're doing the same thing here.
I think you can do that with something like:
const reloadFlags = nsIWebNavigation.LOAD_FLAGS_BYPASS_PROXY | nsIWebNavigation.LOAD_FLAGS_BYPASS_CACHE;
gBrowser.selectedBrowser.reloadWithFlags(reloadFlags);
Attachment #8791023 -
Flags: review?(MattN+bmo) → review-
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Whiteboard: [stockwell unknown]
Comment hidden (Intermittent Failures Robot) |
Updated•5 years ago
|
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•