[New CL] Add a tab helper to record the last visit date for each bookmark

chrome/browser/bookmarks/bookmark_model_factory.h

Index: chrome/browser/bookmarks/bookmark_model_factory.h
diff --git a/chrome/browser/bookmarks/bookmark_model_factory.h b/chrome/browser/bookmarks/bookmark_model_factory.h
index b4a67bf28d8d44e18baf3eed6967dd1027a254c4..113453eeaff4178aff88c733e8eff895dc15db41 100644
--- a/chrome/browser/bookmarks/bookmark_model_factory.h
+++ b/chrome/browser/bookmarks/bookmark_model_factory.h
@@ -21,6 +21,9 @@ class BookmarkModel;
 // Singleton that owns all BookmarkModels and associates them with Profiles.
 class BookmarkModelFactory : public BrowserContextKeyedServiceFactory {
  public:
+  static bookmarks::BookmarkModel* GetForBrowserContext(
+      content::BrowserContext* context);
+
   static bookmarks::BookmarkModel* GetForProfile(Profile* profile);

[sky, 2016/08/03 19:30:27]

Remove this and convert to using new functions.

[Philipp Keck, 2016/08/04 11:25:48]

Ok. Since that will affect almost 100 places in the codebase and because we have
feature freeze coming up with a couple of other CLs depending on this one, I'd
like to do these changes in a separate CL.

I added a GetForBrowserContextIfExists for consistency, and a TODO(pke) comment
for removing GetForProfile[IfExists]. The new CL will follow very soon, at the
latest right after feature freeze. I hope we can merge this CL as is -- or do
you need me to remove the two new methods for now and reinsert them later?

 
   static bookmarks::BookmarkModel* GetForProfileIfExists(Profile* profile);