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

Unified Diff: third_party/WebKit/Source/core/frame/FrameView.cpp

Issue 2556993005: [blink] Split TaskType::Internal into InternalTimer and InternalLoading. (Closed)
Patch Set: Created 4 years 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/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 8f14f007e448d32c2a6764b3b3b04a6adad430ce..9e9335298077ee852a38aac502d23fc161b4d0ae 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -156,12 +156,14 @@ FrameView::FrameView(LocalFrame& frame)
m_canHaveScrollbars(true),
m_hasPendingLayout(false),
m_inSynchronousPostLayout(false),
- m_postLayoutTasksTimer(TaskRunnerHelper::get(TaskType::Internal, &frame),
- this,
- &FrameView::postLayoutTimerFired),
- m_updateWidgetsTimer(TaskRunnerHelper::get(TaskType::Internal, &frame),
- this,
- &FrameView::updateWidgetsTimerFired),
+ m_postLayoutTasksTimer(
+ TaskRunnerHelper::get(TaskType::InternalLoading, &frame),
+ this,
+ &FrameView::postLayoutTimerFired),
+ m_updateWidgetsTimer(
+ TaskRunnerHelper::get(TaskType::InternalLoading, &frame),
Sami 2016/12/08 17:05:08 I would classify both of these as timers since the
altimin 2016/12/08 17:47:40 Done.
+ this,
+ &FrameView::updateWidgetsTimerFired),
m_isTransparent(false),
m_baseBackgroundColor(Color::white),
m_mediaType(MediaTypeNames::screen),

Powered by Google App Engine
This is Rietveld 408576698