Inject CanAddURLToHistory into TopSitesImpl

Inject CanAddURLToHistory into TopSitesImpl

Remove dependency on //chrome/browser/history/history_utils.h by
injecting the CanAddURLToHistory function (via a base::Callback)
into TopSitesImpl.

BUG=479174
Committed: https://crrev.com/577751e8c060b6b9a80f1a160518cff14a7dfc3e
Cr-Commit-Position: refs/heads/master@{#327014}

[sdefresne, 2015-04-21 16:17:48]

sdefresne@chromium.org changed reviewers:
+ droger@chromium.org

[sdefresne, 2015-04-21 16:17:49]

PTAL

[droger, 2015-04-22 08:38:17]

LGTM

Optional:

I see that CanAddURLToHistory does:

if (!url.is_valid())
    return false;

It seems that this check could be done by the component, as it does not require
any knowledge from the embedder.

We cannot remove the check from CanAddURLToHistory, because there are other
callers, but we could duplicate the check in the component and see later if we
can remove it from CanAddURLToHistory().

[sdefresne, 2015-04-24 12:50:59]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2015-04-24 12:51:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100763002/1

[commit-bot: I haz the power, 2015-04-24 12:54:28]

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

[commit-bot: I haz the power, 2015-04-24 12:54:29]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[sdefresne, 2015-04-24 12:56:46]

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

[commit-bot: I haz the power, 2015-04-24 12:56:51]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100763002/20001

[commit-bot: I haz the power, 2015-04-24 13:13:07]

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

[commit-bot: I haz the power, 2015-04-24 13:13:08]

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

[sdefresne, 2015-04-24 15:09:51]

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

[commit-bot: I haz the power, 2015-04-24 15:10:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100763002/40001

[sdefresne, 2015-04-24 15:10:43]

sdefresne@chromium.org changed reviewers:
+ sky@chromium.org

[sdefresne, 2015-04-24 15:10:44]

sky: OWNERS for 
  chrome/browser/thumbnails/thumbnail_service_unittest.cc

[commit-bot: I haz the power, 2015-04-24 15:23:31]

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

[commit-bot: I haz the power, 2015-04-24 15:23:32]

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

[sky, 2015-04-24 18:07:20]

LGTM

[sdefresne, 2015-04-25 11:13:20]

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

[commit-bot: I haz the power, 2015-04-25 11:13:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100763002/60001

[commit-bot: I haz the power, 2015-04-25 13:09:53]

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

[commit-bot: I haz the power, 2015-04-25 13:09:54]

Dry run: This issue passed the CQ dry run.

[sdefresne, 2015-04-27 07:59:48]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2015-04-27 07:59:48]

The patchset sent to the CQ was uploaded after l-g-t-m from droger@chromium.org,
sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1100763002/#ps60001
(title: "Fix rebase issue")

[commit-bot: I haz the power, 2015-04-27 07:59:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100763002/60001

[commit-bot: I haz the power, 2015-04-27 08:40:23]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-04-27 08:41:15]

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

[sdefresne, 2015-04-28 14:19:31]

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