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

Side by Side Diff: third_party/WebKit/public/platform/WebScheduler.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: 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef WebScheduler_h 5 #ifndef WebScheduler_h
6 #define WebScheduler_h 6 #define WebScheduler_h
7 7
8 #include "WebCommon.h" 8 #include "WebCommon.h"
9 #include "public/platform/WebTaskRunner.h" 9 #include "public/platform/WebTaskRunner.h"
10 #include "public/platform/WebThread.h" 10 #include "public/platform/WebThread.h"
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 // Suspends the timer queue and increments the timer queue suspension count. 74 // Suspends the timer queue and increments the timer queue suspension count.
75 // May only be called from the main thread. 75 // May only be called from the main thread.
76 virtual void suspendTimerQueue() = 0; 76 virtual void suspendTimerQueue() = 0;
77 77
78 // Decrements the timer queue suspension count and re-enables the timer queu e 78 // Decrements the timer queue suspension count and re-enables the timer queu e
79 // if the suspension count is zero and the current scheduler policy allows i t. 79 // if the suspension count is zero and the current scheduler policy allows i t.
80 virtual void resumeTimerQueue() = 0; 80 virtual void resumeTimerQueue() = 0;
81 81
82 // Tells the scheduler that a navigation task is pending. 82 // Tells the scheduler that a navigation task is pending.
83 // TODO(alexclarke): Long term should this be a task trait? 83 // TODO(alexclarke): Long term should this be a task trait?
84 virtual void addPendingNavigation() = 0; 84 virtual void addPendingNavigation(bool mainFrameNavigating) = 0;
Sami 2016/05/09 16:45:21 I'm tempted to suggest making this an enum instead
alex clarke (OOO till 29th) 2016/05/09 17:08:09 Yeah I was tempted to do that too, but didn't init
85 85
86 // Tells the scheduler that a navigation task is no longer pending. 86 // Tells the scheduler that a navigation task is no longer pending.
87 virtual void removePendingNavigation() = 0; 87 virtual void removePendingNavigation(bool mainFrameNavigating) = 0;
88 88
89 // Tells the scheduler that an expected navigation was started. 89 // Tells the scheduler that an expected navigation was started.
90 virtual void onNavigationStarted() = 0; 90 virtual void onNavigationStarted() = 0;
91 91
92 #ifdef INSIDE_BLINK 92 #ifdef INSIDE_BLINK
93 // Helpers for posting bound functions as tasks. 93 // Helpers for posting bound functions as tasks.
94 typedef Function<void(double deadlineSeconds)> IdleTask; 94 typedef Function<void(double deadlineSeconds)> IdleTask;
95 95
96 void postIdleTask(const WebTraceLocation&, PassOwnPtr<IdleTask>); 96 void postIdleTask(const WebTraceLocation&, PassOwnPtr<IdleTask>);
97 void postNonNestableIdleTask(const WebTraceLocation&, PassOwnPtr<IdleTask>); 97 void postNonNestableIdleTask(const WebTraceLocation&, PassOwnPtr<IdleTask>);
98 void postIdleTaskAfterWakeup(const WebTraceLocation&, PassOwnPtr<IdleTask>); 98 void postIdleTaskAfterWakeup(const WebTraceLocation&, PassOwnPtr<IdleTask>);
99 #endif 99 #endif
100 }; 100 };
101 101
102 } // namespace blink 102 } // namespace blink
103 103
104 #endif // WebScheduler_h 104 #endif // WebScheduler_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698