chromeos: changes DisconnectedAppHandler to be associated with a single window

ash/mus/disconnected_app_handler.cc

 // Copyright 2016 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 "ash/mus/disconnected_app_handler.h"
 
-#include "ash/aura/wm_window_aura.h"
-#include "ash/public/cpp/shell_window_ids.h"
 #include "ui/aura/window.h"
+#include "ui/aura/window_property.h"
+
+DECLARE_WINDOW_PROPERTY_TYPE(ash::mus::DisconnectedAppHandler*);
 
 namespace ash {
 namespace mus {
 namespace {
 
-bool IsContainer(aura::Window* window) {
-  return WmWindowAura::Get(window)->GetShellWindowId() !=
-         kShellWindowId_Invalid;
-}
+DEFINE_OWNED_WINDOW_PROPERTY_KEY(DisconnectedAppHandler,

[sadrul, 2017/01/12 23:26:54]

This ensures the window handler is destroyed with the window itself, right?

[sky, 2017/01/13 03:01:56]

You got it.

+                                 kDisconnectedAppHandlerKey,
+                                 nullptr);
 
 }  // namespace
 
-DisconnectedAppHandler::DisconnectedAppHandler(aura::Window* root_window) {
-  ash::WmWindowAura* root = ash::WmWindowAura::Get(root_window);
-  for (int shell_window_id = kShellWindowId_Min;
-       shell_window_id < kShellWindowId_Max; ++shell_window_id) {
-    // kShellWindowId_VirtualKeyboardContainer is lazily created.
-    // TODO(sky): http://crbug.com/616909 .
-    // kShellWindowId_PhantomWindow is not a container, but a window.
-    if (shell_window_id == kShellWindowId_VirtualKeyboardContainer ||
-        shell_window_id == kShellWindowId_PhantomWindow)
-      continue;
-
-// kShellWindowId_MouseCursorContainer is chromeos specific.
-#if !defined(OS_CHROMEOS)
-    if (shell_window_id == kShellWindowId_MouseCursorContainer)
-      continue;
-#endif
-
-    ash::WmWindowAura* container = static_cast<ash::WmWindowAura*>(
-        root->GetChildByShellWindowId(shell_window_id));
-    Add(container->aura_window());
-
-    // Add any pre-existing windows in the container to
-    // |disconnected_app_handler_|.
-    for (aura::Window* child : container->aura_window()->children()) {
-      if (!IsContainer(child))
-        Add(child);
-    }
-  }
+// static
+void DisconnectedAppHandler::Create(aura::Window* window) {
+  window->SetProperty(kDisconnectedAppHandlerKey,
+                      new DisconnectedAppHandler(window));
 }
 
-DisconnectedAppHandler::~DisconnectedAppHandler() {}
+DisconnectedAppHandler::DisconnectedAppHandler(aura::Window* window)
+    : window_(window) {
+  window->AddObserver(this);
+}
+
+DisconnectedAppHandler::~DisconnectedAppHandler() {
+  window_->RemoveObserver(this);
+}
 
 void DisconnectedAppHandler::OnEmbeddedAppDisconnected(aura::Window* window) {
-  if (!IsContainer(window))
-    delete window;
-}
-
-void DisconnectedAppHandler::OnWindowHierarchyChanging(
-    const HierarchyChangeParams& params) {
-  if (params.old_parent == params.receiver && IsContainer(params.old_parent))
-    Remove(params.target);
-
-  if (params.new_parent == params.receiver && IsContainer(params.new_parent))
-    Add(params.target);
-
-  aura::WindowTracker::OnWindowHierarchyChanging(params);
+  delete window_;
 }
 
 }  // namespace mus
 }  // namespace ash