Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(305)

Unified Diff: components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h

Issue 2518033002: [Bookmark suggestions] Clean-up in the api: switch const* to const& (Closed)
Patch Set: Rebase Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h
diff --git a/components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h b/components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h
index 7b59832397ff0a1515aa7d5bd5871779a7dea01f..3f354cf4ee83f668a09d4fa2df274130a876bbe2 100644
--- a/components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h
+++ b/components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h
@@ -38,7 +38,7 @@ void UpdateBookmarkOnURLVisitedInMainFrame(
// present and |creation_date_fallback| is true, the date when the bookmarks is
// created is used as the alst visit date (most likely, it is the date when the
// bookmark was synced to this device).
-bool GetLastVisitDateForNTPBookmark(const bookmarks::BookmarkNode* node,
+bool GetLastVisitDateForNTPBookmark(const bookmarks::BookmarkNode& node,
bool creation_date_fallback,
bool consider_visits_from_desktop,
base::Time* out);
@@ -48,7 +48,7 @@ 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);
+bool IsDismissedFromNTPForBookmark(const bookmarks::BookmarkNode& node);
// Removes the dismissed flag from all bookmarks (only for debugging).
void MarkAllBookmarksUndismissed(bookmarks::BookmarkModel* bookmark_model);

Powered by Google App Engine
This is Rietveld 408576698