Always anchor to top of the anchor element

Always anchor to top of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

This CL anchors to the top left for ltr and vertical-lr writing mode and top
right for rtl and vertical-rl writing mode.

BUG=600891, 594877
Committed: https://crrev.com/d91478719a0603374c12efbd79d9cd248770e7ad
Cr-Commit-Position: refs/heads/master@{#392774}

[ymalik, 2016-05-09 18:31:48]

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

[skobes, 2016-05-09 18:39:21]

I wonder if we still need corner for RTL (crbug.com/594877)?

Also can you add a new test for a partially overlapping anchor node?

[ymalik, 2016-05-09 22:50:40]

Description was changed from

==========
Always anchor to top left corner of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

BUG=600891
==========

to

==========
Always anchor to top of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

This CL anchors to the top left for ltr and vertical-lr writing mode and top
right for rtl and vertical-rl writing mode.

BUG=600891,594877
==========

[skobes, 2016-05-10 01:25:11]

lgtm

[ymalik, 2016-05-10 17:04:07]

The CQ bit was checked by ymalik@chromium.org

[commit-bot: I haz the power, 2016-05-10 17:04:48]

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

[commit-bot: I haz the power, 2016-05-10 22:19:29]

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

[commit-bot: I haz the power, 2016-05-10 22:19:30]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[ymalik, 2016-05-10 22:28:12]

The CQ bit was checked by ymalik@chromium.org

[skobes, 2016-05-10 22:28:37]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2016-05-10 22:28:58]

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

[commit-bot: I haz the power, 2016-05-10 23:43:13]

Description was changed from

==========
Always anchor to top of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

This CL anchors to the top left for ltr and vertical-lr writing mode and top
right for rtl and vertical-rl writing mode.

BUG=600891,594877
==========

to

==========
Always anchor to top of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

This CL anchors to the top left for ltr and vertical-lr writing mode and top
right for rtl and vertical-rl writing mode.

BUG=600891,594877
==========

[commit-bot: I haz the power, 2016-05-10 23:43:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-10 23:44:13]

Description was changed from

==========
Always anchor to top of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

This CL anchors to the top left for ltr and vertical-lr writing mode and top
right for rtl and vertical-rl writing mode.

BUG=600891,594877
==========

to

==========
Always anchor to top of the anchor element

Anchoring to the bottom corner results in unnecessary adjustment content
is inserted above the anchor position but within the viewport. Always
anchoring to the top has the problem that we wont adjust when content is
inserted below the anchor position but above the viewport. The latter (not
anchoring at all) is better than anchoring to the wrong position.

This CL anchors to the top left for ltr and vertical-lr writing mode and top
right for rtl and vertical-rl writing mode.

BUG=600891,594877

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

[commit-bot: I haz the power, 2016-05-10 23:44:17]

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