Introduce BookmarkClient interface to abstract embedder

chrome/browser/bookmarks/bookmark_stats.h

Index: chrome/browser/bookmarks/bookmark_stats.h
diff --git a/chrome/browser/bookmarks/bookmark_stats.h b/chrome/browser/bookmarks/bookmark_stats.h
index af4d6ac0c25d0d058bac87ac001cfde9e2422818..6863ed6c5876c96afc9879ca16919220d0a7a9ff 100644
--- a/chrome/browser/bookmarks/bookmark_stats.h
+++ b/chrome/browser/bookmarks/bookmark_stats.h
@@ -5,6 +5,7 @@
 #ifndef CHROME_BROWSER_BOOKMARKS_BOOKMARK_STATS_H_

[sky, 2014/04/18 17:05:04]

Are you modifying this file in expectation of moving to components? I think it's
chrome specific and shouldn't need to move.

[sdefresne, 2014/04/18 22:25:49]

The function RecordBookmarkLaunch is currently used on iOS (with
BOOKMARK_LAUNCH_LOCATION_OMNIBOX).

I modified the file because I tried to remove all dependencies on content/ from
files that were in chrome/browser/bookmarks and were used by iOS. If you think
it is better not to move this file into the component, and I can revert the
corresponding changes.

I've reverted the changes for now, but we may have to consider how to handle
that dependency later.

 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_STATS_H_
 
+class BookmarkClient;
 class BookmarkNode;
 
 // This enum is used for the Bookmarks.EntryPoint histogram.
@@ -40,13 +41,16 @@ enum BookmarkLaunchLocation {
 };
 
 // Records the launch of a bookmark for UMA purposes.
-void RecordBookmarkLaunch(const BookmarkNode* node,
+void RecordBookmarkLaunch(BookmarkClient* client,
+                          const BookmarkNode* node,
                           BookmarkLaunchLocation location);
 
 // Records the user opening a folder of bookmarks for UMA purposes.
-void RecordBookmarkFolderOpen(BookmarkLaunchLocation location);
+void RecordBookmarkFolderOpen(BookmarkClient* client,
+                              BookmarkLaunchLocation location);
 
 // Records the user opening the apps page for UMA purposes.
-void RecordBookmarkAppsPageOpen(BookmarkLaunchLocation location);
+void RecordBookmarkAppsPageOpen(BookmarkClient* client,
+                                BookmarkLaunchLocation location);
 
 #endif  // CHROME_BROWSER_BOOKMARKS_BOOKMARK_STATS_H_