[Webkit-unassigned] [Bug 151112] Web Inspector: Timeline Recording across page navigations behaves poorly

[bugzilla-daemon at webkit.org]

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

Timothy Hatcher <timothy at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #265207|review?, commit-queue?      |review-
              Flags|                            |

--- Comment #5 from Timothy Hatcher <timothy at apple.com> ---
Comment on attachment 265207
  --> https://bugs.webkit.org/attachment.cgi?id=265207
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=265207&action=review

> Source/WebCore/inspector/InspectorPageAgent.cpp:-810
> -    if (frame.isMainFrame()) {
> -        auto stopwatch = m_environment.executionStopwatch();
> -        stopwatch->reset();
> -        stopwatch->start();
> -    }

Do we need to start the stopwatch if it isn't already started? I made this change to make timestamps work in the Network tab, when Timeline recoding isn't always happening. I suspect those timestamps are always zero now.

-- 
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/20151110/95475eea/attachment.html>