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 19 matching lines @@
 
 /**
  * @constructor
  * @extends {WebInspector.Object}
  */
 WebInspector.TimelineManager = function()
 {
     WebInspector.Object.call(this);
     this._dispatcher = new WebInspector.TimelineDispatcher(this);
     this._enablementCount = 0;
+    TimelineAgent.enable();
 }
 
 WebInspector.TimelineManager.EventTypes = {
     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");
+        this._enablementCount--;
+        if (this._enablementCount < 0) {

[caseq, 2013/09/13 15:00:23]

So if we actually hit this, we now let enablemenetCount become negative and
hence it would likely be impossible to enable timeline again until devtools
restart. Is this intentional?

[pfeldman, 2013/09/13 15:21:03]

Yes, it is anyways impossible to recover from the unbalanced state.

+            console.error("WebInspector.TimelineManager start/stop calls are unbalanced " + new Error().stack);
             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) {
+            // Wake up timeline panel module.
+            WebInspector.panel("timeline");
+        }
+        this._manager.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStarted, consoleTimeline);
+    },
+
+    /**
+     * @param {boolean=} consoleTimeline
+     */
+    stopped: function(consoleTimeline)
+    {
+        this._manager.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStopped, consoleTimeline);
     }
 }
 
 /**
  * @type {WebInspector.TimelineManager}
  */
 WebInspector.timelineManager;