[Webkit-unassigned] [Bug 143225] Web Inspector: debugger pause reason not updated when stepping after initial pause

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=143225

--- Comment #3 from Brian Burg <burg at cs.washington.edu> ---
(In reply to comment #2)
> I deliberately wanted the current behavior. When stepping, I want to know
> the original pause reason. For a few reasons:
> 
>   - if you step around a bit, you will always be able to jump back to the
> original pause location

It's weird that there's a jump button but the message can't be clicked on. Showing an inline marker (like for errors) would be more consistent, as would linkifying the error message.

>   - if the original pause was on a breakpoint, you can edit/disable that
> breakpoint right from the pause reason section no matter where you are in
> source code

I think the current behavior is good in this respect.

> There does exist a small problem in the implementation right now though. If
> you pause on a breakpoint, hit continue, and quickly pause on another
> breakpoint, it won't update to the new breakpoint. That is because of the
> way our Paused / Resumed events work in the frontend. So there is room to
> improve this here. That said, we could probably work around this in the
> frontend if the user definitely did a "continue" and paused soon after.

Maybe keep a timeout flag in DebuggerManager and reset it if 'continue' is requested from UI.

PS. Go back on vacation! :-)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150330/65cd463c/attachment-0002.html>