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

Unified Diff: components/network_hints/browser/network_hints_impl.h

Issue 2144533002: Convert network hints to Mojo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add untracked dependency on mojom Created 4 years, 5 months 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/network_hints/browser/network_hints_impl.h
diff --git a/components/network_hints/browser/network_hints_message_filter.h b/components/network_hints/browser/network_hints_impl.h
similarity index 52%
rename from components/network_hints/browser/network_hints_message_filter.h
rename to components/network_hints/browser/network_hints_impl.h
index f30216ef426f55d4ca11f75ec199e80533755a93..d6a77ba50372392b139ef771afdbe5b1432058fc 100644
--- a/components/network_hints/browser/network_hints_message_filter.h
+++ b/components/network_hints/browser/network_hints_impl.h
@@ -6,7 +6,8 @@
#define COMPONENTS_NETWORK_HINTS_BROWSER_NETWORK_HINTS_MESSAGE_FILTER_H_
#include "base/macros.h"
-#include "content/public/browser/browser_message_filter.h"
+#include "components/network_hints/public/interfaces/network_hints.mojom.h"
+#include "mojo/public/cpp/bindings/binding_set.h"
namespace net {
class HostResolver;
@@ -18,21 +19,25 @@ struct LookupRequest;
// Simple browser-side handler for DNS prefetch requests.
// Passes prefetch requests to the provided net::HostResolver.
// Each renderer process requires its own filter.
-class NetworkHintsMessageFilter : public content::BrowserMessageFilter {
+class NetworkHintsImpl : public network_hints::mojom::NetworkHints {
public:
- explicit NetworkHintsMessageFilter(net::HostResolver* host_resolver);
+ explicit NetworkHintsImpl(net::HostResolver* host_resolver);
+ ~NetworkHintsImpl() override;
- // content::BrowserMessageFilter implementation:
- bool OnMessageReceived(const IPC::Message& message) override;
+ void Bind(mojo::InterfaceRequest<mojom::NetworkHints> request);
private:
- ~NetworkHintsMessageFilter() override;
-
- void OnDnsPrefetch(const LookupRequest& lookup_request);
+ // network_hints::mojom::NetworkHints implementation:
+ void DNSPrefetch(const LookupRequest& lookup_request) override;
+ void Preconnect(const GURL& url,
+ bool allow_credentials,
+ int32_t count) override {}
net::HostResolver* host_resolver_;
- DISALLOW_COPY_AND_ASSIGN(NetworkHintsMessageFilter);
+ mojo::BindingSet<mojom::NetworkHints> bindings_;
+
+ DISALLOW_COPY_AND_ASSIGN(NetworkHintsImpl);
};
} // namespace network_hints

Powered by Google App Engine
This is Rietveld 408576698