Reland [Home] Close the BottomSheet when a URL is loaded on the NTP

Reland [Home] Close the BottomSheet when a URL is loaded on the NTP

This ensures that a click on the doodle will close the BottomSheet.
This reverts commit 38ce623f9ffd74711a06046ff47d371e5ca5a48c.

Previously the ChromeHomeNewTabPageTest cases were failing because the
sheet state was set twice causing the state change observer to be
notified while the sheet was still SCROLLING and not at its final
resting place. This patch is the same as the original commit except
that it ChromeHomeNewTabPageBase does not set the sheet state to
PEEK if the URL being loaded is the NTP.

BUG=712607
Review-Url: https://codereview.chromium.org/2831773002
Cr-Commit-Position: refs/heads/master@{#466006}
Committed: https://chromium.googlesource.com/chromium/src/+/2e59ca5cb92b38cd1fbf209c728f3cb3487b63f0

[Theresa, 2017-04-19 21:38:19]

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

[Theresa, 2017-04-19 21:38:52]

twellington@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Theresa, 2017-04-19 21:38:53]

ptal

[commit-bot: I haz the power, 2017-04-19 21:39:01]

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, 2017-04-19 22:28:31]

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

[commit-bot: I haz the power, 2017-04-19 22:28:32]

Dry run: This issue passed the CQ dry run.

[Michael van Ouwerkerk, 2017-04-20 10:59:40]

lgtm

[Theresa, 2017-04-20 14:48:18]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2017-04-20 14:48:32]

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-04-20 14:51:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492699698311460, "parent_rev":
"975b115ff82c6d407cd346afb4d4838f5d380eab", "commit_rev":
"2e59ca5cb92b38cd1fbf209c728f3cb3487b63f0"}

[commit-bot: I haz the power, 2017-04-20 14:52:05]

Description was changed from

==========
Reland [Home] Close the BottomSheet when a URL is loaded on the NTP

This ensures that a click on the doodle will close the BottomSheet.
This reverts commit 38ce623f9ffd74711a06046ff47d371e5ca5a48c.

Previously the ChromeHomeNewTabPageTest cases were failing because the
sheet state was set twice causing the state change observer to be
notified while the sheet was still SCROLLING and not at its final
resting place. This patch is the same as the original commit except
that it ChromeHomeNewTabPageBase does not set the sheet state to
PEEK if the URL being loaded is the NTP.

BUG=712607
==========

to

==========
Reland [Home] Close the BottomSheet when a URL is loaded on the NTP

This ensures that a click on the doodle will close the BottomSheet.
This reverts commit 38ce623f9ffd74711a06046ff47d371e5ca5a48c.

Previously the ChromeHomeNewTabPageTest cases were failing because the
sheet state was set twice causing the state change observer to be
notified while the sheet was still SCROLLING and not at its final
resting place. This patch is the same as the original commit except
that it ChromeHomeNewTabPageBase does not set the sheet state to
PEEK if the URL being loaded is the NTP.

BUG=712607

Review-Url: https://codereview.chromium.org/2831773002
Cr-Commit-Position: refs/heads/master@{#466006}
Committed:
https://chromium.googlesource.com/chromium/src/+/2e59ca5cb92b38cd1fbf209c728f...
==========

[commit-bot: I haz the power, 2017-04-20 14:52:06]

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