MacViews: Allows the toolkit-views Zoom Dialog to be used

chrome/browser/ui/views/location_bar/zoom_bubble_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 "chrome/browser/ui/views/location_bar/zoom_bubble_view.h"
 
 #include "base/i18n/number_formatting.h"
 #include "base/i18n/rtl.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/platform_util.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/browser_window.h"
-#include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "chrome/browser/ui/views/location_bar/zoom_view.h"
 #include "chrome/common/extensions/api/extension_action/action_info.h"
 #include "chrome/grit/generated_resources.h"
 #include "chrome/grit/theme_resources.h"
 #include "components/zoom/page_zoom.h"
 #include "components/zoom/zoom_controller.h"
 #include "content/public/browser/notification_source.h"
 #include "extensions/browser/extension_zoom_request_client.h"
 #include "extensions/common/manifest_handlers/icons_handler.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/material_design/material_design_controller.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "ui/base/ui_features.h"
 #include "ui/gfx/favicon_size.h"
 #include "ui/views/controls/button/image_button.h"
 #include "ui/views/controls/button/md_text_button.h"
 #include "ui/views/controls/separator.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/widget/widget.h"
 
+#if !defined(OS_MACOSX) || BUILDFLAG(MAC_VIEWS_BROWSER)
+#include "chrome/browser/ui/views/frame/browser_view.h"
+#endif
+
 // static
 ZoomBubbleView* ZoomBubbleView::zoom_bubble_ = nullptr;
 
 // static
 void ZoomBubbleView::ShowBubble(content::WebContents* web_contents,
+                                const gfx::Point& anchor_point,
                                 DisplayReason reason) {
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
   DCHECK(browser && browser->window() &&
          browser->exclusive_access_manager()->fullscreen_controller());
 
+  views::View* anchor_view = nullptr;
+  ImmersiveModeController* immersive_mode_controller = nullptr;
+  bool is_fullscreen = browser->window()->IsFullscreen();
+#if !defined(OS_MACOSX) || BUILDFLAG(MAC_VIEWS_BROWSER)
   BrowserView* browser_view = BrowserView::GetBrowserViewForBrowser(browser);
-  bool is_fullscreen = browser_view->IsFullscreen();
-  views::View* anchor_view = nullptr;
   if (!is_fullscreen ||
       browser_view->immersive_mode_controller()->IsRevealed()) {
     if (ui::MaterialDesignController::IsSecondaryUiMaterial())
       anchor_view = browser_view->GetLocationBarView();
     else
       anchor_view = browser_view->GetLocationBarView()->zoom_view();
   }
+  immersive_mode_controller = browser_view->immersive_mode_controller();
+#endif
 
   // Find the extension that initiated the zoom change, if any.
   zoom::ZoomController* zoom_controller =
       zoom::ZoomController::FromWebContents(web_contents);
   const zoom::ZoomRequestClient* client = zoom_controller->last_client();
 
   // If the bubble is already showing in this window and the zoom change was not
   // initiated by an extension, then the bubble can be reused and only the label
   // text needs to be updated.
   if (zoom_bubble_ && zoom_bubble_->GetAnchorView() == anchor_view && !client) {
     DCHECK_EQ(web_contents, zoom_bubble_->web_contents_);
     zoom_bubble_->Refresh();
     return;
   }
 
   // If the bubble is already showing but in a different tab, the current
   // bubble must be closed and a new one created.
   CloseCurrentBubble();
 
-  zoom_bubble_ = new ZoomBubbleView(anchor_view, web_contents, reason,
-                                    browser_view->immersive_mode_controller());
+  zoom_bubble_ = new ZoomBubbleView(anchor_view, anchor_point, web_contents,
+                                    reason, immersive_mode_controller);
 
   // If the zoom change was initiated by an extension, capture the relevent
   // information from it.
   if (client) {
     zoom_bubble_->SetExtensionInfo(
         static_cast<const extensions::ExtensionZoomRequestClient*>(client)
             ->extension());
   }
 
+#if !defined(OS_MACOSX) || BUILDFLAG(MAC_VIEWS_BROWSER)
   // If we do not have an anchor view, parent the bubble to the content area.
   if (!anchor_view)
     zoom_bubble_->set_parent_window(web_contents->GetNativeView());
 
   views::Widget* zoom_bubble_widget =
       views::BubbleDialogDelegateView::CreateBubble(zoom_bubble_);
   if (anchor_view) {
     zoom_bubble_widget->AddObserver(
         browser_view->GetLocationBarView()->zoom_view());
   }
+#else
+  gfx::NativeView parent =
+      platform_util::GetViewForWindow(browser->window()->GetNativeWindow());
+  DCHECK(parent);
+  zoom_bubble_->set_arrow(views::BubbleBorder::TOP_RIGHT);
+  zoom_bubble_->set_parent_window(parent);
+  views::BubbleDialogDelegateView::CreateBubble(zoom_bubble_);
+#endif
 
   // Adjust for fullscreen after creation as it relies on the content size.
   if (is_fullscreen)
-    zoom_bubble_->AdjustForFullscreen(browser_view->GetBoundsInScreen());
+    zoom_bubble_->AdjustForFullscreen(browser->window()->GetBounds());
 
   zoom_bubble_->ShowForReason(reason);
 }
 
 // static
 void ZoomBubbleView::CloseCurrentBubble() {
   if (zoom_bubble_)
     zoom_bubble_->CloseBubble();
 }
 
 // static
 ZoomBubbleView* ZoomBubbleView::GetZoomBubble() {
   return zoom_bubble_;
 }
 
