DevTools: switch Timeline frontend into buffered mode and remove the corresponding experiment.

Source/devtools/front_end/timeline/TimelineModelImpl.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @extends {WebInspector.TimelineModel}
  * @param {!WebInspector.TimelineManager} timelineManager
  */
 WebInspector.TimelineModelImpl = function(timelineManager)
@@ skipping 31 matching lines @@
     {
         console.assert(!capturePictures, "Legacy timeline does not support capturing pictures");
         this._clientInitiatedRecording = true;
         this.reset();
         var maxStackFrames = captureStacks ? 30 : 0;
         var includeGPUEvents = WebInspector.experimentsSettings.gpuTimeline.isEnabled();
         var liveEvents = [ WebInspector.TimelineModel.RecordType.BeginFrame,
                            WebInspector.TimelineModel.RecordType.DrawFrame,
                            WebInspector.TimelineModel.RecordType.RequestMainThreadFrame,
                            WebInspector.TimelineModel.RecordType.ActivateLayerTree ];
-        this._timelineManager.start(maxStackFrames, WebInspector.experimentsSettings.timelineNoLiveUpdate.isEnabled(), liveEvents.join(","), captureMemory, includeGPUEvents, this._fireRecordingStarted.bind(this));
+        this._timelineManager.start(maxStackFrames, liveEvents.join(","), captureMemory, includeGPUEvents, this._fireRecordingStarted.bind(this));
     },
 
     stopRecording: function()
     {
         if (!this._clientInitiatedRecording) {
-            this._timelineManager.start(undefined, undefined, undefined, undefined, undefined, stopTimeline.bind(this));
+            this._timelineManager.start(undefined, undefined, undefined, undefined, stopTimeline.bind(this));
             return;
         }
 
         /**
          * Console started this one and we are just sniffing it. Initiate recording so that we
          * could stop it.
          * @this {WebInspector.TimelineModelImpl}
          */
         function stopTimeline()
         {
@@ skipping 10 matching lines @@
     records: function()
     {
         return this._records;
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onRecordAdded: function(event)
     {
-        if (this._collectionEnabled)

[yurys, 2014/07/17 10:42:24]

Without this check we will display events if timeline recording was initiated by
an extension. @caseq, do you think it is OK?

-            this._addRecord(/** @type {!TimelineAgent.TimelineEvent} */(event.data));
+        this._addRecord(/** @type {!TimelineAgent.TimelineEvent} */(event.data));
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onStarted: function(event)
     {
         if (event.data) {
             // Started from console.
             this._fireRecordingStarted();
         }
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onStopped: function(event)
     {
-        // If we were buffering events, discard those that got through, the real ones are coming!
-        if (WebInspector.experimentsSettings.timelineNoLiveUpdate.isEnabled())
-            this.reset();
+        // We were buffering events, discard those that got through, the real ones are coming!
+        this.reset();
         if (event.data) {
             // Stopped from console.
             this._fireRecordingStopped(null, null);
         }
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onProgress: function(event)
     {
         this.dispatchEventToListeners(WebInspector.TimelineModel.Events.RecordingProgress, event.data);
     },
 
     _fireRecordingStarted: function()
     {
-        this._collectionEnabled = true;
         this.dispatchEventToListeners(WebInspector.TimelineModel.Events.RecordingStarted);
     },
 
     /**
      * @param {?Protocol.Error} error
      * @param {?ProfilerAgent.CPUProfile} cpuProfile
      */
     _fireRecordingStopped: function(error, cpuProfile)
     {
-        this._collectionEnabled = false;
         if (cpuProfile)
             WebInspector.TimelineJSProfileProcessor.mergeJSProfileIntoTimeline(this, cpuProfile);
         this.dispatchEventToListeners(WebInspector.TimelineModel.Events.RecordingStopped);
     },
 
     /**
      * @param {!TimelineAgent.TimelineEvent} payload
      */
     _addRecord: function(payload)
     {
@@ skipping 552 matching lines @@
                 break;
             length += itemLength;
             data.push(item);
             ++this._recordIndex;
         }
         if (this._recordIndex === this._payloads.length)
             data.push(data.pop() + "]");
         stream.write(data.join(separator), this._writeNextChunk.bind(this));
     }
 }