Fixing the dynamic positioning (move with anchor) for the app launcher

ui/app_list/views/app_list_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/app_list/views/app_list_view.h"
 
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "ui/app_list/app_list_constants.h"
@@ skipping 46 matching lines @@
     delegate_->SetModel(model_.get());
   model_->AddObserver(this);
 }
 
 AppListView::~AppListView() {
   model_->RemoveObserver(this);
   // Models are going away, ensure their references are cleared.
   RemoveAllChildViews(true);
 }
 
-void AppListView::InitAsBubble(gfx::NativeView parent,
-                               PaginationModel* pagination_model,
-                               views::View* anchor,
-                               const gfx::Point& anchor_point,
-                               views::BubbleBorder::Arrow arrow,
-                               bool border_accepts_events) {
-  app_list_main_view_ = new AppListMainView(delegate_.get(),
-                                            model_.get(),
-                                            pagination_model,
-                                            parent);
-  AddChildView(app_list_main_view_);
-#if defined(USE_AURA)
-  app_list_main_view_->SetPaintToLayer(true);
-  app_list_main_view_->SetFillsBoundsOpaquely(false);
-  app_list_main_view_->layer()->SetMasksToBounds(true);
-#endif
+void AppListView::InitAsBubbleAttachedToAnchor(
+    gfx::NativeView parent,
+    PaginationModel* pagination_model,
+    views::View* anchor,
+    const gfx::Point& anchor_offset,
+    views::BubbleBorder::Arrow arrow,
+    bool border_accepts_events) {
+  set_anchor_view(anchor);
+  set_anchor_offset(anchor_offset);
+  InitAsBubbleInternal(parent, pagination_model, arrow, border_accepts_events);
+}
 
-  signin_view_ = new SigninView(
-      GetSigninDelegate(),
-      app_list_main_view_->GetPreferredSize().width());
-  AddChildView(signin_view_);
-
-  OnSigninStatusChanged();
-
-  set_anchor_view(anchor);
-  set_anchor_rect(gfx::Rect(anchor_point, gfx::Size()));
-  set_color(kContentsBackgroundColor);
-  set_margins(gfx::Insets());
-  set_move_with_anchor(true);
-  set_parent_window(parent);
-  set_close_on_deactivate(false);
-  set_close_on_esc(false);
-  set_anchor_view_insets(gfx::Insets(kArrowOffset, kArrowOffset,
-                                     kArrowOffset, kArrowOffset));
-  set_border_accepts_events(border_accepts_events);
-  set_shadow(views::BubbleBorder::BIG_SHADOW);
-#if defined(USE_AURA) && defined(OS_WIN)
-  if (!ui::win::IsAeroGlassEnabled() ||
-      CommandLine::ForCurrentProcess()->HasSwitch(
-      switches::kDisableDwmComposition)) {
-    set_shadow(views::BubbleBorder::NO_SHADOW_OPAQUE_BORDER);
-  }
-#endif
-  views::BubbleDelegateView::CreateBubble(this);
-  SetBubbleArrow(arrow);
-
-#if defined(USE_AURA)
-  GetWidget()->GetNativeWindow()->layer()->SetMasksToBounds(true);
-  GetBubbleFrameView()->set_background(new AppListBackground(
-      GetBubbleFrameView()->bubble_border()->GetBorderCornerRadius(),
-      app_list_main_view_));
-  set_background(NULL);
-#else
-  set_background(new AppListBackground(
-      GetBubbleFrameView()->bubble_border()->GetBorderCornerRadius(),
-      app_list_main_view_));
-
-  // On non-aura the bubble has two widgets, and it's possible for the border
-  // to be shown independently in odd situations. Explicitly hide the bubble
-  // widget to ensure that any WM_WINDOWPOSCHANGED messages triggered by the
-  // window manager do not have the SWP_SHOWWINDOW flag set which would cause
-  // the border to be shown. See http://crbug.com/231687 .
-  GetWidget()->Hide();
-#endif
+void AppListView::InitAsBubbleAtFixedLocation(
+    gfx::NativeView parent,
+    PaginationModel* pagination_model,
+    const gfx::Point& anchor_point_in_screen,
+    views::BubbleBorder::Arrow arrow,
+    bool border_accepts_events) {
+  set_anchor_view(NULL);
+  set_anchor_rect(gfx::Rect(anchor_point_in_screen, gfx::Size()));
+  InitAsBubbleInternal(parent, pagination_model, arrow, border_accepts_events);
 }
 
 void AppListView::SetBubbleArrow(views::BubbleBorder::Arrow arrow) {
   GetBubbleFrameView()->bubble_border()->set_arrow(arrow);
   SizeToContents();  // Recalcuates with new border.
   GetBubbleFrameView()->SchedulePaint();
 }
 
 void AppListView::SetAnchorPoint(const gfx::Point& anchor_point) {
   set_anchor_rect(gfx::Rect(anchor_point, gfx::Size()));
@@ skipping 65 matching lines @@
 #if defined(USE_AURA)
   gfx::NativeWindow window =
       GetWidget()->GetTopLevelWidget()->GetNativeWindow();
   return window->GetRootWindow()->GetAcceleratedWidget();
 #else
   return GetWidget()->GetTopLevelWidget()->GetNativeWindow();
 #endif
 }
 #endif
 
+void AppListView::InitAsBubbleInternal(gfx::NativeView parent,
+                                       PaginationModel* pagination_model,
+                                       views::BubbleBorder::Arrow arrow,
+                                       bool border_accepts_events) {
+  app_list_main_view_ = new AppListMainView(delegate_.get(),
+                                            model_.get(),
+                                            pagination_model,
+                                            parent);
+  AddChildView(app_list_main_view_);
+#if defined(USE_AURA)
+  app_list_main_view_->SetPaintToLayer(true);
+  app_list_main_view_->SetFillsBoundsOpaquely(false);
+  app_list_main_view_->layer()->SetMasksToBounds(true);
+#endif
+
+  signin_view_ = new SigninView(
+      GetSigninDelegate(),
+      app_list_main_view_->GetPreferredSize().width());
+  AddChildView(signin_view_);
+
+  OnSigninStatusChanged();
+  set_color(kContentsBackgroundColor);
+  set_margins(gfx::Insets());
+  set_move_with_anchor(true);
+  set_parent_window(parent);
+  set_close_on_deactivate(false);
+  set_close_on_esc(false);
+  set_anchor_view_insets(gfx::Insets(kArrowOffset, kArrowOffset,
+                                     kArrowOffset, kArrowOffset));
+  set_border_accepts_events(border_accepts_events);
+  set_shadow(views::BubbleBorder::BIG_SHADOW);
+#if defined(USE_AURA) && defined(OS_WIN)
+  if (!ui::win::IsAeroGlassEnabled() ||
+      CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kDisableDwmComposition)) {
+    set_shadow(views::BubbleBorder::NO_SHADOW_OPAQUE_BORDER);
+  }
+#endif
+  views::BubbleDelegateView::CreateBubble(this);
+  SetBubbleArrow(arrow);
+
+#if defined(USE_AURA)
+  GetWidget()->GetNativeWindow()->layer()->SetMasksToBounds(true);
+  GetBubbleFrameView()->set_background(new AppListBackground(
+      GetBubbleFrameView()->bubble_border()->GetBorderCornerRadius(),
+      app_list_main_view_));
+  set_background(NULL);
+#else
+  set_background(new AppListBackground(
+      GetBubbleFrameView()->bubble_border()->GetBorderCornerRadius(),
+      app_list_main_view_));
+
+  // On non-aura the bubble has two widgets, and it's possible for the border
+  // to be shown independently in odd situations. Explicitly hide the bubble
+  // widget to ensure that any WM_WINDOWPOSCHANGED messages triggered by the
+  // window manager do not have the SWP_SHOWWINDOW flag set which would cause
+  // the border to be shown. See http://crbug.com/231687 .
+  GetWidget()->Hide();
+#endif
+}
+
 views::View* AppListView::GetInitiallyFocusedView() {
   return app_list_main_view_->search_box_view()->search_box();
 }
 
 gfx::ImageSkia AppListView::GetWindowIcon() {
   if (delegate_)
     return delegate_->GetWindowIcon();
 
   return gfx::ImageSkia();
 }
@@ skipping 67 matching lines @@
 
 void AppListView::OnAppListModelSigninStatusChanged() {
   OnSigninStatusChanged();
 }
 
 void AppListView::OnAppListModelCurrentUserChanged() {
   OnSigninStatusChanged();
 }
 
 }  // namespace app_list