Respect time range in browsing data removal for last-visited data.

components/ntp_snippets/bookmarks/bookmark_last_visit_utils.cc

 // 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.
 
 #include "components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h"
 
 #include <algorithm>
 #include <numeric>
 #include <set>
 #include <string>
@@ skipping 24 matching lines @@
 
 const char kBookmarkLastVisitDateOnMobileKey[] = "last_visited";
 const char kBookmarkLastVisitDateOnDesktopKey[] = "last_visited_desktop";
 const char kBookmarkDismissedFromNTP[] = "dismissed_from_ntp";
 
 std::string FormatLastVisitDate(const base::Time& date) {
   return base::Int64ToString(date.ToInternalValue());
 }
 
 bool ExtractLastVisitDate(const BookmarkNode& node,
-                 const std::string& meta_info_key,
-                 base::Time* out) {
+                          const std::string& meta_info_key,
+                          base::Time* out) {
   std::string last_visit_date_string;
   if (!node.GetMetaInfo(meta_info_key, &last_visit_date_string)) {
     return false;
   }
 
   int64_t date = 0;
   if (!base::StringToInt64(last_visit_date_string, &date) || date < 0) {
     return false;
   }
 
@@ skipping 205 matching lines @@
 
   // Insert into |result|.
   std::vector<const BookmarkNode*> result;
   for (const BookmarkModel::URLAndTitle& bookmark : bookmarks) {
     result.push_back(
         bookmark_model->GetMostRecentlyAddedUserNodeForURL(bookmark.url));
   }
   return result;
 }
 
-void RemoveAllLastVisitDates(bookmarks::BookmarkModel* bookmark_model) {
+namespace {
+
+void ClearNTPMetadata(bookmarks::BookmarkModel* model,
+                      const BookmarkNode* node) {
+  model->DeleteNodeMetaInfo(node, kBookmarkLastVisitDateOnMobileKey);
+  model->DeleteNodeMetaInfo(node, kBookmarkLastVisitDateOnDesktopKey);
+  model->DeleteNodeMetaInfo(node, kBookmarkDismissedFromNTP);
+}
+
+}  // namespace
+
+void RemoveLastVisitedDatesBetween(const base::Time& begin,
+                                   const base::Time& end,
+                                   base::Callback<bool(const GURL& url)> filter,

[jkrcal, 2017/01/04 12:14:22]

Why do you accept |filter| when you not use it? Why not call the filter with
|url_and_title.url|?

[tschumann, 2017/01/04 13:32:32]

Good point! Will add this functionality. (Sending my responses now to get
Martin's input on the other comment).

[tschumann, 2017/01/04 15:08:44]

Done.
+Martin: Can you double-check I got the filter semantics right? (couldn't find a
clear documentation if filtering out means out of the history or out of the
removal process).
Somebody remind me that I shouldn't go grocery shopping today [milk / bread
rolls...] xD

[msramek, 2017/01/04 15:16:26]

Yeah, sorry for that. I guess we really should have used the word 'predicate'
primarily... url should be deleted iff filter(url), so you got the semantics
right.

+                                   bookmarks::BookmarkModel* bookmark_model) {
   // Get all the bookmark URLs.
   std::vector<BookmarkModel::URLAndTitle> bookmark_urls;
   bookmark_model->GetBookmarks(&bookmark_urls);
 
   for (const BookmarkModel::URLAndTitle& url_and_title : bookmark_urls) {
     // Get all bookmarks for the given URL.
     std::vector<const BookmarkNode*> bookmarks_for_url;
     bookmark_model->GetNodesByURL(url_and_title.url, &bookmarks_for_url);
 
     for (const BookmarkNode* bookmark : bookmarks_for_url) {
-      bookmark_model->DeleteNodeMetaInfo(bookmark,
-                                         kBookmarkLastVisitDateOnMobileKey);
-      bookmark_model->DeleteNodeMetaInfo(bookmark,
-                                         kBookmarkLastVisitDateOnDesktopKey);
+      base::Time last_visit_time;
+      if (ExtractLastVisitDate(*bookmark, kBookmarkLastVisitDateOnMobileKey,
+                               &last_visit_time) &&
+          begin <= last_visit_time && last_visit_time <= end) {
+        ClearNTPMetadata(bookmark_model, bookmark);
+      }
+      if (ExtractLastVisitDate(*bookmark, kBookmarkLastVisitDateOnDesktopKey,
+                               &last_visit_time) &&
+          begin <= last_visit_time && last_visit_time <= end) {
+        ClearNTPMetadata(bookmark_model, bookmark);
+      }
     }
   }
 }
 
 }  // namespace ntp_snippets