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

Unified Diff: third_party/WebKit/Source/devtools/front_end/network/NetworkPanel.js

Issue 2717393003: [DevTools] Turn starting tracing into promise. (Closed)
Patch Set: more work Created 3 years, 10 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
Index: third_party/WebKit/Source/devtools/front_end/network/NetworkPanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/network/NetworkPanel.js b/third_party/WebKit/Source/devtools/front_end/network/NetworkPanel.js
index f09dbedf04845abf4722852af58274fb81862b44..fe8d988bca36cb644fe02afa29327f48a5bfc17e 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/NetworkPanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/NetworkPanel.js
@@ -679,12 +679,6 @@ Network.NetworkPanel.FilmStripRecorder = class {
/**
* @override
- */
- tracingStarted() {
- }
-
- /**
- * @override
* @param {!Array.<!SDK.TracingManager.EventPayload>} events
*/
traceEventsCollected(events) {
@@ -699,10 +693,12 @@ Network.NetworkPanel.FilmStripRecorder = class {
if (!this._tracingModel || !this._target)
return;
this._tracingModel.tracingComplete();
- SDK.targetManager.resumeReload(this._target);
+ var resourceTreeModel = SDK.ResourceTreeModel.fromTarget(this._target);
this._target = null;
this._callback(new SDK.FilmStripModel(this._tracingModel, this._timeCalculator.minimumBoundary() * 1000));
delete this._callback;
+ if (resourceTreeModel)
+ resourceTreeModel.resumeReload();
}
/**
@@ -747,7 +743,9 @@ Network.NetworkPanel.FilmStripRecorder = class {
return;
this._target.tracingManager.stop();
- SDK.targetManager.suspendReload(this._target);
+ var resourceTreeModel = SDK.ResourceTreeModel.fromTarget(this._target);
+ if (resourceTreeModel)
+ resourceTreeModel.suspendReload();
this._callback = callback;
this._filmStripView.setStatusText(Common.UIString('Fetching frames...'));
}

Powered by Google App Engine
This is Rietveld 408576698