diff options
| author | simonhong@chromium.org <simonhong@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-29 02:23:48 +0000 |
|---|---|---|
| committer | simonhong@chromium.org <simonhong@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-29 02:23:48 +0000 |
| commit | e35027e7404f63c056d369de8d8f583e1d0ae96e (patch) | |
| tree | 428f543f87e209079504141edadc43118612b64d /cc/scheduler/scheduler_state_machine.cc | |
| parent | 4ec4bbf04a3b0c9daa69fe1bb48599c04017efe9 (diff) | |
| download | chromium_src-e35027e7404f63c056d369de8d8f583e1d0ae96e.zip chromium_src-e35027e7404f63c056d369de8d8f583e1d0ae96e.tar.gz chromium_src-e35027e7404f63c056d369de8d8f583e1d0ae96e.tar.bz2 | |
cc: Don't execute UpdateVisibleTiles() right after drawing
Currently UpdateVisibleTiles() is called twice,
right after drawing and in the next impl frame deadline.
We should call it once and it should be called in the
next deadline.
R=brianderson@chromium.org
BUG=378200
TEST=cc_unittests --gtest_filter=SchedulerTest.ShouldUpdateVisibleTiles
Review URL: https://codereview.chromium.org/301563009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273441 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/scheduler/scheduler_state_machine.cc')
| -rw-r--r-- | cc/scheduler/scheduler_state_machine.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/cc/scheduler/scheduler_state_machine.cc b/cc/scheduler/scheduler_state_machine.cc index febe1ff..f7919a6 100644 --- a/cc/scheduler/scheduler_state_machine.cc +++ b/cc/scheduler/scheduler_state_machine.cc @@ -383,6 +383,10 @@ bool SchedulerStateMachine::ShouldUpdateVisibleTiles() const { if (HasUpdatedVisibleTilesThisFrame()) return false; + // We don't want to update visible tiles right after drawing. + if (HasRequestedSwapThisFrame()) + return false; + // There's no reason to check for tiles if we don't have an output surface. if (!HasInitializedOutputSurface()) return false; |