Remove workaround for paint invalidation about percentage transform

Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly by the current code.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged when running the test.
In the current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/469897a8f942f3e8fa46ad71a5c2bac8f3bdcba0
Cr-Commit-Position: refs/heads/master@{#431419}

[Xianzhu, 2016-11-10 22:40:24]

Description was changed from

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged in the test. In the
current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
==========

to

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged in the test. In the
current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2016-11-10 22:40:36]

The CQ bit was checked by wangxianzhu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 22:41:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xianzhu, 2016-11-10 23:28:25]

Description was changed from

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged in the test. In the
current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly by the current code.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged when running the test.
In the current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2016-11-10 23:28:47]

wangxianzhu@chromium.org changed reviewers:
+ chrishtr@chromium.org

[commit-bot: I haz the power, 2016-11-11 00:15:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-11 00:15:04]

Dry run: This issue passed the CQ dry run.

[chrishtr, 2016-11-11 00:34:37]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-11-11 00:34:37]

lgtm

[chrishtr, 2016-11-11 00:34:42]

Nice

[commit-bot: I haz the power, 2016-11-11 00:35:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-11 00:40:49]

Description was changed from

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly by the current code.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged when running the test.
In the current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly by the current code.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged when running the test.
In the current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-11-11 00:40:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-11 00:45:51]

Description was changed from

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly by the current code.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged when running the test.
In the current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Remove workaround for paint invalidation about percentage transform

It was for descendants of objects with percentage transform, to handle
the case that paint offset changed but no other paint invalidation
geometry changed.

Now the case is covered by the ancestor location change condition.
Paint offset change should either caused by self layout change,
self location change or ancestor location change, all of which are
handled properly by the current code.

Debugged a bit with the test case and the version that the logic was
added (https://crrev.com/787d405d41fcb60aa0a8e45b0c6e5d4a9bbe75d5),
and found that positionFromPaintInvalidationBacking of the transformed
contiainer was incorrect. It kept unchanged when running the test.
In the current code the value is correct.

BUG=533277
TEST=paint/invalidation/percentage-transform-paint-offset.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/469897a8f942f3e8fa46ad71a5c2bac8f3bdcba0
Cr-Commit-Position: refs/heads/master@{#431419}
==========

[commit-bot: I haz the power, 2016-11-11 00:45:52]

Patchset 1 (id:??) landed as
https://crrev.com/469897a8f942f3e8fa46ad71a5c2bac8f3bdcba0
Cr-Commit-Position: refs/heads/master@{#431419}