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

Unified Diff: content/renderer/p2p/empty_network_manager.h

Issue 1405963021: Add support for default local address in IpcNetworkManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix a sizeof Created 5 years, 1 month 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: content/renderer/p2p/empty_network_manager.h
diff --git a/content/renderer/p2p/empty_network_manager.h b/content/renderer/p2p/empty_network_manager.h
index 4827d2ba42e79b3451abdb1247034eafcb7d3502..d53cc734cb91f3d2c096e5a093a8adeee4d8f666 100644
--- a/content/renderer/p2p/empty_network_manager.h
+++ b/content/renderer/p2p/empty_network_manager.h
@@ -10,6 +10,10 @@
#include "content/common/content_export.h"
#include "third_party/webrtc/base/network.h"
+namespace rtc {
+class IPAddress;
+} // namespace rtc
+
namespace content {
// A NetworkManager implementation which handles the case where local address
@@ -19,13 +23,16 @@ class EmptyNetworkManager : public rtc::NetworkManagerBase {
public:
// This class is created by WebRTC's signaling thread but used by WebRTC's
// worker thread |task_runner|.
- CONTENT_EXPORT EmptyNetworkManager();
+ CONTENT_EXPORT explicit EmptyNetworkManager(
+ rtc::NetworkManager* network_manager);
CONTENT_EXPORT ~EmptyNetworkManager() override;
// rtc::NetworkManager:
void StartUpdating() override;
void StopUpdating() override;
void GetNetworks(NetworkList* networks) const override;
+ bool GetDefaultLocalAddress(int family,
+ rtc::IPAddress* ipaddress) const override;
private:
void FireEvent();
@@ -36,6 +43,10 @@ class EmptyNetworkManager : public rtc::NetworkManagerBase {
// Track whether StartUpdating() has been called before.
bool updating_started_ = false;
+ // |network_manager_| is just a reference, owned by
+ // PeerConnectionDependencyFactory.
+ rtc::NetworkManager* network_manager_;
+
base::WeakPtrFactory<EmptyNetworkManager> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(EmptyNetworkManager);
« no previous file with comments | « content/renderer/media/webrtc/peer_connection_dependency_factory.cc ('k') | content/renderer/p2p/empty_network_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698