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

Issue 9950004: Merge 112446 - [chromium] Scheduler should not tell FrameRateController to begin a frame when we do… (Closed)

Created:
8 years, 8 months ago by jamesr
Modified:
8 years, 8 months ago
Reviewers:
jamesr
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1084/
Visibility:
Public.

Description

Merge 112446 - [chromium] Scheduler should not tell FrameRateController to begin a frame when we dont swap https://bugs.webkit.org/show_bug.cgi?id=82516 Reviewed by James Robinson. Source/WebCore: * platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::swapBuffers): * platform/graphics/chromium/LayerRendererChromium.h: (LayerRendererChromium): * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp: (WebCore::CCLayerTreeHostImpl::swapBuffers): * platform/graphics/chromium/cc/CCLayerTreeHostImpl.h: (CCLayerTreeHostImpl): * platform/graphics/chromium/cc/CCScheduler.cpp: (WebCore::CCScheduler::processScheduledActions): * platform/graphics/chromium/cc/CCScheduler.h: (WebCore::CCScheduledActionDrawAndSwapResult::CCScheduledActionDrawAndSwapResult): (CCScheduledActionDrawAndSwapResult): (WebCore): (CCSchedulerClient): * platform/graphics/chromium/cc/CCThreadProxy.cpp: (WebCore::CCThreadProxy::scheduledActionDrawAndSwapInternal): (WebCore::CCThreadProxy::scheduledActionDrawAndSwapIfPossible): (WebCore::CCThreadProxy::scheduledActionDrawAndSwapForced): * platform/graphics/chromium/cc/CCThreadProxy.h: (CCThreadProxy): Source/WebKit/chromium: * tests/CCSchedulerTest.cpp: (WebKitTests::FakeCCSchedulerClient::reset): (WebKitTests::FakeCCSchedulerClient::hasAction): (FakeCCSchedulerClient): (WebKitTests::FakeCCSchedulerClient::scheduledActionDrawAndSwapIfPossible): (WebKitTests::FakeCCSchedulerClient::scheduledActionDrawAndSwapForced): (WebKitTests::FakeCCSchedulerClient::setDrawWillHappen): (WebKitTests::FakeCCSchedulerClient::setSwapWillHappenIfDrawHappens): (WebKitTests::SchedulerClientThatSetNeedsDrawInsideDraw::scheduledActionDrawAndSwapIfPossible): (WebKitTests::SchedulerClientThatSetNeedsDrawInsideDraw::scheduledActionDrawAndSwapForced): (SchedulerClientThatSetNeedsDrawInsideDraw): (WebKitTests::TEST): (WebKitTests::SchedulerClientThatSetNeedsCommitInsideDraw::scheduledActionDrawAndSwapIfPossible): (WebKitTests::SchedulerClientThatSetNeedsCommitInsideDraw::scheduledActionDrawAndSwapForced): (SchedulerClientThatSetNeedsCommitInsideDraw): (WebKitTests): TBR=nduca@chromium.org BUG=120406 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=112609

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3950 lines, -42 lines) Patch
M Source/WebCore/ChangeLog View 1 chunk +3716 lines, -0 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp View 2 chunks +3 lines, -2 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCScheduler.h View 1 chunk +17 lines, -2 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCScheduler.cpp View 1 chunk +8 lines, -7 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCThreadProxy.h View 2 chunks +3 lines, -3 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCThreadProxy.cpp View 4 chunks +13 lines, -8 lines 0 comments Download
M Source/WebKit/chromium/ChangeLog View 1 chunk +126 lines, -0 lines 0 comments Download
M Source/WebKit/chromium/tests/CCSchedulerTest.cpp View 11 chunks +60 lines, -16 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
jamesr
8 years, 8 months ago (2012-03-29 23:43:41 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698