MergeNetworkList can't group address under the same interface

content/renderer/p2p/mock_socket_dispatcher.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.

[Sergey Ulanov, 2014/09/10 18:05:32]

year
remove (c)

[guoweis_webrtc, 2014/09/10 19:56:32]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_RENDERER_P2P_MOCK_SOCKET_DISPATCHER_H_
+#define CONTENT_RENDERER_P2P_MOCK_SOCKET_DISPATCHER_H_
+
+#include "content/renderer/p2p/socket_dispatcher.h"
+#include "testing/gmock/include/gmock/gmock.h"
+
+namespace content {
+
+class MockP2PSocketDispatcher : public P2PSocketDispatcherInterface {

[Sergey Ulanov, 2014/09/10 18:05:32]

Unless this class is shared between multiple tests it's not worth putting it
into a separate file. MOve it to ipc_network_manager_unittest.cc?

[guoweis_webrtc, 2014/09/10 19:56:32]

Done.

+ public:
+  virtual void AddNetworkListObserver(
+    NetworkListObserver* network_list_observer) OVERRIDE {}
+
+  virtual void RemoveNetworkListObserver(
+    NetworkListObserver* network_list_observer) OVERRIDE {}
+
+  virtual ~MockP2PSocketDispatcher() {}
+};
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_P2P_MOCK_SOCKET_DISPATCHER_H_