[Popular Sites] Move PopularSitesImpl to dedicated file

[Popular Sites] Move PopularSitesImpl to dedicated file

Trivial follow-up to https://codereview.chromium.org/2570783003/ where
it was suggested to move the new class to a dedicated file.

BUG=619584
Committed: https://crrev.com/b06f14242378cb8ce63fd7ee3ec89d4287757c49
Cr-Commit-Position: refs/heads/master@{#438811}

[mastiz, 2016-12-14 20:16:24]

mastiz@chromium.org changed reviewers:
+ bauerb@chromium.org, sdefresne@chromium.org

[mastiz, 2016-12-14 20:16:24]

bauerb@chromium.org: Please review changes in all dirs except ios/

sdefresne@chromium.org: Please review changes in ios/

[sdefresne, 2016-12-15 10:43:38]

lgtm (reviewed everything, sorry)

[Bernhard Bauer, 2016-12-15 11:18:01]

lgtm

[mastiz, 2016-12-15 11:18:54]

The CQ bit was checked by mastiz@chromium.org

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

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-12-15 11:23:21]

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

[commit-bot: I haz the power, 2016-12-15 11:23:21]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)

[mastiz, 2016-12-15 11:31:24]

The CQ bit was checked by mastiz@chromium.org

[mastiz, 2016-12-15 11:31:25]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org,
sdefresne@chromium.org
Link to the patchset: https://codereview.chromium.org/2575823002/#ps40001
(title: "Added missing forward declaration.")

[commit-bot: I haz the power, 2016-12-15 11:31:48]

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-12-15 12:32:23]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481801484658980,
"parent_rev": "6dd3c21ed1388f98acd2360b0eb7233125af79e2", "commit_rev":
"d2d54d8444189df4f3d5336b18f03fc12bb6e0dd"}

[commit-bot: I haz the power, 2016-12-15 12:32:52]

Description was changed from

==========
[Popular Sites] Move PopularSitesImpl to dedicated file

Trivial follow-up to https://codereview.chromium.org/2570783003/ where
it was suggested to move the new class to a dedicated file.

BUG=619584
==========

to

==========
[Popular Sites] Move PopularSitesImpl to dedicated file

Trivial follow-up to https://codereview.chromium.org/2570783003/ where
it was suggested to move the new class to a dedicated file.

BUG=619584

Review-Url: https://codereview.chromium.org/2575823002
==========

[commit-bot: I haz the power, 2016-12-15 12:32:53]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-15 12:34:46]

Description was changed from

==========
[Popular Sites] Move PopularSitesImpl to dedicated file

Trivial follow-up to https://codereview.chromium.org/2570783003/ where
it was suggested to move the new class to a dedicated file.

BUG=619584

Review-Url: https://codereview.chromium.org/2575823002
==========

to

==========
[Popular Sites] Move PopularSitesImpl to dedicated file

Trivial follow-up to https://codereview.chromium.org/2570783003/ where
it was suggested to move the new class to a dedicated file.

BUG=619584

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

[commit-bot: I haz the power, 2016-12-15 12:34:47]

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