Rename positionNewFloats() to placeNewFloats().

Rename positionNewFloats() to placeNewFloats().

This will distinguish it better from the method named "positionAndLayoutFloat".

Also be explicit about the fact that we use the top margin edge when
positioning floats, as opposed to the top border edge, which is common for all
other object types. So "logicalTop" usually means the logical top of the border
edge. Therefore, use "logicalTopMarginEdge" for floats.

No behavioral changes, just cleanup.

BUG=663942
Committed: https://crrev.com/7892652d84a2a704b5a6118806886720dc8c7127
Cr-Commit-Position: refs/heads/master@{#432895}

[mstensho (USE GERRIT), 2016-11-17 12:49:35]

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

[commit-bot: I haz the power, 2016-11-17 12:49:44]

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-17 14:16:43]

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

[commit-bot: I haz the power, 2016-11-17 14:16:44]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-11-17 14:25:55]

mstensho@opera.com changed reviewers:
+ szager@chromium.org

[szager1, 2016-11-17 15:48:47]

lgtm

[mstensho (USE GERRIT), 2016-11-17 16:42:09]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-11-17 16:42:40]

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-17 16:49:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 16:52:55]

Description was changed from

==========
Rename positionNewFloats() to placeNewFloats().

This will distinguish it better from the method named "positionAndLayoutFloat".

Also be explicit about the fact that we use the top margin edge when
positioning floats, as opposed to the top border edge, which is common for all
other object types. So "logicalTop" usually means the logical top of the border
edge. Therefore, use "logicalTopMarginEdge" for floats.

No behavioral changes, just cleanup.

BUG=663942
==========

to

==========
Rename positionNewFloats() to placeNewFloats().

This will distinguish it better from the method named "positionAndLayoutFloat".

Also be explicit about the fact that we use the top margin edge when
positioning floats, as opposed to the top border edge, which is common for all
other object types. So "logicalTop" usually means the logical top of the border
edge. Therefore, use "logicalTopMarginEdge" for floats.

No behavioral changes, just cleanup.

BUG=663942

Committed: https://crrev.com/7892652d84a2a704b5a6118806886720dc8c7127
Cr-Commit-Position: refs/heads/master@{#432895}
==========

[commit-bot: I haz the power, 2016-11-17 16:52:56]

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