DescriptionRevert of Optimize QuadF's PointIsInTriangle function (patchset #3 id:40001 of https://codereview.chromium.org/458173002/)
Reason for revert:
Caused perf regressions: https://code.google.com/p/chromium/issues/detail?id=404968
Original issue's description:
> Optimize QuadF's PointIsInTriangle function
>
> PointIsInTriangle() is still the hotspot of many CSS animation cases.Take Nexus10 with latest chromium build for example:
> 4.0% case: poster-circle.html
> 1.5% case: morphing-cubes.html
>
> This patch changes the algorithm of testing "Point In Triangle".
> 1. Using double-precision float point.
> 2. A much readable algorithm.
> 3. Better performance due to remove 2 div operation
>
> R=danakj@chromium.org
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290290
TBR=kui.zheng@arm.com
NOTREECHECKS=true
NOTRY=true
Committed: https://crrev.com/9ab9844c93996eb2aafaaa0b653dcfa76ba9f340
Cr-Commit-Position: refs/heads/master@{#301455}
Patch Set 1 #
Created: 6 years, 1 month ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|