+void ZoomBubbleView::Refresh() {
+  zoom::ZoomController* zoom_controller =
+      zoom::ZoomController::FromWebContents(web_contents_);
+  int zoom_percent = zoom_controller->GetZoomPercent();
+  label_->SetText(l10n_util::GetStringFUTF16(
+      IDS_TOOLTIP_ZOOM, base::FormatPercent(zoom_percent)));
+  StartTimerIfNecessary();
+}
+
 ZoomBubbleView::ZoomBubbleView(
     views::View* anchor_view,
+    const gfx::Point& anchor_point,
     content::WebContents* web_contents,
     DisplayReason reason,
     ImmersiveModeController* immersive_mode_controller)
-    : LocationBarBubbleDelegateView(anchor_view, web_contents),
+    : LocationBarBubbleDelegateView(anchor_view, anchor_point, web_contents),
       image_button_(nullptr),
       label_(nullptr),
       web_contents_(web_contents),
       auto_close_(reason == AUTOMATIC),
       immersive_mode_controller_(immersive_mode_controller) {
   set_notify_enter_exit_on_child(true);
-  immersive_mode_controller_->AddObserver(this);
+  if (immersive_mode_controller_)
+    immersive_mode_controller_->AddObserver(this);
   UseCompactMargins();
 }
 
 ZoomBubbleView::~ZoomBubbleView() {
   if (immersive_mode_controller_)
     immersive_mode_controller_->RemoveObserver(this);
 }
 
 void ZoomBubbleView::OnGestureEvent(ui::GestureEvent* event) {
   if (!zoom_bubble_ || !zoom_bubble_->auto_close_ ||
@@ skipping 40 matching lines @@
                             &extension_info_.icon_image->image_skia());
     grid_layout->AddView(image_button_);
   }
 
   // Add zoom label with the new zoom percent.
   zoom::ZoomController* zoom_controller =
       zoom::ZoomController::FromWebContents(web_contents_);
   int zoom_percent = zoom_controller->GetZoomPercent();
   label_ = new views::Label(l10n_util::GetStringFUTF16(
       IDS_TOOLTIP_ZOOM, base::FormatPercent(zoom_percent)));
-  label_->SetFontList(ui::ResourceBundle::GetSharedInstance().GetFontList(
-      ui::ResourceBundle::MediumFont));
+  label_->SetFontList(GetTitleFontList());

[msw, 2017/04/24 18:53:39]

aside: when updating the bubble appearance in a follow up CL to add the +/-
buttons, maybe make this a proper title?

[varkha, 2017/04/26 04:52:59]

Not sure. The new mocks still show this not above the rest of the elements so
maybe just a label.

   grid_layout->AddView(label_);
 
   // Second row.
   grid_layout->AddPaddingRow(0, 8);
   columns = grid_layout->AddColumnSet(1);
   columns->AddColumn(views::GridLayout::FILL, views::GridLayout::FILL, 1,
                      views::GridLayout::USE_PREF, 0, 0);
   grid_layout->StartRow(0, 1);
 
   // Add "Reset to Default" button.
@@ skipping 41 matching lines @@
   immersive_mode_controller_ = nullptr;
 }
 
 void ZoomBubbleView::OnExtensionIconImageChanged(
     extensions::IconImage* /* image */) {
   image_button_->SetImage(views::Button::STATE_NORMAL,
                           &extension_info_.icon_image->image_skia());
   image_button_->SchedulePaint();
 }
 
-void ZoomBubbleView::Refresh() {
-  zoom::ZoomController* zoom_controller =
-      zoom::ZoomController::FromWebContents(web_contents_);
-  int zoom_percent = zoom_controller->GetZoomPercent();
-  label_->SetText(l10n_util::GetStringFUTF16(
-      IDS_TOOLTIP_ZOOM, base::FormatPercent(zoom_percent)));
-  StartTimerIfNecessary();
-}
-
 void ZoomBubbleView::SetExtensionInfo(const extensions::Extension* extension) {
   DCHECK(extension);
   extension_info_.id = extension->id();
   extension_info_.name = extension->name();
 
   ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   const gfx::ImageSkia& default_extension_icon_image =
       *rb.GetImageSkiaNamed(IDR_EXTENSIONS_FAVICON);
   int icon_size = gfx::kFaviconSize;
 
@@ skipping 47 matching lines @@
   }
 }
 
 void ZoomBubbleView::StopTimer() {
   timer_.Stop();
 }
 
 ZoomBubbleView::ZoomBubbleExtensionInfo::ZoomBubbleExtensionInfo() {}
 
 ZoomBubbleView::ZoomBubbleExtensionInfo::~ZoomBubbleExtensionInfo() {}