Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

Call Document::UpdateStyleAndLayoutIgnorePendingStylesheets() in the DOM
entrypoint instead of in each GetBBox implementation/override. Add
DCHECKs as needed in the GetBBox implementations instead to assert the
precondition.
Adjust ResizeObservation::ComputeTargetSize (which shouldn't need to
update the layout anyway) to account for this.

Review-Url: https://codereview.chromium.org/2854123004
Cr-Commit-Position: refs/heads/master@{#469061}
Committed: https://chromium.googlesource.com/chromium/src/+/10404c06ed23864a5d180e2ecab206bb39d18a45

[fs, 2017-05-03 13:37:24]

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

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

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 14:08:46]

Description was changed from

==========
Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

Call Document::UpdateStyleAndLayoutIgnorePendingStylesheets() in the DOM
method instead of in each GetBBox implementation/override. Add DCHECKs
as needed in the GetBBox implementations instead to assert the
precondition.
Adjust ResizeObservation::ComputeTargetSize (which shouldn't need to
update the layout anyway) to account for this.
==========

to

==========
Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

Call Document::UpdateStyleAndLayoutIgnorePendingStylesheets() in the DOM
entrypoint instead of in each GetBBox implementation/override. Add
DCHECKs as needed in the GetBBox implementations instead to assert the
precondition.
Adjust ResizeObservation::ComputeTargetSize (which shouldn't need to
update the layout anyway) to account for this.
==========

[fs, 2017-05-03 15:22:59]

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

[commit-bot: I haz the power, 2017-05-03 15:30:59]

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

[commit-bot: I haz the power, 2017-05-03 15:31:00]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-05-03 18:32:01]

lgtm

[fs, 2017-05-03 18:58:14]

The CQ bit was checked by fs@opera.com

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

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 19:04:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493837894903820, "parent_rev":
"f7c0e7db6402fdb3ca65b7e6ffa44989ee056c13", "commit_rev":
"10404c06ed23864a5d180e2ecab206bb39d18a45"}

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

Description was changed from

==========
Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

Call Document::UpdateStyleAndLayoutIgnorePendingStylesheets() in the DOM
entrypoint instead of in each GetBBox implementation/override. Add
DCHECKs as needed in the GetBBox implementations instead to assert the
precondition.
Adjust ResizeObservation::ComputeTargetSize (which shouldn't need to
update the layout anyway) to account for this.
==========

to

==========
Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

Call Document::UpdateStyleAndLayoutIgnorePendingStylesheets() in the DOM
entrypoint instead of in each GetBBox implementation/override. Add
DCHECKs as needed in the GetBBox implementations instead to assert the
precondition.
Adjust ResizeObservation::ComputeTargetSize (which shouldn't need to
update the layout anyway) to account for this.

Review-Url: https://codereview.chromium.org/2854123004
Cr-Commit-Position: refs/heads/master@{#469061}
Committed:
https://chromium.googlesource.com/chromium/src/+/10404c06ed23864a5d180e2ecab2...
==========

[commit-bot: I haz the power, 2017-05-03 19:05:49]

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