Move transform least common ancestor logic to GeometryMapper (issue 2380423002 by pdr@chromium.org)

[p...@chromium.org]

Reviewers: chrishtr, Xianzhu
CL: https://codereview.chromium.org/2380423002/

Description:
Move transform least common ancestor logic to GeometryMapper

This patch moves the least common ancestor logic off of
GeometryPropertyTreeState and onto GeometryMapper. This will make
it easier to remove GeometryPropertyTreestate in a future patch. With
this patch, we are able to remove some unnecessary templates and
remove the GeometryPropertyTreeStateTest entirely.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Affected files (+70, -100 lines):
M third_party/WebKit/Source/platform/BUILD.gn
M third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h
M third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.cpp
M third_party/WebKit/Source/platform/graphics/paint/GeometryMapperTest.cpp
M third_party/WebKit/Source/platform/graphics/paint/GeometryPropertyTreeState.h
D third_party/WebKit/Source/platform/graphics/paint/GeometryPropertyTreeStateTest.cpp

[wangx...@chromium.org]

lgtm



https://codereview.chromium.org/2380423002/

[p...@chromium.org]

On 2016/09/30 at 20:26:57, wangxianzhu wrote:
> lgtm

Chrishtr, are you okay with this too?

https://codereview.chromium.org/2380423002/

[chri...@chromium.org]

lgtm



https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

CQ is trying da patch. Follow status at

https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2380423002/1


https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

Try jobs failed on following builders:
chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/271193)

https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

CQ is trying da patch. Follow status at

https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2380423002/20001


https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

Try jobs failed on following builders:

android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/40522)

https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

[commit-bot@chromium.org via codereview.chromium.org]

Try jobs failed on following builders:

linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/307989)

https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

[commit-bot@chromium.org via codereview.chromium.org]

Try jobs failed on following builders:
linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,

http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/308034)

https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

[commit-bot@chromium.org via codereview.chromium.org]

Committed patchset #2 (id:20001)

https://codereview.chromium.org/2380423002/

[commit-bot@chromium.org via codereview.chromium.org]

Patchset 2 (id:??) landed as
https://crrev.com/95f4e0b2bd03fa3443f782ae3cb3cbfc08a74503
Cr-Commit-Position: refs/heads/master@{#422332}

https://codereview.chromium.org/2380423002/