Eliminate WindowTreeConnection

components/mus/public/cpp/lib/window.cc

 // Copyright 2014 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 "components/mus/public/cpp/window.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <set>
 #include <string>
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "components/mus/common/transient_window_utils.h"
 #include "components/mus/public/cpp/lib/window_private.h"
-#include "components/mus/public/cpp/lib/window_tree_client_impl.h"
 #include "components/mus/public/cpp/property_type_converters.h"
 #include "components/mus/public/cpp/window_observer.h"
 #include "components/mus/public/cpp/window_property.h"
 #include "components/mus/public/cpp/window_surface.h"
 #include "components/mus/public/cpp/window_tracker.h"
+#include "components/mus/public/cpp/window_tree_client.h"
 #include "components/mus/public/interfaces/window_manager.mojom.h"
 #include "ui/display/display.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace mus {
 
 namespace {
 
 void NotifyWindowTreeChangeAtReceiver(
@@ skipping 107 matching lines @@
   }
 
  private:
   Window* window_;
   const gfx::Rect old_bounds_;
   const gfx::Rect new_bounds_;
 
   DISALLOW_COPY_AND_ASSIGN(ScopedSetBoundsNotifier);
 };
 
-// Some operations are only permitted in the connection that created the window.
-bool OwnsWindow(WindowTreeConnection* connection, Window* window) {
-  return !connection ||
-         static_cast<WindowTreeClientImpl*>(connection)->OwnsWindow(window);
+// Some operations are only permitted in the client that created the window.
+bool OwnsWindow(WindowTreeClient* client, Window* window) {
+  return !client || client->OwnsWindow(window);
 }
 
-bool IsConnectionRoot(Window* window) {
-  return window->connection() &&
-         window->connection()->GetRoots().count(window) > 0;
+bool IsClientRoot(Window* window) {
+  return window->window_tree() &&
+         window->window_tree()->GetRoots().count(window) > 0;
 }
 
 bool OwnsWindowOrIsRoot(Window* window) {
-  return OwnsWindow(window->connection(), window) || IsConnectionRoot(window);
+  return OwnsWindow(window->window_tree(), window) || IsClientRoot(window);
 }
 
 void EmptyEmbedCallback(bool result) {}
 
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // Window, public:
 
 void Window::Destroy() {
   if (!OwnsWindowOrIsRoot(this))
     return;
 
-  if (connection_)
-    tree_client()->DestroyWindow(this);
+  if (client_)
+    window_tree()->DestroyWindow(this);

[sky, 2016/05/27 20:45:55]

window_tree() -> client_

   while (!children_.empty()) {
     Window* child = children_.front();
-    if (!OwnsWindow(connection_, child)) {
+    if (!OwnsWindow(client_, child)) {
       WindowPrivate(child).ClearParent();
       children_.erase(children_.begin());
     } else {
       child->Destroy();
       DCHECK(std::find(children_.begin(), children_.end(), child) ==
              children_.end());
     }
   }
   LocalDestroy();
 }
 
 void Window::SetBounds(const gfx::Rect& bounds) {
   if (!OwnsWindowOrIsRoot(this))
     return;
   if (bounds_ == bounds)
     return;
-  if (connection_)
-    tree_client()->SetBounds(this, bounds_, bounds);
+  if (client_)
+    window_tree()->SetBounds(this, bounds_, bounds);
   LocalSetBounds(bounds_, bounds);
 }
 
 gfx::Rect Window::GetBoundsInRoot() const {
   gfx::Vector2d offset;
   for (const Window* w = parent(); w != nullptr; w = w->parent())
     offset += w->bounds().OffsetFromOrigin();
   return bounds() + offset;
 }
 
 void Window::SetClientArea(
     const gfx::Insets& client_area,
     const std::vector<gfx::Rect>& additional_client_areas) {
   if (!OwnsWindowOrIsRoot(this))
     return;
 
-  if (connection_)
-    tree_client()->SetClientArea(server_id_, client_area,
+  if (client_)
+    window_tree()->SetClientArea(server_id_, client_area,
                                  additional_client_areas);
   LocalSetClientArea(client_area, additional_client_areas);
 }
 
 void Window::SetHitTestMask(const gfx::Rect& mask) {
   if (!OwnsWindowOrIsRoot(this))
     return;
 
   if (hit_test_mask_ && *hit_test_mask_ == mask)
     return;
 
-  if (connection_)
-    tree_client()->SetHitTestMask(server_id_, mask);
+  if (client_)
+    window_tree()->SetHitTestMask(server_id_, mask);
   hit_test_mask_.reset(new gfx::Rect(mask));
 }
 
 void Window::ClearHitTestMask() {
   if (!OwnsWindowOrIsRoot(this))
     return;
 
   if (!hit_test_mask_)
     return;
 
-  if (connection_)
-    tree_client()->ClearHitTestMask(server_id_);
+  if (client_)
+    window_tree()->ClearHitTestMask(server_id_);
   hit_test_mask_.reset();
 }
 
 void Window::SetVisible(bool value) {
   if (visible_ == value)
     return;
 
-  if (connection_)
-    tree_client()->SetVisible(this, value);
+  if (client_)
+    window_tree()->SetVisible(this, value);
   LocalSetVisible(value);
 }
 
 void Window::SetOpacity(float opacity) {
-  if (connection_)
-    tree_client()->SetOpacity(this, opacity);
+  if (client_)
+    window_tree()->SetOpacity(this, opacity);
   LocalSetOpacity(opacity);
 }
 
 void Window::SetPredefinedCursor(mus::mojom::Cursor cursor_id) {
   if (cursor_id_ == cursor_id)
     return;
 
-  if (connection_)
-    tree_client()->SetPredefinedCursor(server_id_, cursor_id);
+  if (client_)
+    window_tree()->SetPredefinedCursor(server_id_, cursor_id);
   LocalSetPredefinedCursor(cursor_id);
 }
 
 bool Window::IsDrawn() const {
   if (!visible_)
     return false;
   return parent_ ? parent_->IsDrawn() : parent_drawn_;
 }
 
 std::unique_ptr<WindowSurface> Window::RequestSurface(mojom::SurfaceType type) {
   std::unique_ptr<WindowSurfaceBinding> surface_binding;
   std::unique_ptr<WindowSurface> surface =
       WindowSurface::Create(&surface_binding);
   AttachSurface(type, std::move(surface_binding));
   return surface;
 }
 
 void Window::AttachSurface(
     mojom::SurfaceType type,
     std::unique_ptr<WindowSurfaceBinding> surface_binding) {
-  tree_client()->AttachSurface(
+  window_tree()->AttachSurface(
       server_id_, type, std::move(surface_binding->surface_request_),
       mojo::MakeProxy(std::move(surface_binding->surface_client_)));
 }
 
 void Window::ClearSharedProperty(const std::string& name) {
   SetSharedPropertyInternal(name, nullptr);
 }
 
 bool Window::HasSharedProperty(const std::string& name) const {
   return properties_.count(name) > 0;
 }
 
 void Window::AddObserver(WindowObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void Window::RemoveObserver(WindowObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
 const Window* Window::GetRoot() const {
   const Window* root = this;
   for (const Window* parent = this; parent; parent = parent->parent())
     root = parent;
   return root;
 }
 
 void Window::AddChild(Window* child) {
-  // TODO(beng): not necessarily valid to all connections, but possibly to the
+  // TODO(beng): not necessarily valid to all clients, but possibly to the
   //             embeddee in an embedder-embeddee relationship.
-  if (connection_)
-    CHECK_EQ(child->connection(), connection_);
+  if (client_)
+    CHECK_EQ(child->window_tree(), client_);
   // Roots can not be added as children of other windows.
-  if (tree_client() && tree_client()->IsRoot(child))
+  if (window_tree() && window_tree()->IsRoot(child))
     return;
   LocalAddChild(child);
-  if (connection_)
-    tree_client()->AddChild(this, child->server_id());
+  if (client_)
+    window_tree()->AddChild(this, child->server_id());
 }
 
 void Window::RemoveChild(Window* child) {
-  // TODO(beng): not necessarily valid to all connections, but possibly to the
+  // TODO(beng): not necessarily valid to all clients, but possibly to the
   //             embeddee in an embedder-embeddee relationship.
-  if (connection_)
-    CHECK_EQ(child->connection(), connection_);
+  if (client_)
+    CHECK_EQ(child->window_tree(), client_);
   LocalRemoveChild(child);
-  if (connection_)
-    tree_client()->RemoveChild(this, child->server_id());
+  if (client_)
+    window_tree()->RemoveChild(this, child->server_id());
 }
 
 void Window::Reorder(Window* relative, mojom::OrderDirection direction) {
   if (!LocalReorder(relative, direction))
     return;
-  if (connection_)
-    tree_client()->Reorder(this, relative->server_id(), direction);
+  if (client_)
+    window_tree()->Reorder(this, relative->server_id(), direction);
 }
 
 void Window::MoveToFront() {
   if (!parent_ || parent_->children_.back() == this)
     return;
   Reorder(parent_->children_.back(), mojom::OrderDirection::ABOVE);
 }
 
 void Window::MoveToBack() {
   if (!parent_ || parent_->children_.front() == this)
     return;
   Reorder(parent_->children_.front(), mojom::OrderDirection::BELOW);
 }
 
 bool Window::Contains(const Window* child) const {
   if (!child)
     return false;
   if (child == this)
     return true;
-  if (connection_)
-    CHECK_EQ(child->connection_, connection_);
+  if (client_)
+    CHECK_EQ(child->client_, client_);
   for (const Window* p = child->parent(); p; p = p->parent()) {
     if (p == this)
       return true;
   }
   return false;
 }
 
 void Window::AddTransientWindow(Window* transient_window) {
   // A system modal window cannot become a transient child.
   DCHECK(!transient_window->is_modal() || transient_window->transient_parent());
 
-  if (connection_)
-    CHECK_EQ(transient_window->connection(), connection_);
+  if (client_)
+    CHECK_EQ(transient_window->window_tree(), client_);
   LocalAddTransientWindow(transient_window);
-  if (connection_)
-    tree_client()->AddTransientWindow(this, transient_window->server_id());
+  if (client_)
+    window_tree()->AddTransientWindow(this, transient_window->server_id());
 }
 
 void Window::RemoveTransientWindow(Window* transient_window) {
-  if (connection_)
-    CHECK_EQ(transient_window->connection(), connection_);
+  if (client_)
+    CHECK_EQ(transient_window->window_tree(), client_);
   LocalRemoveTransientWindow(transient_window);
-  if (connection_)
-    tree_client()->RemoveTransientWindowFromParent(transient_window);
+  if (client_)
+    window_tree()->RemoveTransientWindowFromParent(transient_window);
 }
 
 void Window::SetModal() {
   if (is_modal_)
     return;
 
   LocalSetModal();
-  if (connection_)
-    tree_client()->SetModal(this);
+  if (client_)
+    window_tree()->SetModal(this);
 }
 
 Window* Window::GetChildByLocalId(int id) {
   if (id == local_id_)
     return this;
   // TODO(beng): this could be improved depending on how we decide to own
   // windows.
   for (Window* child : children_) {
     Window* matching_child = child->GetChildByLocalId(id);
     if (matching_child)
       return matching_child;
   }
   return nullptr;
 }
 
 void Window::SetTextInputState(mojo::TextInputStatePtr state) {
-  if (connection_)
-    tree_client()->SetWindowTextInputState(server_id_, std::move(state));
+  if (client_)
+    window_tree()->SetWindowTextInputState(server_id_, std::move(state));
 }
 
 void Window::SetImeVisibility(bool visible, mojo::TextInputStatePtr state) {
   // SetImeVisibility() shouldn't be used if the window is not editable.
   DCHECK(state.is_null() || state->type != mojo::TextInputType::NONE);
-  if (connection_)
-    tree_client()->SetImeVisibility(server_id_, visible, std::move(state));
+  if (client_)
+    window_tree()->SetImeVisibility(server_id_, visible, std::move(state));
 }
 
 bool Window::HasCapture() const {
-  return connection_ && connection_->GetCaptureWindow() == this;
+  return client_ && client_->GetCaptureWindow() == this;
 }
 
 void Window::SetCapture() {
-  if (connection_)
-    tree_client()->SetCapture(this);
+  if (client_)
+    window_tree()->SetCapture(this);
 }
 
 void Window::ReleaseCapture() {
-  if (connection_)
-    tree_client()->ReleaseCapture(this);
+  if (client_)
+    window_tree()->ReleaseCapture(this);
 }
 
 void Window::SetFocus() {
-  if (connection_ && IsDrawn())
-    tree_client()->SetFocus(this);
+  if (client_ && IsDrawn())
+    window_tree()->SetFocus(this);
 }
 
 bool Window::HasFocus() const {
-  return connection_ && connection_->GetFocusedWindow() == this;
+  return client_ && client_->GetFocusedWindow() == this;
 }
 
 void Window::SetCanFocus(bool can_focus) {
-  if (connection_)
-    tree_client()->SetCanFocus(server_id_, can_focus);
+  if (client_)
+    window_tree()->SetCanFocus(server_id_, can_focus);
 }
 
 void Window::Embed(mus::mojom::WindowTreeClientPtr client) {
   Embed(std::move(client), base::Bind(&EmptyEmbedCallback));
 }
 
 void Window::Embed(mus::mojom::WindowTreeClientPtr client,
                    const EmbedCallback& callback) {
   if (PrepareForEmbed())
-    tree_client()->Embed(server_id_, std::move(client), callback);
+    window_tree()->Embed(server_id_, std::move(client), callback);
   else
     callback.Run(false);
 }
 
 void Window::RequestClose() {
-  if (tree_client())
-    tree_client()->RequestClose(this);
+  if (window_tree())
+    window_tree()->RequestClose(this);
 }
 
 std::string Window::GetName() const {
   if (HasSharedProperty(mojom::WindowManager::kName_Property))
     return GetSharedProperty<std::string>(mojom::WindowManager::kName_Property);
 
   return std::string();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Window, protected:
 
 Window::Window() : Window(nullptr, static_cast<Id>(-1)) {}
 
 Window::~Window() {
   FOR_EACH_OBSERVER(WindowObserver, observers_, OnWindowDestroying(this));
-  if (tree_client())
-    tree_client()->OnWindowDestroying(this);
+  if (window_tree())
+    window_tree()->OnWindowDestroying(this);
 
   if (HasFocus()) {
     // The focused window is being removed. When this happens the server
     // advances focus. We don't want to randomly pick a Window to get focus, so
     // we update local state only, and wait for the next focus change from the
     // server.
-    tree_client()->LocalSetFocus(nullptr);
+    window_tree()->LocalSetFocus(nullptr);
   }
 
   // Remove from transient parent.
   if (transient_parent_)
     transient_parent_->LocalRemoveTransientWindow(this);
 
   // Remove transient children.
   while (!transient_children_.empty()) {
     Window* transient_child = transient_children_.front();
     LocalRemoveTransientWindow(transient_child);
@@ skipping 17 matching lines @@
   for (auto& pair : prop_map_) {
     if (pair.second.deallocator)
       (*pair.second.deallocator)(pair.second.value);
   }
   prop_map_.clear();
 
   FOR_EACH_OBSERVER(WindowObserver, observers_, OnWindowDestroyed(this));
 
   // Invoke after observers so that can clean up any internal state observers
   // may have changed.
-  if (tree_client())
-    tree_client()->OnWindowDestroyed(this);
+  if (window_tree())
+    window_tree()->OnWindowDestroyed(this);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Window, private:
 
-Window::Window(WindowTreeConnection* connection, Id id)
-    : connection_(connection),
+Window::Window(WindowTreeClient* client, Id id)
+    : client_(client),
       server_id_(id),
       parent_(nullptr),
       stacking_target_(nullptr),
       transient_parent_(nullptr),
       is_modal_(false),
       // Matches aura, see aura::Window for details.
       observers_(base::ObserverList<WindowObserver>::NOTIFY_EXISTING_ONLY),
       input_event_handler_(nullptr),
       visible_(false),
       opacity_(1.0f),
       display_id_(display::Display::kInvalidDisplayID),
       cursor_id_(mojom::Cursor::CURSOR_NULL),
       parent_drawn_(false) {}
 
-WindowTreeClientImpl* Window::tree_client() {
-  return static_cast<WindowTreeClientImpl*>(connection_);
-}
-
 void Window::SetSharedPropertyInternal(const std::string& name,
                                        const std::vector<uint8_t>* value) {
   if (!OwnsWindowOrIsRoot(this))
     return;
 
-  if (connection_) {
+  if (client_) {
     mojo::Array<uint8_t> transport_value(nullptr);
     if (value) {
       transport_value.resize(value->size());
       if (value->size())
         memcpy(&transport_value.front(), &(value->front()), value->size());
     }
     // TODO: add test coverage of this (450303).
-    tree_client()->SetProperty(this, name, std::move(transport_value));
+    window_tree()->SetProperty(this, name, std::move(transport_value));
   }
   LocalSetSharedProperty(name, value);
 }
 
 int64_t Window::SetLocalPropertyInternal(const void* key,
                                          const char* name,
                                          PropertyDeallocator deallocator,
                                          int64_t value,
                                          int64_t default_value) {
   int64_t old = GetLocalPropertyInternal(key, default_value);
@@ skipping 64 matching lines @@
 
 bool Window::LocalReorder(Window* relative, mojom::OrderDirection direction) {
   OrderChangedNotifier notifier(this, relative, direction);
   return ReorderImpl(this, relative, direction, &notifier);
 }
 
 void Window::LocalSetBounds(const gfx::Rect& old_bounds,
                             const gfx::Rect& new_bounds) {
   // If this client owns the window, then it should be the only one to change
   // the bounds.
-  DCHECK(!OwnsWindow(connection_, this) || old_bounds == bounds_);
+  DCHECK(!OwnsWindow(client_, this) || old_bounds == bounds_);
   ScopedSetBoundsNotifier notifier(this, old_bounds, new_bounds);
   bounds_ = new_bounds;
 }
 
 void Window::LocalSetClientArea(
     const gfx::Insets& new_client_area,
     const std::vector<gfx::Rect>& additional_client_areas) {
   const std::vector<gfx::Rect> old_additional_client_areas =
       additional_client_areas_;
   const gfx::Insets old_client_area = client_area_;
@@ skipping 138 matching lines @@
 void Window::NotifyWindowVisibilityChangedUp(Window* target) {
   // Start with the parent as we already notified |this|
   // in NotifyWindowVisibilityChangedDown.
   for (Window* window = parent(); window; window = window->parent()) {
     bool ret = window->NotifyWindowVisibilityChangedAtReceiver(target);
     DCHECK(ret);
   }
 }
 
 bool Window::PrepareForEmbed() {
-  if (!OwnsWindow(connection_, this))
+  if (!OwnsWindow(client_, this))
     return false;
 
   while (!children_.empty())
     RemoveChild(children_[0]);
   return true;
 }
 
 void Window::RemoveTransientWindowImpl(Window* transient_window) {
   Window::Children::iterator it = std::find(
       transient_children_.begin(), transient_children_.end(), transient_window);
@@ skipping 62 matching lines @@
     notifier->NotifyWindowReordered();
 
   return true;
 }
 
 // static
 Window** Window::GetStackingTarget(Window* window) {
   return &window->stacking_target_;
 }
 }  // namespace mus