TopSites: filter out non-WebSafe URLs

components/history/core/browser/visitsegment_database.h

Index: components/history/core/browser/visitsegment_database.h
diff --git a/components/history/core/browser/visitsegment_database.h b/components/history/core/browser/visitsegment_database.h
index c5ca21e6ab31c94e0ced37bb6d76689417c6f580..62d67b5321f3e717b8e865047a02b4166e4b384e 100644
--- a/components/history/core/browser/visitsegment_database.h
+++ b/components/history/core/browser/visitsegment_database.h
@@ -7,6 +7,7 @@
 
 #include <memory>
 
+#include "base/callback.h"

[sdefresne, 2016/05/03 11:18:50]

nit: I think you can include "base/callback_forward.h" here and
"base/callback.h" in the implementation.

[Marc Treib, 2016/05/03 11:51:16]

Done.

 #include "base/macros.h"
 #include "components/history/core/browser/history_types.h"
 
@@ -52,11 +53,13 @@ class VisitSegmentDatabase {
   bool IncreaseSegmentVisitCount(SegmentID segment_id, base::Time ts,
                                  int amount);
 
-  // Computes the segment usage since |from_time|.
+  // Computes the segment usage since |from_time|. If |url_filter| is non-null,
+  // then only URLs for which it returns true will be included.
   // Returns the highest-scored segments up to |max_result_count|.
   std::vector<std::unique_ptr<PageUsageData>> QuerySegmentUsage(
       base::Time from_time,
-      int max_result_count);
+      int max_result_count,
+      const base::Callback<bool(const GURL&)>& url_filter);
 
   // Delete all the segment usage data which is older than the provided time
   // stamp.