DevTools: implement console.timeline/timelineEnd.

Source/devtools/front_end/TimelineManager.js

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 32 matching lines @@
     TimelineStarted: "TimelineStarted",
     TimelineStopped: "TimelineStopped",
     TimelineEventRecorded: "TimelineEventRecorded"
 }
 
 WebInspector.TimelineManager.prototype = {
     /**
      * @param {number=} maxCallStackDepth
      * @param {boolean=} includeDomCounters
      * @param {boolean=} includeNativeMemoryStatistics
+     * @param {function(?Protocol.Error)=} callback
      */
-    start: function(maxCallStackDepth, includeDomCounters, includeNativeMemoryStatistics)
+    start: function(maxCallStackDepth, includeDomCounters, includeNativeMemoryStatistics, callback)
     {
         this._enablementCount++;
         if (this._enablementCount === 1)
-            TimelineAgent.start(maxCallStackDepth, includeDomCounters, includeNativeMemoryStatistics, this._started.bind(this));
+            TimelineAgent.start(maxCallStackDepth, includeDomCounters, includeNativeMemoryStatistics, callback);
+        else if (callback)
+            callback(null);
     },
 
-    stop: function()
+    /**
+     * @param {function(?Protocol.Error)=} callback
+     */
+    stop: function(callback)
     {
         if (!this._enablementCount) {
             console.error("WebInspector.TimelineManager start/stop calls are unbalanced");
             return;
         }
         this._enablementCount--;
         if (!this._enablementCount)
-            TimelineAgent.stop(this._stopped.bind(this));
-    },
-
-    _started: function()
-    {
-        this.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStarted);
-    },
-
-    _stopped: function()
-    {
-        this.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStopped);
+            TimelineAgent.stop(callback);
+        else if (callback)
+            callback(null);
     },
 
     __proto__: WebInspector.Object.prototype
 }
 
 /**
  * @constructor
  * @implements {TimelineAgent.Dispatcher}
  */
 WebInspector.TimelineDispatcher = function(manager)
 {
     this._manager = manager;
     InspectorBackend.registerTimelineDispatcher(this);
 }
 
 WebInspector.TimelineDispatcher.prototype = {
+    /**
+     * @param {TimelineAgent.TimelineEvent} record
+     */
     eventRecorded: function(record)
     {
         this._manager.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineEventRecorded, record);
+    },
+
+    /**
+     * @param {boolean=} consoleTimeline
+     */
+    started: function(consoleTimeline)
+    {
+        if (consoleTimeline)
+            this._manager._enablementCount++;

[caseq, 2013/09/06 09:25:46]

So what happens if pages does console.timeline() and then navigates away?

[pfeldman, 2013/09/06 16:10:30]

Good catch. InspectorTimelineAgent::restore is now telling front-end that
console-initiated timeline has stopped.

+        this._manager.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStarted, consoleTimeline);
+    },
+
+    /**
+     * @param {boolean=} consoleTimeline
+     */
+    stopped: function(consoleTimeline)
+    {
+        if (consoleTimeline)
+            this._manager._enablementCount--;
+        this._manager.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStopped, consoleTimeline);
     }
 }
 
 /**
  * @type {WebInspector.TimelineManager}
  */
 WebInspector.timelineManager;