Snap RWHVA and resize the legacy window on Windows whenever the ancestor window's bounds change.

content/browser/renderer_host/render_widget_host_view_aura.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 CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_AURA_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_AURA_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 380 matching lines @@
                            DestroyedAfterCopyRequest);
   FRIEND_TEST_ALL_PREFIXES(RenderWidgetHostViewAuraTest,
                            VisibleViewportTest);
   FRIEND_TEST_ALL_PREFIXES(RenderWidgetHostViewAuraTest,
                            OverscrollResetsOnBlur);
   FRIEND_TEST_ALL_PREFIXES(RenderWidgetHostViewAuraTest,
                            FinishCompositionByMouse);
   FRIEND_TEST_ALL_PREFIXES(WebContentsViewAuraTest,
                            WebContentsViewReparent);
 
-  class WindowObserver;
-  friend class WindowObserver;
-
   void UpdateCursorIfOverSelf();
 
   // Tracks whether SnapToPhysicalPixelBoundary() has been called.
   bool has_snapped_to_boundary() { return has_snapped_to_boundary_; }
   void ResetHasSnappedToBoundary() { has_snapped_to_boundary_ = false; }
 
   // Set the bounds of the window and handle size changes.  Assumes the caller
   // has already adjusted the origin of |rect| to conform to whatever coordinate
   // space is required by the aura::Window.
   void InternalSetBounds(const gfx::Rect& rect);
@@ skipping 79 matching lines @@
   // Helper function to set keyboard focus to the main window.
   void SetKeyboardFocus();
 
   RenderFrameHostImpl* GetFocusedFrame();
 
   // Returns true if the |event| passed in can be forwarded to the renderer.
   bool CanRendererHandleEvent(const ui::MouseEvent* event,
                               bool mouse_locked,
                               bool selection_popup);
 
+  // Called when the parent window bounds change.
+  void HandleParentBoundsChanged();
+
   // The model object.
   RenderWidgetHostImpl* host_;
 
   aura::Window* window_;
 
   scoped_ptr<DelegatedFrameHost> delegated_frame_host_;
 
+  class WindowObserver;

[sky, 2015/04/09 23:44:38]

nit: I believe we try and group class and friend declarations at the beginning
of a section.

[ananta, 2015/04/10 00:36:52]

Moved back to where it was before. I thought it would be better to have it where
it was needed.

+  friend class WindowObserver;
+
+  class WindowAncestorObserver;
+  friend class WindowAncestorObserver;
+
   scoped_ptr<WindowObserver> window_observer_;
 
+  // Tracks the ancestors of the RWHVA window for bounds changed notifications.

[sky, 2015/04/09 23:44:38]

nit: it isn't bounds change, rather screen bounds changing.

[ananta, 2015/04/10 00:36:52]

Reworded.

+  scoped_ptr<WindowAncestorObserver> ancestor_window_observer_;
+
   // Are we in the process of closing?  Tracked so fullscreen views can avoid
   // sending a second shutdown request to the host when they lose the focus
   // after requesting shutdown for another reason (e.g. Escape key).
   bool in_shutdown_;
 
   // True if in the process of handling a window bounds changed notification.
   bool in_bounds_changed_;
 
   // Is this a fullscreen view?
   bool is_fullscreen_;
@@ skipping 136 matching lines @@
   // compositing surface and showing the disambiguation popup.
   gfx::Vector2dF disambiguation_scroll_offset_;
 
   base::WeakPtrFactory<RenderWidgetHostViewAura> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewAura);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_AURA_H_