Debugger Popover does not work when paused in Worker Steps to Reproduce: 1. Pause in a worker's onmessage 2. Hover over something like `event` => No popover Notes: • Wrong target seems to be getting used for popover related code.
Created attachment 354045 [details] [PATCH] Proposed Fix
Comment on attachment 354045 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=354045&action=review > Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.js:1728 > + let target = WI.debuggerManager.activeCallFrame ? WI.debuggerManager.activeCallFrame.target : this.target; NIT: considering that `WI.debuggerManager.activeCallFrame` is used in a few places in this function, making it into a separate variable seems reasonable (e.g. `activeCallFrame` or even just `callFrame`)
(In reply to Devin Rousso from comment #2) Forgot, r=me :P
Comment on attachment 354045 [details] [PATCH] Proposed Fix Clearing flags on attachment: 354045 Committed r238003: <https://trac.webkit.org/changeset/238003>
All reviewed patches have been landed. Closing bug.
<rdar://problem/45924333>