Address some bubble clipping issues

ui/views/window/non_client_view.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 "ui/views/window/non_client_view.h"
 
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/hit_test.h"
 #include "ui/gfx/geometry/rect_conversions.h"
 #include "ui/views/rect_based_targeting_utils.h"
@@ skipping 13 matching lines @@
 
 // The frame view and the client view are always at these specific indices,
 // because the RootView message dispatch sends messages to items higher in the
 // z-order first and we always want the client view to have first crack at
 // handling mouse messages.
 static const int kFrameViewIndex = 0;
 static const int kClientViewIndex = 1;
 // The overlay view is always on top (index == child_count() - 1).
 
 ////////////////////////////////////////////////////////////////////////////////
+// NonClientFrameView, default implementations:
+
+bool NonClientFrameView::GetClientMask(const gfx::Size& size,
+                                       gfx::Path* mask) const {
+  return false;
+}
+
+////////////////////////////////////////////////////////////////////////////////
 // NonClientView, public:
 
 NonClientView::NonClientView()
     : client_view_(nullptr),
       overlay_view_(nullptr) {
   SetEventTargeter(
       scoped_ptr<views::ViewTargeter>(new views::ViewTargeter(this)));
 }
 
 NonClientView::~NonClientView() {
@@ skipping 108 matching lines @@
 gfx::Size NonClientView::GetMaximumSize() const {
   return frame_view_->GetMaximumSize();
 }
 
 void NonClientView::Layout() {
   LayoutFrameView();
 
   // Then layout the ClientView, using those bounds.
   client_view_->SetBoundsRect(frame_view_->GetBoundsForClientView());
 
+  gfx::Path client_clip;
+  if (frame_view_->GetClientMask(client_view_->size(), &client_clip))
+    client_view_->set_clip_path(client_clip);
+
   // We need to manually call Layout on the ClientView as well for the same
   // reason as above.
   client_view_->Layout();
 
   if (overlay_view_ && overlay_view_->visible())
     overlay_view_->SetBoundsRect(GetLocalBounds());
 }
 
 void NonClientView::ViewHierarchyChanged(
     const ViewHierarchyChangedDetails& details) {
@@ skipping 152 matching lines @@
   // the client view.
   return !GetWidget()->client_view()->bounds().Intersects(rect);
 }
 
 void NonClientFrameView::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   // Overridden to do nothing. The NonClientView manually calls Layout on the
   // FrameView when it is itself laid out, see comment in NonClientView::Layout.
 }
 
 }  // namespace views