Use NTPTileSource in MVS interface.

Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and
generally as an interface.

Committed: https://crrev.com/b1815a338609f17686f54d66af9d7426c1893516
Cr-Commit-Position: refs/heads/master@{#418667}

[sfiera, 2016-09-14 01:29:43]

Description was changed from

==========
Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and in
generally as an interface.
==========

to

==========
Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and
generally as an interface.
==========

[sfiera, 2016-09-14 01:30:00]

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

[commit-bot: I haz the power, 2016-09-14 01:30:48]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sfiera, 2016-09-14 01:33:41]

sfiera@chromium.org changed reviewers:
+ treib@chromium.org

[sfiera, 2016-09-14 01:33:41]

Another small one.

[commit-bot: I haz the power, 2016-09-14 02:37:40]

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

[commit-bot: I haz the power, 2016-09-14 02:37:41]

Dry run: This issue passed the CQ dry run.

[Marc Treib, 2016-09-14 08:55:08]

LGTM with one optional suggestion.

https://codereview.chromium.org/2332353006/diff/1/components/ntp_tiles/most_v...
File components/ntp_tiles/most_visited_sites.h (right):

https://codereview.chromium.org/2332353006/diff/1/components/ntp_tiles/most_v...
components/ntp_tiles/most_visited_sites.h:135: int tile_type,
Also use the enum for tile_type?

[sfiera, 2016-09-14 19:09:42]

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

[sfiera, 2016-09-14 19:09:52]

https://codereview.chromium.org/2332353006/diff/1/components/ntp_tiles/most_v...
File components/ntp_tiles/most_visited_sites.h (right):

https://codereview.chromium.org/2332353006/diff/1/components/ntp_tiles/most_v...
components/ntp_tiles/most_visited_sites.h:135: int tile_type,
On 2016/09/14 08:55:08, Marc Treib wrote:
> Also use the enum for tile_type?

Done.

[commit-bot: I haz the power, 2016-09-14 19:11:04]

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-09-14 20:26:19]

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

[commit-bot: I haz the power, 2016-09-14 20:26:20]

Dry run: This issue passed the CQ dry run.

[sfiera, 2016-09-14 20:36:27]

The CQ bit was checked by sfiera@chromium.org

[sfiera, 2016-09-14 20:36:28]

The patchset sent to the CQ was uploaded after l-g-t-m from treib@chromium.org
Link to the patchset: https://codereview.chromium.org/2332353006/#ps20001
(title: "Use tile type enum too.")

[commit-bot: I haz the power, 2016-09-14 20:37:21]

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-09-14 20:52:34]

Description was changed from

==========
Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and
generally as an interface.
==========

to

==========
Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and
generally as an interface.
==========

[commit-bot: I haz the power, 2016-09-14 20:52:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-14 20:54:01]

Description was changed from

==========
Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and
generally as an interface.
==========

to

==========
Use NTPTileSource in MVS interface.

MostVisitedSitesBridge can handle conversion between int and the enum
for the sake of Java. Using the enum itself is better for iOS, and
generally as an interface.

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

[commit-bot: I haz the power, 2016-09-14 20:54:02]

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