Bug 240769 – WebInspector: Regression(r292754) Uncaught Exception: TypeError: undefined is not an object (evaluating 'this.ownerDocument.documentURL')

• Component: Web Inspector
• Status: RESOLVED FIXED

[Patrick Angle]

Uncaught Exception in Web Inspector. Seems like I missed updating the ownerDocument somewhere :/

Steps to Reproduce:
1. Inspect https://patrickangle-static.s3.amazonaws.com/demos/grids/index.html
2. Reload (from origin seems more reliable repro?)

Uncaught Exceptions:
-----------------------
 - TypeError: undefined is not an object (evaluating 'this.ownerDocument.documentURL') (at DOMNode.js:1257:37)
    _createLayoutOverlayColorSettingIfNeeded @ DOMNode.js:1257:37
    layoutOverlayColor @ DOMNode.js:706:54
    layout @ NodeOverlayListSection.js:124:81
    _layoutSubtree @ View.js:308:20
    _layoutSubtree @ View.js:319:32
    _visitViewTreeForLayout @ View.js:378:36
-----------------------

Notes:
Inspected URL:        https://patrickangle-static.s3.amazonaws.com/demos/grids/index.html
Loading completed:    true
Frontend User Agent:  Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko)

[Radar WebKit Bug Importer]

<rdar://problem/93715304>

[Patrick Angle]

The frontend representation of the DOM tree does not match the backend representation. I suspect this is not the only bad thing happening as a result.

Preparing a revert for the caused-by and its followup since the changes were relatively self-contained and the bug they were fixing is less awful than the new results (console inspection of nodes that are detached and reattached without getting the node from the page again).

[Patrick Angle]

Pull request: https://github.com/WebKit/WebKit/pull/890

[EWS]

Committed r294621 (250847@main): <https://commits.webkit.org/250847@main>

Reviewed commits have been landed. Closing PR #890 and removing active labels.