Implements the "Set to default" button on the zoom bubble.

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/bind.h"
 #include "base/message_loop.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/tab_contents/tab_contents.h"
+#include "chrome/browser/ui/zoom/zoom_controller.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/browser/render_view_host.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
-#include "ui/views/layout/fill_layout.h"
+#include "ui/views/controls/button/text_button.h"
+#include "ui/views/controls/separator.h"
+#include "ui/views/layout/box_layout.h"
 
 namespace {
 
 // The number of milliseconds the bubble should stay on the screen for if it
 // will automatically close.
 const int kBubbleCloseDelay = 400;
 
+// The number of pixels between the percentage label and the separator.
+const int kPercentageSeparatorSpacing = 7;

[Peter Kasting, 2012/07/18 03:26:30]

These should be using standardized values from layout_constants.h, e.g.
kRelatedControlVerticalSpacing.

[Kyle Horimoto, 2012/07/18 23:49:38]

Done.

+
+// The number of pixels between the separator and the button.
+const int kSeparatorButtonSpacing = 2;
+
+// The font size of the percentage label.
+const int kPercentageFontSize = 20;

[Peter Kasting, 2012/07/18 03:26:30]

This should not be hard-coded, because otherwise it won't scale as the user
scales up/down their system font size.  Instead you should use DeriveFont() to
offset the default font size by some amount.

[Kyle Horimoto, 2012/07/18 23:49:38]

Done.

+
 }
 
 // static
 ZoomBubbleView* ZoomBubbleView::zoom_bubble_ = NULL;
 
 // static
 void ZoomBubbleView::ShowBubble(views::View* anchor_view,
-                                int zoom_percent,
+                                TabContents* tab_contents,
                                 bool auto_close) {
   // If the bubble is already showing in this window and its |auto_close_| value
   // is equal to |auto_close|, the bubble can be reused and only the label text
   // needs to be updated.
   if (zoom_bubble_ &&
       zoom_bubble_->anchor_view() == anchor_view &&
       zoom_bubble_->auto_close_ == auto_close) {
-    zoom_bubble_->label_->SetText(
-        l10n_util::GetStringFUTF16Int(IDS_ZOOM_PERCENT, zoom_percent));
-
-    if (auto_close) {
-      // If the bubble should be closed automatically, reset the timer so that
-      // it will show for the full amount of time instead of only what remained
-      // from the previous time.
-      zoom_bubble_->timer_.Reset();
-    }
+    zoom_bubble_->Refresh();
   } else {
     // If the bubble is already showing but its |auto_close_| value is not equal
     // to |auto_close|, the bubble's focus properties must change, so the
     // current bubble must be closed and a new one created.
     if (zoom_bubble_)
       zoom_bubble_->Close();
 
-    zoom_bubble_ = new ZoomBubbleView(anchor_view, zoom_percent, auto_close);
+    zoom_bubble_ = new ZoomBubbleView(anchor_view, tab_contents, auto_close);
     views::BubbleDelegateView::CreateBubble(zoom_bubble_);
     zoom_bubble_->Show();
   }
 }
 
 // static
 void ZoomBubbleView::CloseBubble() {
   if (zoom_bubble_)
     zoom_bubble_->Close();
 }
 
 // static
 bool ZoomBubbleView::IsShowing() {
   return zoom_bubble_ != NULL;
 }
 
 ZoomBubbleView::ZoomBubbleView(views::View* anchor_view,
-                               int zoom_percent,
+                               TabContents* tab_contents,
                                bool auto_close)
     : BubbleDelegateView(anchor_view, views::BubbleBorder::TOP_RIGHT),
       label_(NULL),
-      zoom_percent_(zoom_percent),
+      tab_contents_(tab_contents),
       auto_close_(auto_close) {
   set_use_focusless(auto_close);
 }
 
 ZoomBubbleView::~ZoomBubbleView() {
 }
 
+void ZoomBubbleView::Refresh() {
+  int zoom_percent = tab_contents_->zoom_controller()->zoom_percent();
+  label_->SetText(
+      l10n_util::GetStringFUTF16Int(IDS_ZOOM_PERCENT, zoom_percent));
+
+  if (auto_close_) {
+    // If the bubble should be closed automatically, reset the timer so that
+    // it will show for the full amount of time instead of only what remained
+    // from the previous time.
+    timer_.Reset();
+  }
+}
+
 void ZoomBubbleView::Close() {
   GetWidget()->Close();
 }
 
+void ZoomBubbleView::ButtonPressed(views::Button* sender,
+                                   const views::Event& event) {
+  double default_zoom_level = Profile::FromBrowserContext(
+      tab_contents_->web_contents()->GetBrowserContext())->

[Peter Kasting, 2012/07/18 03:26:30]

Nit: Just use tab_contents_->profile()

[Kyle Horimoto, 2012/07/18 23:49:38]

Done.

+      GetPrefs()->GetDouble(prefs::kDefaultZoomLevel);
+  tab_contents_->web_contents()->GetRenderViewHost()->
+      SetZoomLevel(default_zoom_level);
+}
+
 void ZoomBubbleView::Init() {
-  SetLayoutManager(new views::FillLayout());
+  SetLayoutManager(new views::BoxLayout(
+      views::BoxLayout::kVertical, 0, 0, kPercentageSeparatorSpacing));
+
+  int zoom_percent = tab_contents_->zoom_controller()->zoom_percent();
   label_ = new views::Label(
-      l10n_util::GetStringFUTF16Int(IDS_ZOOM_PERCENT, zoom_percent_));
+      l10n_util::GetStringFUTF16Int(IDS_ZOOM_PERCENT, zoom_percent));
+  gfx::Font font(label_->font().GetFontName(), kPercentageFontSize);
+  label_->SetFont(font);
   AddChildView(label_);
 
   if (auto_close_) {

[Peter Kasting, 2012/07/18 03:26:30]

Wait, so in the auto-close case, no button is available?  Why not?  Seems like
not only would it be potentially useful as-is but would also inform users that
they can reopen the bubble to access that functionality.

Also, if we do this, then so long as the user is hovering the text button, we
shouldn't run the timer (and we should reset it when the mouse moves off the
text button so the bubble doesn't instantly disappear).

Also, even if we don't do this, if the auto-close variant bubble is showing and
I click on the icon in the omnibox, do I then get the non-auto-close variant?

[Kyle Horimoto, 2012/07/18 23:49:38]

Done.

     timer_.Start(
         FROM_HERE,
         base::TimeDelta::FromMilliseconds(kBubbleCloseDelay),
         this,
         &ZoomBubbleView::Close);
+  } else {
+    views::View* container = new views::View();
+    container->SetLayoutManager(new views::BoxLayout(

[Peter Kasting, 2012/07/18 03:26:31]

Why do you need a child container with its own layout manager?  Why can't the
separator and text button be added as direct children?

[Kyle Horimoto, 2012/07/18 23:49:38]

This was so that there could be a different spacing between the percentage label
and the separator and the separator and the button. However, I agree with using
the standard spacing as you expressed earlier, so I'll remove this.

+        views::BoxLayout::kVertical, 0, 0, kSeparatorButtonSpacing));
+    container->AddChildView(new views::Separator());
+    container->AddChildView(new views::TextButton(
+        this, l10n_util::GetStringUTF16(IDS_ZOOM_SET_DEFAULT)));
+
+    AddChildView(container);
   }
 }
 
 void ZoomBubbleView::WindowClosing() {
   DCHECK(zoom_bubble_ == this);
   zoom_bubble_ = NULL;
 }