Add PopularSites test.

Add PopularSites test.

There's a lot more here that ought to be tested, but this is a start.

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

[sfiera, 2016-11-04 10:44:16]

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

[sfiera, 2016-11-04 10:44:17]

You get all my reviews this week.

[markusheintz_, 2016-11-04 14:40:16]

LGTM

https://codereview.chromium.org/2471133004/diff/40001/components/ntp_tiles/po...
File components/ntp_tiles/popular_sites_unittest.cc (right):

https://codereview.chromium.org/2471133004/diff/40001/components/ntp_tiles/po...
components/ntp_tiles/popular_sites_unittest.cc:52: // Copied from iOS code.
Perhaps should be in a shared location.
I guess this qualifies as a TODO :-)

[sfiera, 2016-11-07 12:04:01]

The CQ bit was checked by sfiera@chromium.org

[commit-bot: I haz the power, 2016-11-07 12:04:10]

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-07 12:07:18]

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

[commit-bot: I haz the power, 2016-11-07 12:07:19]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sfiera, 2016-11-07 12:31:44]

The CQ bit was checked by sfiera@chromium.org

[sfiera, 2016-11-07 12:31:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
markusheintz@chromium.org
Link to the patchset: https://codereview.chromium.org/2471133004/#ps60001
(title: "Add test deps.")

[commit-bot: I haz the power, 2016-11-07 12:31:56]

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-07 13:34:11]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-07 13:36:03]

Description was changed from

==========
Add PopularSites test.

There's a lot more here that ought to be tested, but this is a start.
==========

to

==========
Add PopularSites test.

There's a lot more here that ought to be tested, but this is a start.

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

[commit-bot: I haz the power, 2016-11-07 13:36:04]

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