Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1827)

Unified Diff: Source/devtools/front_end/extensions/ExtensionServer.js

Issue 477103002: DevTools: Nuke Timeline extension api (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/devtools/front_end/extensions/ExtensionAPI.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/extensions/ExtensionServer.js
diff --git a/Source/devtools/front_end/extensions/ExtensionServer.js b/Source/devtools/front_end/extensions/ExtensionServer.js
index 317925902a3c723ddc3427d5071d8ebe043dd5ad..dcfe6e1f75d77200a210e18fb1cfcc988040b6ef 100644
--- a/Source/devtools/front_end/extensions/ExtensionServer.js
+++ b/Source/devtools/front_end/extensions/ExtensionServer.js
@@ -692,31 +692,6 @@ WebInspector.ExtensionServer.prototype = {
this._notifySourceFrameSelectionChanged);
this._registerResourceContentCommittedHandler(this._notifyUISourceCodeContentCommitted);
- /**
- * @this {WebInspector.ExtensionServer}
- */
- function onTimelineSubscriptionStarted()
- {
- var mainTarget = WebInspector.targetManager.mainTarget();
- mainTarget.timelineManager.addEventListener(WebInspector.TimelineManager.EventTypes.TimelineEventRecorded,
- this._notifyTimelineEventRecorded, this);
- mainTarget.timelineManager.start();
- }
-
- /**
- * @this {WebInspector.ExtensionServer}
- */
- function onTimelineSubscriptionStopped()
- {
- var mainTarget = WebInspector.targetManager.mainTarget();
- mainTarget.timelineManager.stop(function() {});
- mainTarget.timelineManager.removeEventListener(WebInspector.TimelineManager.EventTypes.TimelineEventRecorded,
- this._notifyTimelineEventRecorded, this);
- }
-
- this._registerSubscriptionHandler(WebInspector.extensionAPI.Events.TimelineEventRecorded,
- onTimelineSubscriptionStarted.bind(this), onTimelineSubscriptionStopped.bind(this));
-
WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.InspectedURLChanged,
this._inspectedURLChanged, this);
@@ -776,11 +751,6 @@ WebInspector.ExtensionServer.prototype = {
this._postNotification(WebInspector.extensionAPI.Events.PanelObjectSelected + "elements");
},
- _notifyTimelineEventRecorded: function(event)
- {
- this._postNotification(WebInspector.extensionAPI.Events.TimelineEventRecorded, event.data);
- },
-
/**
* @param {!Array.<!ExtensionDescriptor>} extensionInfos
*/
« no previous file with comments | « Source/devtools/front_end/extensions/ExtensionAPI.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698