Closed Bug 1276418 Opened 8 years ago Closed 8 years ago

Intermittent e10s browser_inspector_initialization.js | Uncaught exception - at :0 - Error: operation not possible on dead CPOW

Categories

(DevTools :: Inspector, defect, P2)

Product:
DevTools
Component:
Inspector
Type:
defect

Tracking

(e10s+, firefox49 wontfix, firefox50 fixed, firefox51 fixed, firefox52 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 52
Tracking Flags:
Tracking Status
e10s + ---
firefox49 --- wontfix
firefox50 --- fixed
firefox51 --- fixed
firefox52 --- fixed

People

(Reporter: aryx, Assigned: pbro)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Bug 1276418 - Remove getNode and all references to it in inspector tests;
58 bytes, text/x-review-board-request
gl
: review+
Details
tracking-e10s: --- → ?
Inspector bug triage (filter on CLIMBING SHOES).
Priority: -- → P2
Patrick, looks like you've cleaned up CPOW usage in this test previously. This particular failure is still one of the more frequent dead CPOW oranges we're hitting, though :(. Can you please take a look when you get a chance?
status-firefox49: affected → wontfix
status-firefox50: --- → affected
status-firefox51: --- → affected
Flags: needinfo?(pbrosset)
Taking this one, the CPOW is quite obvious here. The test still uses getNode which access DOM node in the content page directly. I thought we had already eradicated them all, but some still remain.
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
Flags: needinfo?(pbrosset)
Comment on attachment 8791980 [details]
Bug 1276418 - Remove getNode and all references to it in inspector tests;

https://reviewboard.mozilla.org/r/79254/#review77998

::: devtools/client/inspector/layout/test/head.js:22
(Diff revision 1)
> -}
> -
> -/**
>   * Highlight a node and set the inspector's current selection to the node or
>   * the first match of the given css selector.
> - * @param {String|DOMNode} nodeOrSelector
> + * @param {String} selectorOrNodeFront

Should be {String|NodeFront}
Attachment #8791980 - Flags: review?(gl) → review+
Thanks for the review.
Here's a green try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f695f7accecd&group_state=expanded
(eslint is red, but I've fixed the issue locally since then).
Going to land this.
Pushed by pbrosset@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/690b85efea25
Remove getNode and all references to it in inspector tests; r=gl
https://hg.mozilla.org/mozilla-central/rev/690b85efea25
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Interestingly, both browser_styleinspector_csslogic-content-stylesheets.js and browser_inspector_initialization.js were disabled on debug e10s for docshell leaks. I wonder if that's still true after their deCPOWification.
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: