Introduce BookmarkClient interface to abstract embedder

chrome/browser/ui/bookmarks/bookmark_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/bind.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/timer/timer.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
@@ skipping 62 matching lines @@
   chrome::ExecuteCommand(browser(), IDC_SHOW_BOOKMARK_BAR);
   delay = WaitForBookmarkBarAnimationToFinish();
   LOG(INFO) << "Took " << delay.InMilliseconds() << " ms to hide bookmark bar";
   ASSERT_FALSE(IsVisible());
 }
 
 // Verify that bookmarks persist browser restart.
 IN_PROC_BROWSER_TEST_F(BookmarkBrowsertest, PRE_Persist) {
   BookmarkModel* bookmark_model = WaitForBookmarkModel(browser()->profile());
 
-  bookmark_utils::AddIfNotBookmarked(
-      bookmark_model, GURL(kPersistBookmarkURL),
-      base::ASCIIToUTF16(kPersistBookmarkTitle));
+  bookmark_utils::AddIfNotBookmarked(bookmark_model,
+                                     bookmark_model,
+                                     GURL(kPersistBookmarkURL),
+                                     base::ASCIIToUTF16(kPersistBookmarkTitle));
 }
 
 #if defined(THREAD_SANITIZER)
 // BookmarkBrowsertest.Persist fails under ThreadSanitizer on Linux, see
 // http://crbug.com/340223.
 #define MAYBE_Persist DISABLED_Persist
 #else
 #define MAYBE_Persist Persist
 #endif
 IN_PROC_BROWSER_TEST_F(BookmarkBrowsertest, MAYBE_Persist) {
@@ skipping 17 matching lines @@
 
   BookmarkModel* bookmark_model1 = WaitForBookmarkModel(browser()->profile());
 
   ui_test_utils::BrowserAddedObserver observer;
   g_browser_process->profile_manager()->CreateMultiProfileAsync(
       base::string16(), base::string16(), ProfileManager::CreateCallback(),
       std::string());
   Browser* browser2 = observer.WaitForSingleNewBrowser();
   BookmarkModel* bookmark_model2 = WaitForBookmarkModel(browser2->profile());
 
-  bookmark_utils::AddIfNotBookmarked(
-      bookmark_model1, GURL(kPersistBookmarkURL),
-      base::ASCIIToUTF16(kPersistBookmarkTitle));
+  bookmark_utils::AddIfNotBookmarked(bookmark_model1,
+                                     bookmark_model1,
+                                     GURL(kPersistBookmarkURL),
+                                     base::ASCIIToUTF16(kPersistBookmarkTitle));
   std::vector<BookmarkService::URLAndTitle> urls1, urls2;
   bookmark_model1->GetBookmarks(&urls1);
   bookmark_model2->GetBookmarks(&urls2);
   ASSERT_EQ(1u, urls1.size());
   ASSERT_TRUE(urls2.empty());
 }
 
 #endif