Update bookmark suggestions also after undismiss/delete.

components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_NTP_SNIPPETS_BOOKMARKS_BOOKMARK_LAST_VISIT_UTILS_H_
 #define COMPONENTS_NTP_SNIPPETS_BOOKMARKS_BOOKMARK_LAST_VISIT_UTILS_H_
 
 #include <vector>
 
 class GURL;
@@ skipping 13 matching lines @@
 // to now. If there are multiple bookmarks for a given URL, it updates all of
 // them.
 void UpdateBookmarkOnURLVisitedInMainFrame(
     bookmarks::BookmarkModel* bookmark_model,
     const GURL& url);
 
 // Gets the last visit date for a given bookmark |node|. If the bookmark lacks
 // this info, it returns it creation date.
 base::Time GetLastVisitDateForBookmark(const bookmarks::BookmarkNode* node);
 
+// Like GetLastVisitDateForBookmark, but it returns the unix epoch if the
+// bookmark is dismissed from NTP.
+base::Time GetLastVisitDateForNotDismissedBookmark(

[Marc Treib, 2016/08/11 15:24:54]

Hm, not super happy with this name, but I also can't really think of a better
one... is GetLastVisitDateForBookmarkIfNotDismissed better?

[jkrcal, 2016/08/11 15:41:35]

Better, thanks. Done.

+    const bookmarks::BookmarkNode* node);
+
 // Marks all bookmarks with the given URL as dismissed.
 void MarkBookmarksDismissed(bookmarks::BookmarkModel* bookmark_model,
                             const GURL& url);
 
 // Gets the dismissed flag for a given bookmark |node|. Defaults to false.
 bool IsDismissedFromNTPForBookmark(const bookmarks::BookmarkNode* node);
 
 // Removes the dismissed flag from all bookmarks (only for debugging).
 void MarkAllBookmarksUndismissed(bookmarks::BookmarkModel* bookmark_model);
 
 // Returns the list of most recently visited, non-dismissed bookmarks.
 // For each bookmarked URL, it returns the most recently created bookmark.
 // The result is ordered by visit time (the most recent first). Only bookmarks
 // visited after |min_visit_time| are considered, at most |max_count| bookmarks
 // are returned.
 std::vector<const bookmarks::BookmarkNode*> GetRecentlyVisitedBookmarks(
     bookmarks::BookmarkModel* bookmark_model,
     int max_count,
     const base::Time& min_visit_time);
 
 // Returns the list of all dismissed bookmarks. Only used for debugging.
 std::vector<const bookmarks::BookmarkNode*> GetDismissedBookmarksForDebugging(
     bookmarks::BookmarkModel* bookmark_model);
 
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_BOOKMARKS_BOOKMARK_LAST_VISIT_UTILS_H_