This includes: - double-clicking on tags/attributes/text - context menu actions - Log Element - $0
Created attachment 381320 [details] Patch
This patch modifies the inspector protocol. Please ensure that any frontend changes appropriately use feature checks for new protocol features.
Comment on attachment 381320 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381320&action=review r=me with a question > Source/WebInspectorUI/UserInterface/Views/StyleDetailsPanel.js:94 > + this.dispatchEventToListeners(WI.StyleDetailsPanel.Event.NodeChanged); I don't understand why this change is in the patch. Is it related?
Comment on attachment 381320 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381320&action=review >> Source/WebInspectorUI/UserInterface/Views/StyleDetailsPanel.js:94 >> + this.dispatchEventToListeners(WI.StyleDetailsPanel.Event.NodeChanged); > > I don't understand why this change is in the patch. Is it related? Previously, we would show the add button even if the selected node was inside a UserAgent shadow root. This isn't a good idea because a lot of times the `WI.DOMNode.prototype.appropriateSelectorFor` is just `.div`, and the newly added rule wouldn't even apply to the UserAgent shadow tree anyways, so we'd basically be creating a rule that doesn't do anything. This part of the patch is to make it so that the add button isn't even shown when the selected node is inside a UserAgent shadow tree. This particular line of code was moved because the old position (in `WI.StyleDetailsPanel.prototype.refresh`) wouldn't fire "quick" enough. It would fire _after_ the `WI.DOMNodeStyles` have refreshed, which involves three "concurrent" protocol commands (which are pretty hefty in size), which meant that the add button would sometimes be briefly visible and flash away. By this point we already know the `WI.DOMNode`, so we can determine whether the selected node is inside a UserAgent shadow tree and can act on that. Also, nobody was actually listening for `WI.StyleDetailsPanel.Event.Refreshed`, so it was fine to "replace".
Comment on attachment 381320 [details] Patch Clearing flags on attachment: 381320 Committed r251306: <https://trac.webkit.org/changeset/251306>
All reviewed patches have been landed. Closing bug.
<rdar://problem/56422672>
Comment on attachment 381320 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381320&action=review > Source/WebInspectorUI/UserInterface/Controllers/DOMManager.js:102 > + Nit: Bad newline.