[Webkit-unassigned] [Bug 144650] Web Inspector: REGRESSION(Tabs) Issues reloading a resource with breakpoints

[bugzilla-daemon at webkit.org]

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

--- Comment #3 from Joseph Pecoraro <joepeck at webkit.org> ---
I count a jumble of issues investigating this:

1. DebuggerSidebarPanel needs to clear content views and tree elements on main resource changes.
  - closeAllContentViews
  - breakpointsContentTreeOutline, remove non-global folder tree elements

2. BreakpointTreeElement restoration doesn't work
  - display source code for a WebInspector.Breakpoint represented object

3. Cookie saving is cumulative instead of replacing
  - TabContentView adds new cookie state onto its cookie instead of replacing the cookie

4. TabContentView.restoreStateFromCookie(causedByReload) parameter is never set
  - sounds like a missed opportunity, but it is always unused

5. Order of operations on WebInspector.Frame.Event.MainResourceDidChange is problematic
  - Main.js performs state restoration on MainResourceDidChange
  - NavigationSidebarPanels (Resources, Debugger, etc) perform clean up MainResourceDidChange
  - Typically, Main.js will do its work first, which may be very problematic in restoration!

So, I'm going to hesitantly attach a patch, but Tim, feel free to take over this.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150506/c40ac9c0/attachment.html>