Partially move history_types.{cc,h} to //components/history

Partially move history_types.{cc,h} to //components/history

Move all types from //chrome/browser/history/history_types.{cc,h} that do
not depends on content types to //components/history.

BUG=371816
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288852

[sdefresne, 2014-08-06 13:07:48]

Reland of https://codereview.chromium.org/439713004/

Some notes:
1. I don't think that using a width/height pair instead of gfx::Size is worth it
as //components/history will have to eventually depends on //ui/gfx because of
tops_sites_impl usage of gfx::Image
2. //components/history is expected to be a layered components
(http://www.chromium.org/developers/design-documents/layered-components-design)
thus the //components/history/core subdirectory, if this turn out to be
unnecessary, I'll flatten the component structure (but I'll only be able to
confirm this later)

brettw: for the whole code move

jhawkins: for files affected by the file move (#include changes) in
//chrome/browser/ui/webui, //chrome/browser/jumplist_win.cc and change to
//chrome/chrome_browser.gypi

blundell: for the new dependency on //components/favicon_base (btw,
//components/favicon_base has no OWNERS)

msw: for the new dependency on //ui/gfx

[blundell, 2014-08-06 13:16:20]

LGTM

[msw, 2014-08-06 19:13:46]

On 2014/08/06 13:07:48, sdefresne wrote:
> 1. I don't think that using a width/height pair instead of gfx::Size is worth
it
> as //components/history will have to eventually depends on //ui/gfx because of
> tops_sites_impl usage of gfx::Image
>
> msw: for the new dependency on //ui/gfx

Shrug, adding the ui/gfx dependency lgtm then.

[brettw, 2014-08-06 20:41:22]

LGTM rubberstamp

[sdefresne, 2014-08-08 16:31:29]

On 2014/08/06 20:41:22, brettw wrote:
> LGTM rubberstamp

jhawkins: ping

[James Hawkins, 2014-08-11 13:50:43]

lgtm

[sdefresne, 2014-08-11 16:45:52]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2014-08-11 16:46:29]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sdefresne@chromium.org/448653003/1

[commit-bot: I haz the power, 2014-08-11 23:16:59]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)

[commit-bot: I haz the power, 2014-08-12 00:59:48]

Change committed as 288852