bookmarks: Componentize bookmark_model_unittest.cc.

bookmarks: Componentize bookmark_model_unittest.cc.

Few changes involved:

1) Using TestBookmarkClient to create BookmarkModel.
2) Removes unnecessary TestBrowserThreadBundle
3) With the above changes TestingProfile and BookmarkModelFactory
goes away.

BUG=367834
TEST=components_unittests --gtest_filter=BookmarkModel*
R=blundell@chromium.org, sdefresne@chromium.org

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

[tfarina, 2014-05-16 02:56:13]

minimal changes. ptal.

[blundell, 2014-05-16 07:17:49]

Trybots seem unhappy

[sdefresne, 2014-05-16 09:43:24]

Change looks fine.

I think you forgot to remove browser/bookmarks/bookmark_model_unittests.cc from
chrome/chrome_tests_unit.gypi and this is what causes the try bot failures.

[tfarina, 2014-05-16 15:37:46]

On 2014/05/16 09:43:24, sdefresne wrote:
> Change looks fine.
> 
> I think you forgot to remove browser/bookmarks/bookmark_model_unittests.cc
from
> chrome/chrome_tests_unit.gypi and this is what causes the try bot failures.

[tfarina, 2014-05-16 15:39:00]

On 2014/05/16 15:37:46, tfarina wrote:
> On 2014/05/16 09:43:24, sdefresne wrote:
> > Change looks fine.
> > 
> > I think you forgot to remove browser/bookmarks/bookmark_model_unittests.cc
> from
> > chrome/chrome_tests_unit.gypi and this is what causes the try bot failures.

Ops, that is what I get for doing this too late :/

It should be fixed in Patch set 2 (thanks for catching this). ptal.

thanks,

[blundell, 2014-05-16 15:42:37]

LGTM

[sdefresne, 2014-05-16 16:05:42]

lgtm

[tfarina, 2014-05-16 18:28:13]

Committed patchset #2 manually as r271064 (presubmit successful).