extensions: windows: list all windows from the current profile

chrome/browser/extensions/window_controller.cc

 // 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.
 
 #include "chrome/browser/extensions/window_controller.h"
 
 #include "base/values.h"
 #include "chrome/browser/extensions/api/tabs/tabs_constants.h"
 #include "chrome/browser/extensions/window_controller_list.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/common/extensions/api/windows.h"
 #include "ui/base/base_window.h"
 #include "ui/gfx/geometry/rect.h"
 
 namespace extensions {
 
 ///////////////////////////////////////////////////////////////////////////////
 // WindowController
 
+// A set bitmask values to be used as a WindowTypeFilter. This enum
+// needs to stay in sync with extensions::api::windows::WindowType.
+enum WindowTypeFilterValues {

[not at google - send to devlin, 2015/08/05 17:33:25]

Actually why do you even need this; can you use the api::windows::WINDOW_TYPE...
values directly?

[llandwerlin-old, 2015/08/05 18:12:02]

Done.

+  WINDOW_TYPE_FILTER_NONE = 0,
+  WINDOW_TYPE_FILTER_NORMAL = 1 << 1,
+  WINDOW_TYPE_FILTER_POPUP = 1 << 2,
+  WINDOW_TYPE_FILTER_PANEL = 1 << 3,
+  WINDOW_TYPE_FILTER_APP = 1 << 4,
+  WINDOW_TYPE_FILTER_DEVTOOLS = 1 << 5
+};
+
+// static
+WindowTypeFilter WindowController::GetAllWindowFilter() {
+  // extensions::WindowTypeFilterValues needs to be updated if there
+  // is a change to extensions::api::windows:WindowType.
+  COMPILE_ASSERT(
+      api::windows::WINDOW_TYPE_LAST == api::windows::WINDOW_TYPE_DEVTOOLS,
+      Update_WindowControllerList_to_match_WindowType);

[not at google - send to devlin, 2015/08/05 17:33:25]

I think this COMPILE_ASSERT can be left out. This code is generated.

[llandwerlin-old, 2015/08/05 18:12:02]

I want make sure that if someone adds a new window type they will notice that
this file needs to be updated.
You're right, replacing with == 5.

[not at google - send to devlin, 2015/08/05 18:17:39]

That makes sense.

+  COMPILE_ASSERT(
+      (1 << api::windows::WINDOW_TYPE_DEVTOOLS) == WINDOW_TYPE_FILTER_DEVTOOLS,
+      Update_WindowControllerList_to_match_WindowType);
+  return (WINDOW_TYPE_FILTER_NORMAL | WINDOW_TYPE_FILTER_PANEL |
+          WINDOW_TYPE_FILTER_POPUP | WINDOW_TYPE_FILTER_APP |
+          WINDOW_TYPE_FILTER_DEVTOOLS);
+}
+
+// static
+WindowTypeFilter WindowController::GetDefaultWindowFilter() {
+  return (WINDOW_TYPE_FILTER_NORMAL | WINDOW_TYPE_FILTER_PANEL |
+          WINDOW_TYPE_FILTER_POPUP);
+}
+
 WindowController::WindowController(ui::BaseWindow* window, Profile* profile)
     : window_(window), profile_(profile) {
 }
 
 WindowController::~WindowController() {
 }
 
 Browser* WindowController::GetBrowser() const {
   return NULL;
 }
@@ skipping 27 matching lines @@
   else
     bounds = window()->GetBounds();
   result->SetInteger(keys::kLeftKey, bounds.x());
   result->SetInteger(keys::kTopKey, bounds.y());
   result->SetInteger(keys::kWidthKey, bounds.width());
   result->SetInteger(keys::kHeightKey, bounds.height());
 
   return result;
 }
 
+bool WindowController::MatchesFilter(WindowTypeFilter filter) const {
+  WindowTypeFilter type = 1
+                          << api::windows::ParseWindowType(GetWindowTypeText());
+  return (type & filter) != 0;
+}
+
 }  // namespace extensions