OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2016 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #include "content/browser/loader/resource_hints_controller.h" | |
6 | |
7 #include "base/memory/ptr_util.h" | |
8 #include "content/browser/loader/resource_dispatcher_host_impl.h" | |
9 #include "content/public/browser/browser_context.h" | |
10 #include "content/public/browser/browser_thread.h" | |
11 #include "content/public/browser/render_process_host.h" | |
12 #include "content/public/browser/resource_context.h" | |
13 #include "content/public/browser/resource_dispatcher_host_delegate.h" | |
14 #include "content/public/browser/resource_hints.h" | |
15 #include "net/base/address_list.h" | |
16 #include "net/dns/host_resolver.h" | |
17 #include "net/dns/single_request_host_resolver.h" | |
18 #include "net/url_request/url_request_context.h" | |
19 | |
20 namespace content { | |
21 | |
22 // static | |
23 void ResourceHintsController::RegisterConnection( | |
24 RenderProcessHost* render_process_host, | |
25 mojo::InterfaceRequest<blink::mojom::ResourceHintsDispatcherHost> request) { | |
yzshen1
2016/06/08 17:40:53
nit: in case you don't know, you can use a shorter
Charlie Harrison
2016/06/10 14:20:00
Thank you :) done.
| |
26 DCHECK_CURRENTLY_ON(BrowserThread::UI); | |
27 BrowserThread::PostTask( | |
28 BrowserThread::IO, FROM_HERE, | |
29 base::Bind(&ResourceHintsController::RegisterConnectionOnIOThread, | |
30 render_process_host->GetBrowserContext()->GetResourceContext(), | |
yzshen1
2016/06/08 17:40:53
Is it guaranteed that resource context outlives th
Charlie Harrison
2016/06/10 14:20:00
I'm unsure. What are the semantics of the channel'
yzshen1
2016/06/10 15:45:43
If the renderer doesn't disconnect earlier, the st
| |
31 base::Passed(&request))); | |
32 } | |
33 | |
34 // static | |
35 void ResourceHintsController::RegisterConnectionOnIOThread( | |
36 ResourceContext* resource_context, | |
37 mojo::InterfaceRequest<blink::mojom::ResourceHintsDispatcherHost> request) { | |
38 // Will be deleted when the connection goes away. | |
39 new ResourceHintsController(std::move(request), resource_context); | |
40 } | |
41 | |
42 ResourceHintsController::ResourceHintsController( | |
43 mojo::InterfaceRequest<blink::mojom::ResourceHintsDispatcherHost> request, | |
44 ResourceContext* resource_context) | |
45 : resource_context_(resource_context), binding_(this, std::move(request)) { | |
46 binding_.set_connection_error_handler( | |
47 base::Bind(&ResourceHintsController::HostHadConnectionError, | |
48 base::Unretained(this))); | |
49 } | |
50 | |
51 ResourceHintsController::~ResourceHintsController() {} | |
52 | |
53 void ResourceHintsController::HostHadConnectionError() { | |
54 delete this; | |
yzshen1
2016/06/08 17:40:53
Please take a look at strong_binding.h.
It does t
kinuko
2016/06/09 07:09:19
Yeah we shouldn't need this.
Charlie Harrison
2016/06/10 14:20:00
Done. Thanks for the pointer.
| |
55 } | |
56 | |
57 void ResourceHintsController::DispatchPreresolve(const GURL& url) { | |
58 PreresolveUrl(resource_context_, url, net::CompletionCallback()); | |
59 } | |
60 | |
61 void ResourceHintsController::DispatchPreconnect(const GURL& url, | |
62 bool credentials_flag, | |
63 uint8_t num_connections) { | |
64 // TODO(csharrison): Remove HttpRequestInfo::RequestMotivation and add a | |
65 // better enum at the //content layer. | |
66 PreconnectUrl(resource_context_, url, GURL(), num_connections, | |
67 credentials_flag, net::HttpRequestInfo::EARLY_LOAD_MOTIVATED); | |
68 } | |
69 | |
70 } // namespace content | |
OLD | NEW |