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

Side by Side Diff: content/browser/renderer_host/pepper/pepper_network_monitor_host.cc

Issue 216473002: Replace DCHECK(BrowserThread::CurrentlyOn) with DCHECK_CURRENTLY_ON in content/browser/renderer_hos… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to r260263 Created 6 years, 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/pepper/pepper_network_monitor_host.h" 5 #include "content/browser/renderer_host/pepper/pepper_network_monitor_host.h"
6 6
7 #include "base/task_runner_util.h" 7 #include "base/task_runner_util.h"
8 #include "base/threading/sequenced_worker_pool.h" 8 #include "base/threading/sequenced_worker_pool.h"
9 #include "content/browser/renderer_host/pepper/browser_ppapi_host_impl.h" 9 #include "content/browser/renderer_host/pepper/browser_ppapi_host_impl.h"
10 #include "content/browser/renderer_host/pepper/pepper_socket_utils.h" 10 #include "content/browser/renderer_host/pepper/pepper_socket_utils.h"
11 #include "content/public/browser/browser_thread.h" 11 #include "content/public/browser/browser_thread.h"
12 #include "content/public/common/socket_permission_request.h" 12 #include "content/public/common/socket_permission_request.h"
13 #include "ppapi/proxy/ppapi_messages.h" 13 #include "ppapi/proxy/ppapi_messages.h"
14 #include "ppapi/shared_impl/private/net_address_private_impl.h" 14 #include "ppapi/shared_impl/private/net_address_private_impl.h"
15 15
16 16
17 namespace content { 17 namespace content {
18 18
19 namespace { 19 namespace {
20 20
21 bool CanUseNetworkMonitor(bool external_plugin, 21 bool CanUseNetworkMonitor(bool external_plugin,
22 int render_process_id, 22 int render_process_id,
23 int render_frame_id) { 23 int render_frame_id) {
24 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 24 DCHECK_CURRENTLY_ON(BrowserThread::UI);
25 25
26 SocketPermissionRequest request = SocketPermissionRequest( 26 SocketPermissionRequest request = SocketPermissionRequest(
27 SocketPermissionRequest::NETWORK_STATE, std::string(), 0); 27 SocketPermissionRequest::NETWORK_STATE, std::string(), 0);
28 return pepper_socket_utils::CanUseSocketAPIs( 28 return pepper_socket_utils::CanUseSocketAPIs(
29 external_plugin, false /* private_api */, &request, render_process_id, 29 external_plugin, false /* private_api */, &request, render_process_id,
30 render_frame_id); 30 render_frame_id);
31 } 31 }
32 32
33 scoped_ptr<net::NetworkInterfaceList> GetNetworkList() { 33 scoped_ptr<net::NetworkInterfaceList> GetNetworkList() {
34 scoped_ptr<net::NetworkInterfaceList> list(new net::NetworkInterfaceList()); 34 scoped_ptr<net::NetworkInterfaceList> list(new net::NetworkInterfaceList());
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 host()->SendUnsolicitedReply(pp_resource(), 72 host()->SendUnsolicitedReply(pp_resource(),
73 PpapiPluginMsg_NetworkMonitor_Forbidden()); 73 PpapiPluginMsg_NetworkMonitor_Forbidden());
74 return; 74 return;
75 } 75 }
76 76
77 net::NetworkChangeNotifier::AddIPAddressObserver(this); 77 net::NetworkChangeNotifier::AddIPAddressObserver(this);
78 GetAndSendNetworkList(); 78 GetAndSendNetworkList();
79 } 79 }
80 80
81 void PepperNetworkMonitorHost::GetAndSendNetworkList() { 81 void PepperNetworkMonitorHost::GetAndSendNetworkList() {
82 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 82 DCHECK_CURRENTLY_ON(BrowserThread::IO);
83 83
84 // Call GetNetworkList() on a thread that allows blocking IO. 84 // Call GetNetworkList() on a thread that allows blocking IO.
85 base::PostTaskAndReplyWithResult( 85 base::PostTaskAndReplyWithResult(
86 BrowserThread::GetBlockingPool(), FROM_HERE, 86 BrowserThread::GetBlockingPool(), FROM_HERE,
87 base::Bind(&GetNetworkList), 87 base::Bind(&GetNetworkList),
88 base::Bind(&PepperNetworkMonitorHost::SendNetworkList, 88 base::Bind(&PepperNetworkMonitorHost::SendNetworkList,
89 weak_factory_.GetWeakPtr())); 89 weak_factory_.GetWeakPtr()));
90 } 90 }
91 91
92 void PepperNetworkMonitorHost::SendNetworkList( 92 void PepperNetworkMonitorHost::SendNetworkList(
93 scoped_ptr<net::NetworkInterfaceList> list) { 93 scoped_ptr<net::NetworkInterfaceList> list) {
94 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 94 DCHECK_CURRENTLY_ON(BrowserThread::IO);
95 95
96 scoped_ptr<ppapi::proxy::SerializedNetworkList> list_copy( 96 scoped_ptr<ppapi::proxy::SerializedNetworkList> list_copy(
97 new ppapi::proxy::SerializedNetworkList(list->size())); 97 new ppapi::proxy::SerializedNetworkList(list->size()));
98 for (size_t i = 0; i < list->size(); ++i) { 98 for (size_t i = 0; i < list->size(); ++i) {
99 const net::NetworkInterface& network = list->at(i); 99 const net::NetworkInterface& network = list->at(i);
100 ppapi::proxy::SerializedNetworkInfo& network_copy = list_copy->at(i); 100 ppapi::proxy::SerializedNetworkInfo& network_copy = list_copy->at(i);
101 network_copy.name = network.name; 101 network_copy.name = network.name;
102 102
103 network_copy.addresses.resize( 103 network_copy.addresses.resize(
104 1, ppapi::NetAddressPrivateImpl::kInvalidNetAddress); 104 1, ppapi::NetAddressPrivateImpl::kInvalidNetAddress);
105 bool result = ppapi::NetAddressPrivateImpl::IPEndPointToNetAddress( 105 bool result = ppapi::NetAddressPrivateImpl::IPEndPointToNetAddress(
106 network.address, 0, &(network_copy.addresses[0])); 106 network.address, 0, &(network_copy.addresses[0]));
107 DCHECK(result); 107 DCHECK(result);
108 108
109 // TODO(sergeyu): Currently net::NetworkInterfaceList provides 109 // TODO(sergeyu): Currently net::NetworkInterfaceList provides
110 // only name and one IP address. Add all other fields and copy 110 // only name and one IP address. Add all other fields and copy
111 // them here. 111 // them here.
112 network_copy.type = PP_NETWORKLIST_TYPE_UNKNOWN; 112 network_copy.type = PP_NETWORKLIST_TYPE_UNKNOWN;
113 network_copy.state = PP_NETWORKLIST_STATE_UP; 113 network_copy.state = PP_NETWORKLIST_STATE_UP;
114 network_copy.display_name = network.name; 114 network_copy.display_name = network.name;
115 network_copy.mtu = 0; 115 network_copy.mtu = 0;
116 } 116 }
117 host()->SendUnsolicitedReply( 117 host()->SendUnsolicitedReply(
118 pp_resource(), PpapiPluginMsg_NetworkMonitor_NetworkList(*list_copy)); 118 pp_resource(), PpapiPluginMsg_NetworkMonitor_NetworkList(*list_copy));
119 } 119 }
120 120
121 } // namespace content 121 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698