Introduce ChromeBookmarkClientFactory

Introduce ChromeBookmarkClientFactory

Fix BookmarkModelFactory to really build a BookmarkModel object and
introduce another factory ChromeBookmarkClientFactory to build the
ChromeBookmarkClient and manage the correct dependency between the
two objects.

Change BookmarkModelFactory::GetChromeBookmarkClientForProfile and
clients to instead use ChromeBookmarkClientFactory::GetForProfile.

BUG=364865
TBR=blundell@chromium.org

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

[Joao da Silva, 2014-06-13 17:05:35]

Sylvain, I'm currently blocked by https://codereview.chromium.org/306293006
which isn't applying cleanly. This CL is a rebase of yours that should land
cleanly now, hope you don't mind.

[Joao da Silva, 2014-06-13 17:05:43]

The CQ bit was checked by joaodasilva@chromium.org

[commit-bot: I haz the power, 2014-06-13 17:06:35]

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

[commit-bot: I haz the power, 2014-06-13 21:51:23]

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

[commit-bot: I haz the power, 2014-06-14 00:11:37]

Change committed as 277138

[sdefresne, 2014-06-16 08:14:41]

LGTM

Thank you for re-landing.