Timeline AddTraceProvider API

third_party/WebKit/Source/devtools/front_end/extensions/ExtensionTraceProvider.js

+/**
+ * @constructor
+ * @param {string} extensionOrigin
+ * @param {string} id
+ * @param {string} categoryName
+ * @param {string} categoryTooltip
+ */
+WebInspector.ExtensionTraceProvider = function(extensionOrigin, id, categoryName, categoryTooltip)
+{
+    this._extensionOrigin = extensionOrigin;
+    this._id = id;
+    this._categoryName = categoryName;
+    this._categoryTooltip = categoryTooltip;
+}
+
+WebInspector.ExtensionTraceProvider.prototype = {
+    get id()

[caseq, 2016/07/08 00:57:49]

It doesn't look like we have to expose id externally.

+    {
+        return this._id;
+    },

[caseq, 2016/07/08 00:57:49]

add blank lines after all methods.

+    get categoryName()

[caseq, 2016/07/08 00:57:49]

please add @return annotations. Also, no getters in new code please, just make
it a function.

+    {
+        return this._categoryName;
+    },
+    get categoryTooltip()
+    {
+        return this._categoryTooltip;
+    },
+    run: function()

[caseq, 2016/07/08 00:57:49]

s/run/start/

+    {
+        WebInspector.extensionServer.startTraceRecording(this.id);
+    },
+    /**
+     * @param {!Array.<!WebInspector.NetworkRequest>} requests

[caseq, 2016/07/08 00:57:49]

nit: Array<!WebInspector.NetworkRequest>

+     */
+    stop: function(requests)
+    {
+        WebInspector.extensionServer.stopTraceRecording(this.id);
+    }
+}