Bug 135946

Summary: Web Inspector: Timelines can get caught in requestAnimationFrame loop when not recording
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: burg, commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix none

Description Joseph Pecoraro 2014-08-14 11:37:50 PDT
* SUMMARY
Profiling the inspector the Timelines can get caught in requestAnimationFrame loop when not recording.

TimelineContentView.js:

    _update: function(timestamp)
    {
        if (this._waitingToResetCurrentTime) {
            requestAnimationFrame(this._updateCallback);
            return;
        }
        ...
    }

* STEPS TO REPRODUCE
1. Inspect <http://bogojoker.com/shell/>
2. Reload the page and wait for the Timeline recording to auto stop
3. Reload the page and wait for the Timeline recording to auto stop
  => looping requestAnimationFrame
Comment 1 Radar WebKit Bug Importer 2014-08-14 11:37:58 PDT
<rdar://problem/18021462>
Comment 2 Joseph Pecoraro 2014-08-14 12:49:32 PDT
Created attachment 236610 [details]
[PATCH] Proposed Fix
Comment 3 Timothy Hatcher 2014-08-14 13:07:36 PDT
Comment on attachment 236610 [details]
[PATCH] Proposed Fix

Nice!
Comment 4 WebKit Commit Bot 2014-08-14 13:40:34 PDT
Comment on attachment 236610 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 236610

Committed r172601: <http://trac.webkit.org/changeset/172601>
Comment 5 WebKit Commit Bot 2014-08-14 13:40:37 PDT
All reviewed patches have been landed.  Closing bug.