extensions: windows: list all windows from the current profile

chrome/browser/extensions/api/tabs/windows_event_router.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_TABS_WINDOWS_EVENT_ROUTER_H_
 #define CHROME_BROWSER_EXTENSIONS_API_TABS_WINDOWS_EVENT_ROUTER_H_
 
+#include <map>
 #include <string>
 
 #include "base/basictypes.h"
+#include "base/containers/scoped_ptr_hash_map.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/extensions/window_controller_list_observer.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
+#include "extensions/browser/app_window/app_window_registry.h"
 
 #if !defined(OS_MACOSX)
 #include "ui/views/focus/widget_focus_manager.h"
 #endif
 
 class Profile;
 
 namespace base {
 class ListValue;
 }
 
 namespace extensions {
 
+class AppWindow;
+class AppWindowController;
+
 // The WindowsEventRouter sends chrome.windows.* events to listeners
 // inside extension process renderers. The router listens to *all* events,
 // but will only route events within a profile to extension processes in the
 // same profile.
-class WindowsEventRouter : public WindowControllerListObserver,
+class WindowsEventRouter : public AppWindowRegistry::Observer,
+                           public WindowControllerListObserver,
 #if !defined(OS_MACOSX)
-                          public views::WidgetFocusChangeListener,
+                           public views::WidgetFocusChangeListener,
 #endif
-                          public content::NotificationObserver {
+                           public content::NotificationObserver {
  public:
   explicit WindowsEventRouter(Profile* profile);
   ~WindowsEventRouter() override;
 
+  void DispatchEvents();

[dcheng, 2015/06/29 18:15:49]

The name of this member makes it sound like this actually dispatches events. I'd
suggest renaming it.

[llandwerlin-old, 2015/06/30 10:20:47]

Done.

+
+  // extensions::AppWindowRegistry::Observer:
+  void OnAppWindowAdded(extensions::AppWindow* app_window) override;
+  void OnAppWindowRemoved(extensions::AppWindow* app_window) override;
+
   // WindowControllerListObserver methods:
   void OnWindowControllerAdded(WindowController* window_controller) override;
   void OnWindowControllerRemoved(WindowController* window) override;
 
 #if !defined(OS_MACOSX)
   void OnNativeFocusChanged(gfx::NativeView focused_now) override;
 #endif
 
   // content::NotificationObserver.
   void Observe(int type,
@@ skipping 15 matching lines @@
 
   // The profile the currently focused window belongs to; either the main or
   // incognito profile or NULL (none of the above). We remember this in order
   // to correctly handle focus changes between non-OTR and OTR windows.
   Profile* focused_profile_;
 
   // The currently focused window. We keep this so as to avoid sending multiple
   // windows.onFocusChanged events with the same windowId.
   int focused_window_id_;
 
+  // Whether to dispatch events.
+  bool dispatch_events_;
+
+  // Map of application windows.
+  base::ScopedPtrHashMap<int, scoped_ptr<AppWindowController>> app_windows_;
+
   DISALLOW_COPY_AND_ASSIGN(WindowsEventRouter);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_TABS_WINDOWS_EVENT_ROUTER_H_