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

chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

Index: chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc
diff --git a/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc b/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc
index 96cd31d3a9481f446ae30fbc20edb82f1375fc92..03fe93f1a84ba267ad80ea264c82979171e07b3d 100644
--- a/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc
+++ b/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc
@@ -297,8 +297,10 @@ void ChromeBrowsingDataRemoverDelegate::RemoveEmbedderData(
     // time range and the filter.
     bookmarks::BookmarkModel* bookmark_model =
         BookmarkModelFactory::GetForBrowserContext(profile_);
-    if (bookmark_model)
-        ntp_snippets::RemoveAllLastVisitDates(bookmark_model);
+    if (bookmark_model) {
+      ntp_snippets::RemoveLastVisitedDatesBetween(delete_begin_, delete_end_,

[msramek, 2017/01/04 11:47:12]

Could you update BrowsingDataRemoverTest? You can use
AddURLWithCreationTimeAndMetaInfo to add entries with timestamps in the past.

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

Done. Deeper test coverage is in bookmark_last_visit_utils_unittest.

+                                                  filter, bookmark_model);
+    }
 
 #if BUILDFLAG(ENABLE_EXTENSIONS)
     // The extension activity log contains details of which websites extensions