MacViews: Allows the toolkit-views Manage Passwords Dialog to be used

chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.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 "chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h"
 
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
@@ skipping 31 matching lines @@
 }
 
 void LocationBarBubbleDelegateView::WebContentMouseHandler::OnTouchEvent(
     ui::TouchEvent* event) {
   if (event->type() == ui::ET_TOUCH_PRESSED)
     bubble_->CloseBubble();
 }
 
 LocationBarBubbleDelegateView::LocationBarBubbleDelegateView(
     views::View* anchor_view,
+    const gfx::Point& anchor_point,
+    views::BubbleBorder::Arrow arrow,
+    content::WebContents* web_contents)
+    : BubbleDialogDelegateView(
+          anchor_view,
+          anchor_view ? views::BubbleBorder::TOP_RIGHT : arrow) {
+  SetupAnchor(anchor_view, anchor_point, arrow, web_contents);
+}
+
+LocationBarBubbleDelegateView::LocationBarBubbleDelegateView(
+    views::View* anchor_view,
     content::WebContents* web_contents)
     : BubbleDialogDelegateView(anchor_view,

[tapted, 2017/04/12 05:08:42]

Can the overload just delegate to the other constructor (c++11 feature)? i.e.
initialize LocationBarBubbleDelegateView rather than BubbleDialogDelegateView

[varkha, 2017/04/12 09:16:43]

Done.

                                anchor_view ? views::BubbleBorder::TOP_RIGHT
                                            : views::BubbleBorder::NONE) {
-  // Add observer to close the bubble if the fullscreen state changes.
-  if (web_contents) {
-    Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
-    registrar_.Add(
-        this, chrome::NOTIFICATION_FULLSCREEN_CHANGED,
-        content::Source<FullscreenController>(
-            browser->exclusive_access_manager()->fullscreen_controller()));
-  }
-  // Compensate for built-in vertical padding in the anchor view's image.
-  // In the case of Harmony, this is just compensating for the location bar's
-  // border thickness, as the bubble's top border should overlap it.
-  set_anchor_view_insets(gfx::Insets(
-      GetLayoutConstant(LOCATION_BAR_BUBBLE_ANCHOR_VERTICAL_INSET), 0));
+  SetupAnchor(anchor_view, gfx::Point(), views::BubbleBorder::NONE,
+              web_contents);
 }
 
 LocationBarBubbleDelegateView::~LocationBarBubbleDelegateView() {}
 
 void LocationBarBubbleDelegateView::ShowForReason(DisplayReason reason) {
   if (reason == USER_GESTURE) {
     // In the USER_GESTURE case, the icon will be in an active state so the
     // bubble doesn't need an arrow.
     SetArrowPaintType(views::BubbleBorder::PAINT_TRANSPARENT);
     GetWidget()->Show();
   } else {
     GetWidget()->ShowInactive();
   }
 }
 
 int LocationBarBubbleDelegateView::GetDialogButtons() const {
   return ui::DIALOG_BUTTON_NONE;
 }
 
 void LocationBarBubbleDelegateView::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   DCHECK_EQ(chrome::NOTIFICATION_FULLSCREEN_CHANGED, type);
   GetWidget()->SetVisibilityAnimationTransition(views::Widget::ANIMATE_NONE);
   CloseBubble();
 }
 
+void LocationBarBubbleDelegateView::SetupAnchor(
+    views::View* anchor_view,
+    const gfx::Point& anchor_point,
+    views::BubbleBorder::Arrow arrow,
+    content::WebContents* web_contents) {
+  // Add observer to close the bubble if the fullscreen state changes.
+  if (web_contents) {
+    Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
+    registrar_.Add(
+        this, chrome::NOTIFICATION_FULLSCREEN_CHANGED,
+        content::Source<FullscreenController>(
+            browser->exclusive_access_manager()->fullscreen_controller()));
+  }
+  if (!anchor_view) {
+    SetAnchorRect(gfx::Rect(anchor_point, gfx::Size()));
+    return;
+  }
+
+  // Compensate for built-in vertical padding in the anchor view's image.
+  // In the case of Harmony, this is just compensating for the location bar's
+  // border thickness, as the bubble's top border should overlap it.
+  set_anchor_view_insets(gfx::Insets(
+      GetLayoutConstant(LOCATION_BAR_BUBBLE_ANCHOR_VERTICAL_INSET), 0));
+}
+
 void LocationBarBubbleDelegateView::CloseBubble() {
   GetWidget()->Close();
 }
 
 void LocationBarBubbleDelegateView::AdjustForFullscreen(
     const gfx::Rect& screen_bounds) {
   if (GetAnchorView())
     return;
 
   const int kBubblePaddingFromScreenEdge = 20;
   int horizontal_offset = width() / 2 + kBubblePaddingFromScreenEdge;
   const int x_pos = base::i18n::IsRTL()
                         ? (screen_bounds.x() + horizontal_offset)
                         : (screen_bounds.right() - horizontal_offset);
   SetAnchorRect(gfx::Rect(x_pos, screen_bounds.y(), 0, 0));
 }