DescriptionRevert of cc: Adding DidFinishImplFrame to LTHI. (patchset #10 id:180001 of https://codereview.chromium.org/1111743002/)
Reason for revert:
Broke Win x64 GN build.
http://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/1013/steps/compile/logs/stdio
Original issue's description:
> cc: Adding DidFinishImplFrame to LTHI.
>
> This change moves the responsibility of clearing state inside the LTHI after an
> impl frame is finished from the thread proxies to the LTHI class. This makes
> WillBeginImplFrame and DidFinishImplFrame a logical pair.
>
> This CL also;
> * Adds a test that checks the number of WillBeginImplFrame
> calls matches the DidFinishImplFrame calls.
> * Cleans up classes in cc/test/layer_tree_test.h around WillBeginImplFrame.
> * Removes the UpdateCurrentBeginFrameArgs / ResetCurrentBeginFrameArgs methods.
> * Fixes a bug where STP was calling ResetCurrentBeginFrameArgs twice every frame.
>
> BUG=346230, 481810
> R=brianderson,enne
>
> Committed: https://crrev.com/2caee4f89de17858822c11379da9c6cd80f90708
> Cr-Commit-Position: refs/heads/master@{#328689}
TBR=brianderson@chromium.org,enne@chromium.org,danakj@chromium.org,sunnyps@chromium.org,mithro@mithis.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=346230, 481810
Committed: https://crrev.com/4279b872d0187987853fb1f9283486be50efe26a
Cr-Commit-Position: refs/heads/master@{#328692}
Patch Set 1 #
Created: 5 years, 7 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|