Revert of [Android NTP] Allow overlapping snap scroll regions.

Revert of [Android NTP] Allow overlapping snap scroll regions. (patchset #3 id:40001 of https://codereview.chromium.org/2708653002/ )

Reason for revert:
NewTabPageRecyclerViewTest#testSnapScrollCondensedLayout failing on tablets

BUG=694851
Original issue's description:
> [Android NTP] Allow overlapping snap scroll regions.
> 
> Previously the code assumed that snap scroll regions would never overlap.
> That assumption became false with the condensed NTP layout, with the result that
> only the first snap would be applied and the NTP would come to rest in a
> position it should have scrolled out of.
> 
> We now apply all snaps in order and have an assertion that the desired position
> does not change after one pass.
> 
> BUG=693095
> 
> Review-Url: https://codereview.chromium.org/2708653002
> Cr-Commit-Position: refs/heads/master@{#451767}
> Committed: https://chromium.googlesource.com/chromium/src/+/6617e48c6e0733436a8fc430a17399038975926e

TBR=mvanouwerkerk@chromium.org,peconn@chromium.org,bauerb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=693095
Review-Url: https://codereview.chromium.org/2705343003
Cr-Commit-Position: refs/heads/master@{#451861}
Committed: https://chromium.googlesource.com/chromium/src/+/8a0be9febc6a33b487eaade15f7ec08cdd5e6b0d

[aelias_OOO_until_Jul13, 2017-02-22 00:59:39]

The CQ bit was checked by aelias@chromium.org

[aelias_OOO_until_Jul13, 2017-02-22 00:59:39]

Created Revert of [Android NTP] Allow overlapping snap scroll regions.

[commit-bot: I haz the power, 2017-02-22 01:01:00]

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, 2017-02-22 01:05:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487725179339720, "parent_rev":
"0068bc112d1ffb74bb19a28641bab499766854d7", "commit_rev":
"8a0be9febc6a33b487eaade15f7ec08cdd5e6b0d"}

[commit-bot: I haz the power, 2017-02-22 01:06:52]

Description was changed from

==========
Revert of [Android NTP] Allow overlapping snap scroll regions. (patchset #3
id:40001 of https://codereview.chromium.org/2708653002/ )

Reason for revert:
NewTabPageRecyclerViewTest#testSnapScrollCondensedLayout failing on tablets

BUG=694851

Original issue's description:
> [Android NTP] Allow overlapping snap scroll regions.
> 
> Previously the code assumed that snap scroll regions would never overlap.
> That assumption became false with the condensed NTP layout, with the result
that
> only the first snap would be applied and the NTP would come to rest in a
> position it should have scrolled out of.
> 
> We now apply all snaps in order and have an assertion that the desired
position
> does not change after one pass.
> 
> BUG=693095
> 
> Review-Url: https://codereview.chromium.org/2708653002
> Cr-Commit-Position: refs/heads/master@{#451767}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6617e48c6e0733436a8fc430a173...

TBR=mvanouwerkerk@chromium.org,peconn@chromium.org,bauerb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=693095
==========

to

==========
Revert of [Android NTP] Allow overlapping snap scroll regions. (patchset #3
id:40001 of https://codereview.chromium.org/2708653002/ )

Reason for revert:
NewTabPageRecyclerViewTest#testSnapScrollCondensedLayout failing on tablets

BUG=694851

Original issue's description:
> [Android NTP] Allow overlapping snap scroll regions.
> 
> Previously the code assumed that snap scroll regions would never overlap.
> That assumption became false with the condensed NTP layout, with the result
that
> only the first snap would be applied and the NTP would come to rest in a
> position it should have scrolled out of.
> 
> We now apply all snaps in order and have an assertion that the desired
position
> does not change after one pass.
> 
> BUG=693095
> 
> Review-Url: https://codereview.chromium.org/2708653002
> Cr-Commit-Position: refs/heads/master@{#451767}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6617e48c6e0733436a8fc430a173...

TBR=mvanouwerkerk@chromium.org,peconn@chromium.org,bauerb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=693095

Review-Url: https://codereview.chromium.org/2705343003
Cr-Commit-Position: refs/heads/master@{#451861}
Committed:
https://chromium.googlesource.com/chromium/src/+/8a0be9febc6a33b487eaade15f7e...
==========

[commit-bot: I haz the power, 2017-02-22 01:06:53]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8a0be9febc6a33b487eaade15f7e...