Hoist "layout size changed" check out of SVGLayoutSupport::layoutChildren

Hoist "layout size changed" check out of SVGLayoutSupport::layoutChildren

For LayoutSVGRoot, this predicate can be computed quite easily, so
calling layoutSizeOfNearestViewportChanged() will be an unnecessary
detour.
Also rearrange layoutSizeOfNearestViewportChanged() to avoid an
unnecessary virtual call.

BUG=603956
Committed: https://crrev.com/e5da17ed9ca012c47e7980ef258f85d2d782dd79
Cr-Commit-Position: refs/heads/master@{#388332}

[fs, 2016-04-19 17:19:34]

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

[pdr., 2016-04-19 18:49:22]

LGTM

[fs, 2016-04-19 20:22:50]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-04-19 20:23:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1902073002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1902073002/1

[commit-bot: I haz the power, 2016-04-19 20:58:07]

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

[commit-bot: I haz the power, 2016-04-19 20:58:08]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[fs, 2016-04-19 21:40:12]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-04-19 21:40:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1902073002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1902073002/1

[commit-bot: I haz the power, 2016-04-19 22:36:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:17:13]

Description was changed from

==========
Hoist "layout size changed" check out of SVGLayoutSupport::layoutChildren

For LayoutSVGRoot, this predicate can be computed quite easily, so
calling layoutSizeOfNearestViewportChanged() will be an unnecessary
detour.
Also rearrange layoutSizeOfNearestViewportChanged() to avoid an
unnecessary virtual call.

BUG=603956
==========

to

==========
Hoist "layout size changed" check out of SVGLayoutSupport::layoutChildren

For LayoutSVGRoot, this predicate can be computed quite easily, so
calling layoutSizeOfNearestViewportChanged() will be an unnecessary
detour.
Also rearrange layoutSizeOfNearestViewportChanged() to avoid an
unnecessary virtual call.

BUG=603956

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

[commit-bot: I haz the power, 2016-04-22 19:17:14]

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