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

Unified Diff: components/scheduler/renderer/renderer_scheduler.h

Issue 1962053002: Allow expensive task blocking if there is pending iframe navigation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use an enum Created 4 years, 7 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: components/scheduler/renderer/renderer_scheduler.h
diff --git a/components/scheduler/renderer/renderer_scheduler.h b/components/scheduler/renderer/renderer_scheduler.h
index 7bf5c7fe9e928370ffe302f3cfc7fcd2e32ca161..16f4770ccf36e17c4e3a1ccb964ec23bc22ea6da 100644
--- a/components/scheduler/renderer/renderer_scheduler.h
+++ b/components/scheduler/renderer/renderer_scheduler.h
@@ -13,6 +13,7 @@
#include "components/scheduler/child/single_thread_idle_task_runner.h"
#include "components/scheduler/renderer/render_widget_scheduling_state.h"
#include "components/scheduler/scheduler_export.h"
+#include "third_party/WebKit/public/platform/WebScheduler.h"
#include "third_party/WebKit/public/web/WebInputEvent.h"
namespace base {
@@ -128,14 +129,17 @@ class SCHEDULER_EXPORT RendererScheduler : public ChildScheduler {
// Must be called on the main thread.
virtual void OnRendererForegrounded() = 0;
- // Tells the scheduler that a navigation task is pending. While any navigation
- // tasks are pending, the scheduler will ensure that loading tasks are not
- // blocked even if they are expensive. Must be called on the main thread.
- virtual void AddPendingNavigation() = 0;
+ // Tells the scheduler that a navigation task is pending. While any main-frame
+ // navigation tasks are pending, the scheduler will ensure that loading tasks
+ // are not blocked even if they are expensive. Must be called on the main
+ // thread.
+ virtual void AddPendingNavigation(
+ blink::WebScheduler::NavigatingFrameType type) = 0;
// Tells the scheduler that a navigation task is no longer pending.
// Must be called on the main thread.
- virtual void RemovePendingNavigation() = 0;
+ virtual void RemovePendingNavigation(
+ blink::WebScheduler::NavigatingFrameType type) = 0;
// Tells the scheduler that a navigation has started. The scheduler will
// prioritize loading tasks for a short duration afterwards.
« no previous file with comments | « components/scheduler/child/web_scheduler_impl.h ('k') | components/scheduler/renderer/renderer_scheduler_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698