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

Unified Diff: third_party/WebKit/public/platform/scheduler/renderer/renderer_scheduler.h

Issue 2890173002: EQT: Record the max queueing time from navigation start to navigation away (Closed)
Patch Set: fix mock_renderer_scheduler Created 3 years, 6 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/public/platform/scheduler/renderer/renderer_scheduler.h
diff --git a/third_party/WebKit/public/platform/scheduler/renderer/renderer_scheduler.h b/third_party/WebKit/public/platform/scheduler/renderer/renderer_scheduler.h
index 65b3f5ab4feb83e9a70f03f7ee2d9cf37c19e429..86a362e8d81fedc78239170b3bd3c588598403af 100644
--- a/third_party/WebKit/public/platform/scheduler/renderer/renderer_scheduler.h
+++ b/third_party/WebKit/public/platform/scheduler/renderer/renderer_scheduler.h
@@ -151,8 +151,15 @@ class BLINK_PLATFORM_EXPORT RendererScheduler : public ChildScheduler {
// Tells the scheduler that a navigation has started. The scheduler will
// prioritize loading tasks for a short duration afterwards.
// Must be called from the main thread.
+ // TODO(maxlg): remove OnNavigationStarted and migrate this part into
+ // OnCommitProvisionalLoad.
virtual void OnNavigationStarted() = 0;
+ // Tells the scheduler that the provisional load has been committed. This
+ // signals the point that activity stops happening in the old page and the
+ // renderer starts parsing and rendering the new page.
+ virtual void OnCommitProvisionalLoad() = 0;
+
// Returns true if the scheduler has reason to believe that high priority work
// may soon arrive on the main thread, e.g., if gesture events were observed
// recently.

Powered by Google App Engine
This is Rietveld 408576698