DescriptionRevert of skia/ext: Early out from analysis when we have more than 1 draw op. (https://codereview.chromium.org/418093003/)
Reason for revert:
This change broke some tests in Blink
Additionally, it renders DevTools shortcuts screen unusable (Open devTools -> F1 -> Shortcuts tab)
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=virtual%2Fandroid%2Ffullscreen%2Ffull-screen-fixed-pos-parent.html%2Cvirtual%2Fandroid%2Ffullscreen%2Ffull-screen-iframe-zIndex.html%2Cvirtual%2Fandroid%2Ffullscreen%2Ffull-screen-zIndex.html
Original issue's description:
> skia/ext: Early out from analysis when we have more than 1 draw op.
>
> This patch puts in a heuristic that earlies out from analysis when
> we have more than one draw op. The intent here is that in most cases,
> if we see more than one draw operation, then analysis is likely not
> to be solid.
>
> It also removes HasText from analysis canvas, since the only
> remaining use for this was a previous early out heuristic.
>
> BUG=397198
> R=enne
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285499
TBR=enne@chromium.org,senorblanco@google.com,junov@chromium.org,vmpstr@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=397198
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285554
Patch Set 1 #
Created: 6 years, 5 months ago
(Patch set is too large to download)
Messages
Total messages: 3 (0 generated)
|