Simplify the SVGGraphicsElement ...CTM methods

Simplify the SVGGraphicsElement ...CTM methods

GetCTM() only has a single caller, so can be trivially folded. Similarly
GetScreenCTM which has one kAllowStyleUpdate and one
kDisallowStyleUpdate, so it can be folded while hoisting the call to
update style and layout.
This means that there are no longer a collision between internal methods
and the ones exposed in the DOM, so the "...FromJavascript" suffixes can
be dropped.

Review-Url: https://codereview.chromium.org/2858913002
Cr-Commit-Position: refs/heads/master@{#469055}
Committed: https://chromium.googlesource.com/chromium/src/+/9b15a04b5406ec7b03a525aed953c804ff15813e

[fs, 2017-05-03 11:56:54]

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

[commit-bot: I haz the power, 2017-05-03 11:57:01]

Dry run: CQ is trying da patch.

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

[fs, 2017-05-03 13:02:19]

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

[commit-bot: I haz the power, 2017-05-03 13:34:05]

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

[commit-bot: I haz the power, 2017-05-03 13:34:06]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-05-03 18:21:29]

Took me a while to verify, but yes, lgtm.

[fs, 2017-05-03 18:46:51]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2017-05-03 18:48:11]

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, 2017-05-03 18:55:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493837211980850, "parent_rev":
"763e91015ba203feaa3b27c77ce4aa52638ec6e4", "commit_rev":
"9b15a04b5406ec7b03a525aed953c804ff15813e"}

[commit-bot: I haz the power, 2017-05-03 18:55:55]

Description was changed from

==========
Simplify the SVGGraphicsElement ...CTM methods

GetCTM() only has a single caller, so can be trivially folded. Similarly
GetScreenCTM which has one kAllowStyleUpdate and one
kDisallowStyleUpdate, so it can be folded while hoisting the call to
update style and layout.
This means that there are no longer a collision between internal methods
and the ones exposed in the DOM, so the "...FromJavascript" suffixes can
be dropped.
==========

to

==========
Simplify the SVGGraphicsElement ...CTM methods

GetCTM() only has a single caller, so can be trivially folded. Similarly
GetScreenCTM which has one kAllowStyleUpdate and one
kDisallowStyleUpdate, so it can be folded while hoisting the call to
update style and layout.
This means that there are no longer a collision between internal methods
and the ones exposed in the DOM, so the "...FromJavascript" suffixes can
be dropped.

Review-Url: https://codereview.chromium.org/2858913002
Cr-Commit-Position: refs/heads/master@{#469055}
Committed:
https://chromium.googlesource.com/chromium/src/+/9b15a04b5406ec7b03a525aed953...
==========

[commit-bot: I haz the power, 2017-05-03 18:55:59]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9b15a04b5406ec7b03a525aed953...