Clamping shouldn't clear the fragment anchor.

Clamping shouldn't clear the fragment anchor.

Also rename fragment anchor tests for clarity.

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

Committed: https://crrev.com/1c73e92a39a2e894b32a318900d0730f8ae7b68e
Cr-Commit-Position: refs/heads/master@{#429581}

[skobes, 2016-11-02 23:39:49]

Description was changed from

==========
Clamping shouldn't clear the fragment anchor.

BUG=661359
==========

to

==========
Clamping shouldn't clear the fragment anchor.

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

[skobes, 2016-11-02 23:39:55]

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

[commit-bot: I haz the power, 2016-11-02 23:40:38]

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

[skobes, 2016-11-03 00:21:51]

Description was changed from

==========
Clamping shouldn't clear the fragment anchor.

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

to

==========
Clamping shouldn't clear the fragment anchor.

Also rename fragment anchor tests for clarity.

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

[skobes, 2016-11-03 00:22:44]

skobes@chromium.org changed reviewers:
+ bokan@chromium.org

[skobes, 2016-11-03 00:22:50]

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

[commit-bot: I haz the power, 2016-11-03 00:23:19]

Dry run: 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-11-03 02:55:35]

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

[commit-bot: I haz the power, 2016-11-03 02:55:37]

Dry run: This issue passed the CQ dry run.

[bokan, 2016-11-03 11:19:12]

lgtm

https://codereview.chromium.org/2474813002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/loader/clamping-preserves-fragment-anchor.html
(right):

https://codereview.chromium.org/2474813002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/loader/clamping-preserves-fragment-anchor.html:30:
document.body.className = "shrink";
This both shrinks the body and moves the <a>. Maybe split this up into two
statements that explicitly shrink the body first, then move the <a> up? It'd be
less dependent on the order the clamp and fragment anchoring happen.

[skobes, 2016-11-03 13:31:03]

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

[commit-bot: I haz the power, 2016-11-03 13:31:40]

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

[skobes, 2016-11-03 13:31:42]

skobes@chromium.org changed reviewers:
+ jbroman@chromium.org

[skobes, 2016-11-03 13:31:42]

+jbroman for Source/platform/OWNERS

https://codereview.chromium.org/2474813002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/loader/clamping-preserves-fragment-anchor.html
(right):

https://codereview.chromium.org/2474813002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/loader/clamping-preserves-fragment-anchor.html:30:
document.body.className = "shrink";
On 2016/11/03 11:19:12, bokan wrote:
> This both shrinks the body and moves the <a>. Maybe split this up into two
> statements that explicitly shrink the body first, then move the <a> up? It'd
be
> less dependent on the order the clamp and fragment anchoring happen.

Done.

[jbroman, 2016-11-03 14:20:16]

platform lgtm

[skobes, 2016-11-03 14:33:19]

The CQ bit was unchecked by skobes@chromium.org

[skobes, 2016-11-03 14:33:44]

The CQ bit was checked by skobes@chromium.org

[skobes, 2016-11-03 14:33:45]

The patchset sent to the CQ was uploaded after l-g-t-m from bokan@chromium.org
Link to the patchset: https://codereview.chromium.org/2474813002/#ps40001
(title: "address review comments")

[commit-bot: I haz the power, 2016-11-03 14:34:07]

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-11-03 14:48:07]

Description was changed from

==========
Clamping shouldn't clear the fragment anchor.

Also rename fragment anchor tests for clarity.

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

to

==========
Clamping shouldn't clear the fragment anchor.

Also rename fragment anchor tests for clarity.

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

[commit-bot: I haz the power, 2016-11-03 14:48:10]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-03 14:50:21]

Description was changed from

==========
Clamping shouldn't clear the fragment anchor.

Also rename fragment anchor tests for clarity.

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

to

==========
Clamping shouldn't clear the fragment anchor.

Also rename fragment anchor tests for clarity.

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

Committed: https://crrev.com/1c73e92a39a2e894b32a318900d0730f8ae7b68e
Cr-Commit-Position: refs/heads/master@{#429581}
==========

[commit-bot: I haz the power, 2016-11-03 14:50:22]

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