DescriptionRevert of re-land: Use browser compositor rendering stats in smoothness (https://codereview.chromium.org/174663008/)
Reason for revert:
Broke the smoothness perf tests on multiple bots.
BUG=349338
Original issue's description:
> re-land: Use browser compositor rendering stats in smoothness
>
> PERF SHERRIFS: This patch can change the metrics of smoothness up or down.
> The new values should be more accurate than what we had before. If there are
> large changes on a platform, we should still sanity check that everything works
> as expected.
>
> Switch smoothness benchmark to use rendering stats from the top level
> compositor; i.e. if data from a browser compositor is available, use that.
> Otherwise use the data from the renderer compositor.
>
> The re-land fixes the problem where a browser compositor was present, but
> didn't record rendering stats. That was the case with the reference
> builds (that didn't have the patch that enabled recording) and on Android.
> The new version checks if the browser compositor rendering stats events
> actually have frames in them. If not, the render compositor stats will be used.
> There is also a fix on the way to enable the recording on Android
> (https://codereview.chromium.org/168193004/).
>
> R=tonyg@chromium.org,nduca@chromium.org
> BUG=340753
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=254811
TBR=nduca@chromium.org,tonyg@chromium.org,ernstm@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=340753
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=254969
Patch Set 1 #
Created: 6 years, 9 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|