Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956, 664961
Committed: https://crrev.com/a229e828a692a3ba58fa5136173f81df1ac8b327
Cr-Commit-Position: refs/heads/master@{#437767}

[fs, 2016-12-08 13:27:57]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-08 13:28:08]

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

[fs, 2016-12-08 13:34:06]

Description was changed from

==========
Strength-reduce the "scale-factor changed" condition for LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
Also use this new state to narrow the condition for paint-invalidation.

BUG=603956,664961
==========

to

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
Also use this new state to narrow the condition for paint-invalidation.

BUG=603956,664961
==========

[commit-bot: I haz the power, 2016-12-08 14:37:11]

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

[commit-bot: I haz the power, 2016-12-08 14:37:13]

Dry run: 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_...)

[fs, 2016-12-09 14:06:11]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-09 14:06:24]

Dry run: 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-12-09 15:54:14]

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

[commit-bot: I haz the power, 2016-12-09 15:54:15]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[fs, 2016-12-09 15:58:46]

Description was changed from

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
Also use this new state to narrow the condition for paint-invalidation.

BUG=603956,664961
==========

to

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)
Also use this new state to narrow the condition for paint-invalidation.

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961
==========

[fs, 2016-12-09 15:59:00]

Description was changed from

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)
Also use this new state to narrow the condition for paint-invalidation.

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961
==========

to

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961
==========

[fs, 2016-12-09 17:18:28]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-09 17:18:48]

Dry run: 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-12-09 20:07:48]

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

[commit-bot: I haz the power, 2016-12-09 20:07:48]

Dry run: This issue passed the CQ dry run.

[fs, 2016-12-09 20:12:55]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2016-12-10 06:32:12]

LGTM

[fs, 2016-12-10 09:33:34]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-12-10 09:33:46]

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-12-10 10:35:37]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481362414223100,
"parent_rev": "132ce37caebfeafa9559518c9d4644798ea7c741", "commit_rev":
"79c2ed73ac9f8474ed436d51e201383fb72893ca"}

[commit-bot: I haz the power, 2016-12-10 10:36:25]

Description was changed from

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961
==========

to

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961

Review-Url: https://codereview.chromium.org/2559123003
==========

[commit-bot: I haz the power, 2016-12-10 10:36:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-12 15:06:53]

Description was changed from

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961

Review-Url: https://codereview.chromium.org/2559123003
==========

to

==========
Strength-reduce the "scale-factor changed" condition in LayoutSVGRoot

Spend some cycles examining the difference between the old and new
local-to-border-box transforms, and only signal scale-factor changes
if that part of the transform changed.
This also means that we now detect changes scale that we previously
didn't (like [1].)

[1] paint/invalidation/svg/absolute-sized-content-with-resources.xhtml

BUG=603956,664961

Committed: https://crrev.com/a229e828a692a3ba58fa5136173f81df1ac8b327
Cr-Commit-Position: refs/heads/master@{#437767}
==========

[commit-bot: I haz the power, 2016-12-12 15:06:54]

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