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

Unified Diff: Source/WebCore/platform/graphics/chromium/cc/CCScheduler.cpp

Issue 9950004: Merge 112446 - [chromium] Scheduler should not tell FrameRateController to begin a frame when we do… (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/1084/
Patch Set: Created 8 years, 9 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: Source/WebCore/platform/graphics/chromium/cc/CCScheduler.cpp
===================================================================
--- Source/WebCore/platform/graphics/chromium/cc/CCScheduler.cpp (revision 112607)
+++ Source/WebCore/platform/graphics/chromium/cc/CCScheduler.cpp (working copy)
@@ -163,17 +163,18 @@
m_client->scheduledActionCommit();
break;
case CCSchedulerStateMachine::ACTION_DRAW_IF_POSSIBLE: {
- bool drawSuccess = m_client->scheduledActionDrawAndSwapIfPossible();
- m_stateMachine.didDrawIfPossibleCompleted(drawSuccess);
- if (drawSuccess)
+ CCScheduledActionDrawAndSwapResult result = m_client->scheduledActionDrawAndSwapIfPossible();
+ m_stateMachine.didDrawIfPossibleCompleted(result.didDraw);
+ if (result.didSwap)
m_frameRateController->didBeginFrame();
break;
}
- case CCSchedulerStateMachine::ACTION_DRAW_FORCED:
- m_client->scheduledActionDrawAndSwapForced();
- m_frameRateController->didBeginFrame();
+ case CCSchedulerStateMachine::ACTION_DRAW_FORCED: {
+ CCScheduledActionDrawAndSwapResult result = m_client->scheduledActionDrawAndSwapForced();
+ if (result.didSwap)
+ m_frameRateController->didBeginFrame();
break;
- case CCSchedulerStateMachine::ACTION_BEGIN_CONTEXT_RECREATION:
+ } case CCSchedulerStateMachine::ACTION_BEGIN_CONTEXT_RECREATION:
m_client->scheduledActionBeginContextRecreation();
break;
}

Powered by Google App Engine
This is Rietveld 408576698