Componentize WebHistoryService

Componentize WebHistoryService

Instead of using the factories to retrieve the dependended upon keyed service,
pass them explicitly to WebHistoryService from the WebHistoryServiceFactory.

Fix the dependency of WebHistoryService to add the service used and to remove
the service unused.

Move file into //components/history/core/browser, remove unnecessary #include
and fix files that did not #include all the files they used.

BUG=371835, 454524
Committed: https://crrev.com/ff13143269bcc9010afc9490fc83147d5fad5a81
Cr-Commit-Position: refs/heads/master@{#316622}

[sdefresne, 2015-02-10 17:29:42]

sdefresne@chromium.org changed reviewers:
+ droger@chromium.org, jhawkins@chromium.org, rlp@chromium.org,
rogerta@chromium.org

[sdefresne, 2015-02-10 17:29:43]

This is a reland of https://codereview.chromium.org/882753002 with a fix for the
crash introduced (http://crbug/454524). The crash was due to request_context_
not being initialized in the constructor of RequestImpl.

First patch is the same a original CL rebased on ToT, while second patch
includes the fix (and a DEPS fix).

droger: please take a look

rogerta: please review new DEPS on //google/apis and //components/signin as
OWNERS

rlp: please review //chrome/browser/extensions and //chrome/browser/search as
OWNERS

jhawkins: please review //chrome/browser/ui/webui as OWNERS

[droger, 2015-02-11 12:31:08]

lgtm

[James Hawkins, 2015-02-11 17:35:59]

lgtm

[Roger Tawa OOO till Jul 10th, 2015-02-11 18:10:14]

lgtm for deps

[sdefresne, 2015-02-16 09:32:03]

rlp: ping for

- chrome/browser/extensions/api/hotword_private
- chrome/browser/search/hotword_*

[sdefresne, 2015-02-17 10:15:27]

New patchsets have been uploaded after l-g-t-m from
droger@chromium.org,jhawkins@chromium.org,rogerta@chromium.org

[sdefresne, 2015-02-17 10:17:54]

rlp: ping?

[rpetterson, 2015-02-17 18:11:26]

sorry, thought I had reviewed this before.

LGTM

[sdefresne, 2015-02-17 18:12:16]

On 2015/02/17 at 18:11:26, rlp wrote:
> sorry, thought I had reviewed this before.
> 
> LGTM

No problem. This was a reland, of a CL that you reviewed, but I couldn't reuse
the original CL. Sorry about that.

[sdefresne, 2015-02-17 18:12:25]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2015-02-17 18:13:22]

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

[commit-bot: I haz the power, 2015-02-17 19:23:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-17 19:24:13]

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