Bug 192478

Summary: Web Inspector: remove DOM.BackendNodeId and associated commands/events
Product: WebKit Reporter: Devin Rousso <hi>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ews-watchlist, hi, inspector-bugzilla-changes, joepeck, keith_miller, mark.lam, mattbaker, msaboff, saam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Devin Rousso
Reported 2018-12-06 15:41:59 PST
This code hasn't been used in WebInspector for many years now. We should remove it.
Attachments
Patch (39.38 KB, patch)
2018-12-06 15:46 PST, Devin Rousso 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2018-12-06 15:46:50 PST
Created attachment 356759 [details] Patch
EWS Watchlist
Comment 2 2018-12-06 15:48:26 PST
This patch modifies the inspector protocol. Please ensure that any frontend changes appropriately use feature checks for new protocol features.
Matt Baker
Comment 3 2018-12-13 14:54:07 PST
Comment on attachment 356759 [details] Patch rs=me
WebKit Commit Bot
Comment 4 2018-12-13 15:20:42 PST
Comment on attachment 356759 [details] Patch Clearing flags on attachment: 356759 Committed r239183: <https://trac.webkit.org/changeset/239183>
WebKit Commit Bot
Comment 5 2018-12-13 15:20:44 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2018-12-13 15:21:26 PST
<rdar://problem/46711547>
Note You need to log in before you can comment on or make changes to this bug.