Avoid style recalc and layout when not necessary for client size.

Avoid style recalc and layout when not necessary for client size.

The clientWidth and clientHeight values do not rely on style recalc for
the documentElement in strict mode, and the body element in quirks mode
when we use overlay scrollbars. Skip the forced layout update in those
cases.

BUG=481412
Committed: https://crrev.com/b87800cbc5af30ef4fad30f874121702c857b753
Cr-Commit-Position: refs/heads/master@{#391187}

[rune, 2016-05-02 14:05:30]

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

[commit-bot: I haz the power, 2016-05-02 14:05:38]

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

[commit-bot: I haz the power, 2016-05-02 15:33:36]

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

[commit-bot: I haz the power, 2016-05-02 15:33:36]

Dry run: This issue passed the CQ dry run.

[rune, 2016-05-02 18:25:19]

rune@opera.com changed reviewers:
+ esprehn@chromium.org, rbyers@chromium.org

[esprehn, 2016-05-02 19:02:12]

This is only correct it you're the top level document I think. Your patch would
skip the layout for an iframe where the parent document layout impacts the child
document.

You probably need a check for isLocalRoot.

https://codereview.chromium.org/1935043002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/dom/body-quirk-client-size.html
(right):

https://codereview.chromium.org/1935043002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/dom/body-quirk-client-size.html:1: <script
src="../../resources/js-test.js"></script>
Can you add a comment at the top like <!-- quirks mode -->

[rune, 2016-05-02 22:48:12]

On 2016/05/02 19:02:12, esprehn wrote:
> This is only correct it you're the top level document I think. Your patch
would
> skip the layout for an iframe where the parent document layout impacts the
child
> document.
> 
> You probably need a check for isLocalRoot.

You're right. Done + added a test.

[rune, 2016-05-02 22:48:22]

https://codereview.chromium.org/1935043002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/dom/body-quirk-client-size.html
(right):

https://codereview.chromium.org/1935043002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/dom/body-quirk-client-size.html:1: <script
src="../../resources/js-test.js"></script>
On 2016/05/02 19:02:12, esprehn wrote:
> Can you add a comment at the top like <!-- quirks mode -->

Done.

[esprehn, 2016-05-03 03:03:59]

lgtm

[rune, 2016-05-03 06:14:53]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-05-03 06:14:58]

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

[commit-bot: I haz the power, 2016-05-03 08:09:04]

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

[commit-bot: I haz the power, 2016-05-03 08:09:05]

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...)

[rune, 2016-05-03 08:20:20]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-05-03 08:20:29]

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

[commit-bot: I haz the power, 2016-05-03 09:22:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-03 09:23:39]

Description was changed from

==========
Avoid style recalc and layout when not necessary for client size.

The clientWidth and clientHeight values do not rely on style recalc for
the documentElement in strict mode, and the body element in quirks mode
when we use overlay scrollbars. Skip the forced layout update in those
cases.

BUG=481412
==========

to

==========
Avoid style recalc and layout when not necessary for client size.

The clientWidth and clientHeight values do not rely on style recalc for
the documentElement in strict mode, and the body element in quirks mode
when we use overlay scrollbars. Skip the forced layout update in those
cases.

BUG=481412

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

[commit-bot: I haz the power, 2016-05-03 09:23:40]

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