Add an OriginFilterBuilder class for [white|black]listing origins.

content/browser/download/download_manager_impl.h

Index: content/browser/download/download_manager_impl.h
diff --git a/content/browser/download/download_manager_impl.h b/content/browser/download/download_manager_impl.h
index ebac07ab08179a900b59b3ceb6d69402f9b8567d..3ad9f773eafc3081a3162e9721b6edf34178c337 100644
--- a/content/browser/download/download_manager_impl.h
+++ b/content/browser/download/download_manager_impl.h
@@ -74,11 +74,10 @@ class CONTENT_EXPORT DownloadManagerImpl : public DownloadManager,
       scoped_ptr<ByteStreamReader> stream,
       const DownloadUrlParameters::OnStartedCallback& on_started) override;
 
-  int RemoveDownloadsByOriginAndTime(const url::Origin& origin,
-                                     base::Time remove_begin,
-                                     base::Time remove_end) override;
-  int RemoveDownloadsBetween(base::Time remove_begin,
-                             base::Time remove_end) override;
+  int RemoveDownloadsByOriginAndTime(

[Timo Reimann, 2016/02/05 19:01:24]

I think we should consider renaming this method and related test names now that
we are passing a generic URL filter as we're not limited to just origins
anymore.

Not sure what would be a good name though. RemoveDownloadsByPredicateAndTime is
probably correct but not perfectly concise.

[msramek, 2016/02/10 14:30:37]

Done. I replaced Origin with URL, as the filter technically doesn't have to be
scoped to origins. I think we don't need to spell out that the URLs are passed
in in a predicate, as that's visible from the method parameters.

+      const base::Callback<bool(const GURL&)>& url_filter,
+      base::Time remove_begin,
+      base::Time remove_end) override;
   int RemoveDownloads(base::Time remove_begin) override;
   int RemoveAllDownloads() override;
   void DownloadUrl(scoped_ptr<DownloadUrlParameters> params) override;