Window controls

mojo/services/view_manager/connection_manager.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 "mojo/services/view_manager/connection_manager.h"
 
 #include "base/logging.h"
 #include "mojo/public/cpp/application/application_connection.h"
 #include "mojo/public/interfaces/application/service_provider.mojom.h"
 #include "mojo/services/public/cpp/input_events/input_events_type_converters.h"
@@ skipping 245 matching lines @@
     const ServerView* old_parent) {
   if (!display_manager_.in_setup())
     ProcessWillChangeViewHierarchy(view, new_parent, old_parent);
 }
 
 void ConnectionManager::OnViewHierarchyChanged(const ServerView* view,
                                                const ServerView* new_parent,
                                                const ServerView* old_parent) {
   if (!display_manager_.in_setup())
     ProcessViewHierarchyChanged(view, new_parent, old_parent);
+  // TODO(beng): optimize.
+  if (old_parent)
+    display_manager_.SchedulePaint(old_parent, old_parent->bounds());

[sky, 2014/09/11 19:32:22]

I think you want gfx::Rect(old_parent()->bounds().size()) here and
gfx::Rect(new_parent->bounds().size()) below.

+  if (new_parent)
+    display_manager_.SchedulePaint(new_parent, new_parent->bounds());
 }
 
 void ConnectionManager::OnViewBoundsChanged(const ServerView* view,
                                             const gfx::Rect& old_bounds,
                                             const gfx::Rect& new_bounds) {
   ProcessViewBoundsChanged(view, old_bounds, new_bounds);
   if (!view->parent())
     return;
 
   // TODO(sky): optimize this.
   display_manager_.SchedulePaint(view->parent(), old_bounds);
   display_manager_.SchedulePaint(view->parent(), new_bounds);
 }
 
 void ConnectionManager::OnViewBitmapChanged(const ServerView* view) {
   display_manager_.SchedulePaint(view, gfx::Rect(view->bounds().size()));
 }
 
 void ConnectionManager::OnWillChangeViewVisibility(const ServerView* view) {
   for (ConnectionMap::iterator i = connection_map_.begin();
        i != connection_map_.end();
        ++i) {
     i->second->ProcessWillChangeViewVisibility(view, IsChangeSource(i->first));
   }
 }
 
 }  // namespace service
 }  // namespace mojo