Issue 32919: Update the Zoom NSMenuItems (Zoom-In/Zoom-out/Actual-Size) when the zoom state changes.

chrome/browser/ui/zoom/zoom_controller_unittest.cc

Index: chrome/browser/ui/zoom/zoom_controller_unittest.cc
diff --git a/chrome/browser/ui/zoom/zoom_controller_unittest.cc b/chrome/browser/ui/zoom/zoom_controller_unittest.cc
index cba7933eef340254d99e31d9b7bc48a618e77919..4c37ad3bbfef55feee52de583c79292142dd5c84 100644
--- a/chrome/browser/ui/zoom/zoom_controller_unittest.cc
+++ b/chrome/browser/ui/zoom/zoom_controller_unittest.cc
@@ -2,22 +2,36 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+
 #include "base/message_loop/message_loop.h"
 #include "base/prefs/pref_service.h"
+#include "chrome/app/chrome_command_ids.h"
+#include "chrome/browser/chrome_page_zoom.h"
 #include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_commands.h"
+#include "chrome/browser/ui/browser_command_controller.h"
 #include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/zoom/zoom_controller.h"
 #include "chrome/browser/ui/zoom/zoom_observer.h"
 #include "chrome/common/pref_names.h"
+#include "chrome/test/base/browser_with_test_window_test.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/browser/host_zoom_map.h"
 #include "content/public/browser/navigation_details.h"
+#include "content/public/browser/site_instance.h"
+#include "content/public/browser/web_contents.h"
 #include "content/public/common/frame_navigate_params.h"
 #include "content/public/test/test_utils.h"
+#include "content/public/test/web_contents_tester.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+using content::SiteInstance;

[Ben Goodger (Google), 2014/06/16 17:37:34]

nit: prevailing style in chromium is to avoid doing this unless you are having
significant trouble keeping within the 80 col limit. (significant trouble
doesn't mean line wrapping a single statement, it means you're having trouble
fitting broken lines onto a single line).

[sarka, 2014/06/16 20:25:57]

Hi Ben, Did you mean to comment about a particular line in this file or just
something more general.

[Alexei Svitkine (slow), 2014/06/17 19:44:01]

I think Ben is suggesting to remove the "using" statements and just prefix
things with their namespace in the file itself.

+using content::WebContents;
+using content::WebContentsTester;
+
 class TestZoomObserver : public ZoomObserver {
  public:
   MOCK_METHOD2(OnZoomChanged, void(content::WebContents*, bool));
@@ -61,3 +75,126 @@ TEST_F(ZoomControllerTest, Observe) {
 
   host_zoom_map->SetZoomLevelForHost(std::string(), 110.0f);
 }
+
+class ZoomCommandsUnitTest : public BrowserWithTestWindowTest {
+ public:
+  ZoomCommandsUnitTest() {}
+  virtual ~ZoomCommandsUnitTest() {}
+
+  WebContents* CreateTestWebContents() {
+    return WebContentsTester::CreateTestWebContents(
+        profile(), SiteInstance::Create(profile()));
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ZoomCommandsUnitTest);
+};
+
+TEST_F(ZoomCommandsUnitTest, OnMaxZoomIn) {
+  TabStripModel* tab_strip_model = browser()->tab_strip_model();
+
+  GURL url("http://www.google.com");
+  WebContents* contents1 = CreateTestWebContents();
+  tab_strip_model->AppendWebContents(contents1, true);
+  WebContentsTester::For(contents1)->NavigateAndCommit(url);
+
+  for (int i = 0; i < 9; ++i) {
+    chrome_page_zoom::Zoom(contents1, content::PAGE_ZOOM_IN);
+  }
+
+  bool dummy = false;
+
+  EXPECT_EQ(contents1->GetZoomPercent(&dummy, &dummy), 500.0f);
+  EXPECT_FALSE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_PLUS));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_NORMAL));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_MINUS));
+}
+
+TEST_F(ZoomCommandsUnitTest, OnMaxZoomOut) {
+  TabStripModel* tab_strip_model = browser()->tab_strip_model();
+
+  GURL url("http://www.google.com");
+  WebContents* contents1 = CreateTestWebContents();
+  tab_strip_model->AppendWebContents(contents1, true);
+  WebContentsTester::For(contents1)->NavigateAndCommit(url);
+
+  for (int i = 0; i < 7; ++i) {
+    chrome_page_zoom::Zoom(contents1, content::PAGE_ZOOM_OUT);
+  }
+
+  bool dummy = false;
+
+  EXPECT_EQ(contents1->GetZoomPercent(&dummy, &dummy), 25.0f);
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_PLUS));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_NORMAL));
+  EXPECT_FALSE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_MINUS));
+}
+
+TEST_F(ZoomCommandsUnitTest, OnZoomReset) {
+  TabStripModel* tab_strip_model = browser()->tab_strip_model();
+
+  GURL url("http://www.google.com");
+  WebContents* contents1 = CreateTestWebContents();
+  tab_strip_model->AppendWebContents(contents1, true);
+  WebContentsTester::For(contents1)->NavigateAndCommit(url);
+
+  chrome_page_zoom::Zoom(contents1, content::PAGE_ZOOM_RESET);
+
+  bool dummy = false;
+
+  EXPECT_EQ(contents1->GetZoomPercent(&dummy, &dummy), 100.0f);
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_PLUS));
+  EXPECT_FALSE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_NORMAL));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_MINUS));
+}
+
+TEST_F(ZoomCommandsUnitTest, OnZoomLevelChanged) {
+  TabStripModel* tab_strip_model = browser()->tab_strip_model();
+
+  GURL url("http://www.google.com");
+  WebContents* contents1 = CreateTestWebContents();
+  tab_strip_model->AppendWebContents(contents1, true);
+  WebContentsTester::For(contents1)->NavigateAndCommit(url);
+
+  chrome_page_zoom::Zoom(contents1, content::PAGE_ZOOM_IN);
+
+  bool dummy = false;
+
+  EXPECT_EQ(contents1->GetZoomPercent(&dummy, &dummy), 110.0f);
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_PLUS));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_NORMAL));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_MINUS));
+}
+
+TEST_F(ZoomCommandsUnitTest, OnZoomChangedForActiveTab) {
+  TabStripModel* tab_strip_model = browser()->tab_strip_model();
+
+  GURL url("http://www.google.com");
+  GURL url1("http://code.google.com");
+
+  // Add First tab
+  WebContents* contents1 = CreateTestWebContents();
+  tab_strip_model->AppendWebContents(contents1, true);
+  WebContentsTester::For(contents1)->NavigateAndCommit(url);
+
+  bool dummy = false;
+
+  EXPECT_EQ(contents1->GetZoomPercent(&dummy, &dummy), 100.0f);
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_PLUS));
+  EXPECT_FALSE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_NORMAL));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_MINUS));
+
+  // Add Second tab
+  WebContents* contents2 = CreateTestWebContents();
+  tab_strip_model->AppendWebContents(contents2, true);
+  WebContentsTester::For(contents2)->NavigateAndCommit(url1);
+
+  tab_strip_model->ActivateTabAt(1, true);
+  EXPECT_TRUE(tab_strip_model->IsTabSelected(1));
+  chrome_page_zoom::Zoom(contents2, content::PAGE_ZOOM_OUT);
+
+  EXPECT_EQ(contents2->GetZoomPercent(&dummy, &dummy), 90.0f);
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_PLUS));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_NORMAL));
+  EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_ZOOM_MINUS));
+}