Base infrastructure for Networking Private API on Windows and Mac.

chrome/browser/extensions/api/networking_private/networking_private_event_router_nonchromeos.cc

Index: chrome/browser/extensions/api/networking_private/networking_private_event_router_nonchromeos.cc
diff --git a/chrome/browser/extensions/api/networking_private/networking_private_event_router_nonchromeos.cc b/chrome/browser/extensions/api/networking_private/networking_private_event_router_nonchromeos.cc
new file mode 100644
index 0000000000000000000000000000000000000000..364ea52b4b666334281fa20e32d683d3405688c3
--- /dev/null
+++ b/chrome/browser/extensions/api/networking_private/networking_private_event_router_nonchromeos.cc
@@ -0,0 +1,111 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/extensions/api/networking_private/networking_private_event_router.h"
+
+#include "chrome/browser/extensions/api/networking_private/networking_private_api.h"
+#include "chrome/browser/extensions/api/networking_private/networking_private_process_client.h"
+#include "chrome/browser/extensions/extension_system.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/extensions/api/networking_private.h"
+
+namespace extensions {
+
+// This is an event router that will observe listeners to |NetworksChanged| and
+// |NetworkListChanged| events. Shutdown NetworkingPrivateProcessClient if there
+// are no listeners..
+class NetworkingPrivateEventRouterImpl : public NetworkingPrivateEventRouter {
+ public:
+  explicit NetworkingPrivateEventRouterImpl(Profile* profile);
+  virtual ~NetworkingPrivateEventRouterImpl();
+
+ protected:
+  // BrowserContextKeyedService overrides:
+  virtual void Shutdown() OVERRIDE;
+
+  // EventRouter::Observer overrides:
+  virtual void OnListenerAdded(const EventListenerInfo& details) OVERRIDE;
+  virtual void OnListenerRemoved(const EventListenerInfo& details) OVERRIDE;
+
+ private:
+  // Decide if we should listen for network changes or not. If there are any
+  // JavaScript listeners registered for the onNetworkChanged event, then we
+  // want to register for change notification from the network state handler.
+  // Otherwise, we want to unregister and not be listening to network changes.
+  void StartOrStopListeningForNetworkChanges();
+
+  Profile* profile_;
+  bool listening_;
+
+  DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateEventRouterImpl);
+};
+
+NetworkingPrivateEventRouterImpl::NetworkingPrivateEventRouterImpl(
+    Profile* profile)
+    : profile_(profile), listening_(false) {
+  // Register with the event router so we know when renderers are listening to
+  // our events. We first check and see if there *is* an event router, because
+  // some unit tests try to create all profile services, but don't initialize
+  // the event router first.
+  EventRouter* event_router = ExtensionSystem::Get(profile_)->event_router();
+  if (event_router) {
+    event_router->RegisterObserver(
+        this, api::networking_private::OnNetworksChanged::kEventName);
+    event_router->RegisterObserver(
+        this, api::networking_private::OnNetworkListChanged::kEventName);
+    StartOrStopListeningForNetworkChanges();
+  }
+}
+
+NetworkingPrivateEventRouterImpl::~NetworkingPrivateEventRouterImpl() {
+  DCHECK(!listening_);
+}
+
+void NetworkingPrivateEventRouterImpl::Shutdown() {
+  // Unregister with the event router. We first check and see if there *is* an
+  // event router, because some unit tests try to shutdown all profile services,
+  // but didn't initialize the event router first.
+  EventRouter* event_router = ExtensionSystem::Get(profile_)->event_router();
+  if (event_router)
+    event_router->UnregisterObserver(this);
+
+  listening_ = false;
+}
+
+void NetworkingPrivateEventRouterImpl::OnListenerAdded(
+    const EventListenerInfo& details) {
+  // Start listening to events from the network state handler.
+  StartOrStopListeningForNetworkChanges();
+}
+
+void NetworkingPrivateEventRouterImpl::OnListenerRemoved(
+    const EventListenerInfo& details) {
+  // Stop listening to events from the network state handler if there are no
+  // more listeners.
+  StartOrStopListeningForNetworkChanges();
+}
+
+void NetworkingPrivateEventRouterImpl::StartOrStopListeningForNetworkChanges() {
+  EventRouter* event_router = ExtensionSystem::Get(profile_)->event_router();
+  bool should_listen =
+      event_router->HasEventListener(
+          api::networking_private::OnNetworksChanged::kEventName) ||
+      event_router->HasEventListener(
+          api::networking_private::OnNetworkListChanged::kEventName);
+
+  if (!should_listen && listening_) {
+    scoped_refptr<NetworkingPrivateProcessClient> process_client(
+        NetworkingPrivateProcessClient::GetProcessClientForProfile(profile_));
+    process_client->ShutdownProcessClient();

[cbentzel, 2013/10/16 14:17:40]

I think we should only shutdown if there are no event handlers and there are no
outstanding API calls - this only catches the case of transitioning to no event
handlers.

[mef, 2013/10/17 02:33:25]

Done.

+  }
+
+  listening_ = should_listen;
+}
+
+NetworkingPrivateEventRouter* NetworkingPrivateEventRouter::Create(
+    Profile* profile) {
+  return new NetworkingPrivateEventRouterImpl(profile);
+}
+
+}  // namespace extensions