Separate forced subtree invalidation reasons for location change and SVG resource change

Separate forced subtree invalidation reasons for location change and SVG resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/2902b7fc2999ea4c1f3f647be6f3029ee7e3e168
Cr-Commit-Position: refs/heads/master@{#430453}

[Xianzhu, 2016-11-07 20:26:36]

Description was changed from

==========
Separate forced subtree reasons for location change and SVG resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
==========

to

==========
Separate forced subtree reasons for location change and SVG resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2016-11-07 20:27:37]

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

[Xianzhu, 2016-11-07 20:28:21]

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

[commit-bot: I haz the power, 2016-11-07 20:28:25]

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

[pdr., 2016-11-07 21:30:50]

LGTM

[Xianzhu, 2016-11-07 21:41:51]

Description was changed from

==========
Separate forced subtree reasons for location change and SVG resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Separate forced subtree invalidation reasons for location change and SVG
resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-11-07 22:35:23]

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

[commit-bot: I haz the power, 2016-11-07 22:35:24]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Xianzhu, 2016-11-07 23:29:49]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-11-07 23:29:49]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/2483953002/#ps40001
(title: "rebaseline-cl")

[commit-bot: I haz the power, 2016-11-07 23:30:49]

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-08 01:05:42]

Description was changed from

==========
Separate forced subtree invalidation reasons for location change and SVG
resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Separate forced subtree invalidation reasons for location change and SVG
resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-11-08 01:05:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-08 01:17:22]

Description was changed from

==========
Separate forced subtree invalidation reasons for location change and SVG
resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Separate forced subtree invalidation reasons for location change and SVG
resource change

This has several purposes:

- To reduce layout test result differences between normal path and
  SlimmingPaintInvalidation which will add paint offset change detection
  soon.

- To avoid unnecessary invalidation of SVG display item clients when
  an non-SVG/SVG-root ancestor changed location. Previously we
  conservatively assumed paint offsets of all descendants changed if
  an ancestor is set forcedSubtreeInvalidationCheckion when the
  ancestor's location changed, but normal SVG object always have
  zero paint offsets which won't change.

- For SlimmingPaintInvalidation to explicitly invalidate display item
  clients on ancestor SVG resource change. Previously this logic was
  shadowed by the convervative paint offset change logic.

BUG=660195
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/2902b7fc2999ea4c1f3f647be6f3029ee7e3e168
Cr-Commit-Position: refs/heads/master@{#430453}
==========

[commit-bot: I haz the power, 2016-11-08 01:17:23]

Patchset 3 (id:??) landed as
https://crrev.com/2902b7fc2999ea4c1f3f647be6f3029ee7e3e168
Cr-Commit-Position: refs/heads/master@{#430453}