Don't restore in LayoutBlock::layout if clamping is delayed.

Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/0cd0e9c225f682449743476969ced1ab51e6b1eb
Cr-Commit-Position: refs/heads/master@{#412450}

[skobes, 2016-08-12 22:05:13]

Description was changed from

==========
[WIP] Fix dev tools.

BUG=624534
==========

to

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
==========

[skobes, 2016-08-12 22:05:19]

Patchset #1 (id:1) has been deleted

[skobes, 2016-08-16 23:45:02]

Description was changed from

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
==========

to

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[skobes, 2016-08-16 23:45:07]

The CQ bit was checked by skobes@chromium.org to run a CQ dry run

[skobes, 2016-08-16 23:45:17]

Patchset #1 (id:20001) has been deleted

[commit-bot: I haz the power, 2016-08-16 23:45:34]

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

[skobes, 2016-08-16 23:45:48]

Description was changed from

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534,629269
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[skobes, 2016-08-17 00:23:41]

Description was changed from

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534,629269
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[skobes, 2016-08-17 01:06:56]

skobes@chromium.org changed reviewers:
+ szager@chromium.org, ymalik@chromium.org

[szager1, 2016-08-17 01:09:48]

lgtm

[ymalik, 2016-08-17 01:23:22]

lgtm

[ymalik, 2016-08-17 01:25:28]

https://codereview.chromium.org/2220133002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/layout/LayoutBlock.cpp (right):

https://codereview.chromium.org/2220133002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/layout/LayoutBlock.cpp:383: // If clamping is
delayed, we will restore in
PaintLayerScrollableArea::clampScrollPositionsAfterLayout.
nit: can you add a comment here why we need to special case delayed clamping?

[skobes, 2016-08-17 01:27:48]

https://codereview.chromium.org/2220133002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/layout/LayoutBlock.cpp (right):

https://codereview.chromium.org/2220133002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/layout/LayoutBlock.cpp:383: // If clamping is
delayed, we will restore in
PaintLayerScrollableArea::clampScrollPositionsAfterLayout.
On 2016/08/17 01:25:28, ymalik wrote:
> nit: can you add a comment here why we need to special case delayed clamping?

Done.

[skobes, 2016-08-17 01:27:51]

The CQ bit was checked by skobes@chromium.org

[skobes, 2016-08-17 01:27:51]

The patchset sent to the CQ was uploaded after l-g-t-m from ymalik@chromium.org,
szager@chromium.org
Link to the patchset: https://codereview.chromium.org/2220133002/#ps60001
(title: "add comment")

[commit-bot: I haz the power, 2016-08-17 01:28:16]

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-08-17 04:40:33]

Description was changed from

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-08-17 04:40:34]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-08-17 04:42:41]

Description was changed from

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't restore in LayoutBlock::layout if clamping is delayed.

BUG=624534
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/0cd0e9c225f682449743476969ced1ab51e6b1eb
Cr-Commit-Position: refs/heads/master@{#412450}
==========

[commit-bot: I haz the power, 2016-08-17 04:42:42]

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