Chromium Code Reviews| Index: webrtc/base/network.cc |
| diff --git a/webrtc/base/network.cc b/webrtc/base/network.cc |
| index 488c475137a2ee8d7ddbea1fa2ace59175cc3f36..6d0f2de9b882c190cc30c1dfebf617774883ece5 100644 |
| --- a/webrtc/base/network.cc |
| +++ b/webrtc/base/network.cc |
| @@ -731,7 +731,7 @@ void BasicNetworkManager::StartNetworkMonitor() { |
| if (factory == nullptr) { |
| return; |
| } |
| - network_monitor_.reset(factory->CreateNetworkMonitor()); |
| + network_monitor_ = factory->GetOrCreateNetworkMonitor(); |
|
pthatcher1
2016/01/14 20:07:24
Why is it necessary for the factory to own the net
honghaiz3
2016/01/15 01:00:38
It is a good point. I revised it to own the monito
|
| if (!network_monitor_) { |
| return; |
| } |
| @@ -745,7 +745,6 @@ void BasicNetworkManager::StopNetworkMonitor() { |
| return; |
| } |
| network_monitor_->Stop(); |
| - network_monitor_.reset(); |
| } |
| void BasicNetworkManager::OnMessage(Message* msg) { |