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)

x86
Linux
defect

Tracking

()

RESOLVED WORKSFORME
Tracking Status
e10s + ---
firefox49 --- wontfix
firefox50 --- affected
firefox51 --- affected

People

(Reporter: RyanVM, Unassigned)

References

(Blocks 1 open bug)

Details

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

Attachments

(1 file)

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
Intermittent e10s test failure
Priority: -- → P5
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?
Flags: needinfo?(kevchan85)
Hello, Would I change this to contentDocument.location.reload(true)?
I changed line 119... is this right?
Flags: needinfo?(kevchan85)
Attachment #8791023 - Flags: review?(ryanvm)
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 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-
Whiteboard: [stockwell unknown]
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.

Attachment

General

Creator:
Created:
Updated:
Size: