[Home] Make the new NTP over the tab switcher design the default

[Home] Make the new NTP over the tab switcher design the default

Removes the flags guarding the new NTP over the tab switcher design
and replaces ChromeHomeNewTabPageTest with
BottomSheetNewTabControllerTest.

BUG=695973
Review-Url: https://codereview.chromium.org/2891343003
Cr-Commit-Position: refs/heads/master@{#473751}
Committed: https://chromium.googlesource.com/chromium/src/+/075c148ae5e1e965596c1456c68704c1443df3e6

[Theresa, 2017-05-19 19:05:21]

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

[commit-bot: I haz the power, 2017-05-19 19:06:04]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Theresa, 2017-05-19 19:07:23]

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

[Theresa, 2017-05-19 19:07:24]

ptal

https://codereview.chromium.org/2891343003/diff/20001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetNewTabControllerTest.java
(right):

https://codereview.chromium.org/2891343003/diff/20001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetNewTabControllerTest.java:80:
// The sheet should now be closed and a regular tab should have been created
nit: add a period to the end of this line and 122

[commit-bot: I haz the power, 2017-05-19 22:04:07]

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

[commit-bot: I haz the power, 2017-05-19 22:04:08]

Dry run: This issue passed the CQ dry run.

[mdjones, 2017-05-22 21:34:13]

lgtm

[Theresa, 2017-05-22 21:46:01]

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

[Theresa, 2017-05-22 21:46:02]

+tedchoc@ for OWNERS

[Ted C, 2017-05-22 22:49:41]

lgtm

[Theresa, 2017-05-22 22:54:46]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2017-05-22 22:55:19]

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-05-23 00:00:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495493686146720,
"parent_rev": "ee69e6f69b1498d2f7b91a8e8ffc2a77603951e8", "commit_rev":
"075c148ae5e1e965596c1456c68704c1443df3e6"}

[commit-bot: I haz the power, 2017-05-23 00:00:50]

Description was changed from

==========
[Home] Make the new NTP over the tab switcher design the default

Removes the flags guarding the new NTP over the tab switcher design
and replaces ChromeHomeNewTabPageTest with
BottomSheetNewTabControllerTest.

BUG=695973
==========

to

==========
[Home] Make the new NTP over the tab switcher design the default

Removes the flags guarding the new NTP over the tab switcher design
and replaces ChromeHomeNewTabPageTest with
BottomSheetNewTabControllerTest.

BUG=695973

Review-Url: https://codereview.chromium.org/2891343003
Cr-Commit-Position: refs/heads/master@{#473751}
Committed:
https://chromium.googlesource.com/chromium/src/+/075c148ae5e1e965596c1456c687...
==========

[commit-bot: I haz the power, 2017-05-23 00:00:50]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/075c148ae5e1e965596c1456c687...