 Chromium Code Reviews
 Chromium Code Reviews Issue 2144533002:
  Convert network hints to Mojo  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 2144533002:
  Convert network hints to Mojo  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: components/network_hints/renderer/renderer_dns_prefetch.h | 
| diff --git a/components/network_hints/renderer/renderer_dns_prefetch.h b/components/network_hints/renderer/renderer_dns_prefetch.h | 
| index 9df9e5b9e00bdca1e54cac5b72fb531d2c34d2d1..cd9d81e7277267a39a2a9b4d18918e15ee09f2d0 100644 | 
| --- a/components/network_hints/renderer/renderer_dns_prefetch.h | 
| +++ b/components/network_hints/renderer/renderer_dns_prefetch.h | 
| @@ -28,10 +28,13 @@ | 
| #include "base/macros.h" | 
| #include "base/memory/weak_ptr.h" | 
| +#include "components/network_hints/common/network_hints.mojom.h" | 
| #include "components/network_hints/renderer/dns_prefetch_queue.h" | 
| namespace network_hints { | 
| +class NetworkHints; | 
| + | 
| // An internal interface to the network_hints component for efficiently sending | 
| // DNS prefetch requests to the net stack. | 
| class RendererDnsPrefetch { | 
| @@ -77,6 +80,9 @@ class RendererDnsPrefetch { | 
| // This discards ALL queue entries, and map entries. | 
| void Reset(); | 
| + // Get a connected NetworkHints. | 
| + mojom::NetworkHintsPtr& GetNetworkHints(); | 
| 
Sam McNally
2016/07/20 00:03:01
Why not return a mojom::NetworkHints&?
 
tibell
2016/07/20 00:29:34
Done.
 | 
| + | 
| // We use c_string_queue_ to hold lists of names supplied typically) by the | 
| // renderer. It queues the names, at minimal cost to the renderer's thread, | 
| // and allows this class to process them when time permits (in a later task). | 
| @@ -101,6 +107,9 @@ class RendererDnsPrefetch { | 
| int buffer_full_discard_count_; | 
| int numeric_ip_discard_count_; | 
| + // Don't use directly. Instead call |GetNetworkHints()|. | 
| + mojom::NetworkHintsPtr network_hints_; | 
| + | 
| base::WeakPtrFactory<RendererDnsPrefetch> weak_factory_; | 
| DISALLOW_COPY_AND_ASSIGN(RendererDnsPrefetch); |