Index: chrome/browser/ui/zoom/zoom_controller_browsertest.cc |
diff --git a/chrome/browser/ui/zoom/zoom_controller_browsertest.cc b/chrome/browser/ui/zoom/zoom_controller_browsertest.cc |
index 1d44bcc9e476756b6710d261274b2bd8ac0b69e5..4bd94e8116fac5a2f8de58fb14016a43ba2664d0 100644 |
--- a/chrome/browser/ui/zoom/zoom_controller_browsertest.cc |
+++ b/chrome/browser/ui/zoom/zoom_controller_browsertest.cc |
@@ -4,16 +4,67 @@ |
#include "chrome/browser/ui/zoom/zoom_controller.h" |
+#include "base/prefs/pref_service.h" |
#include "base/process/kill.h" |
+#include "chrome/browser/profiles/profile.h" |
#include "chrome/browser/ui/browser.h" |
#include "chrome/browser/ui/tabs/tab_strip_model.h" |
+#include "chrome/common/pref_names.h" |
#include "chrome/test/base/in_process_browser_test.h" |
#include "chrome/test/base/ui_test_utils.h" |
#include "content/public/browser/render_process_host.h" |
#include "content/public/browser/web_contents.h" |
#include "content/public/test/browser_test_utils.h" |
+#include "testing/gmock/include/gmock/gmock.h" |
-typedef InProcessBrowserTest ZoomControllerBrowserTest; |
+bool operator==(const ZoomController::ZoomChangedEventData& lhs, |
+ const ZoomController::ZoomChangedEventData& rhs) { |
+ return lhs.web_contents == rhs.web_contents && |
+ lhs.old_zoom_level == rhs.old_zoom_level && |
+ lhs.new_zoom_level == rhs.new_zoom_level && |
+ lhs.zoom_mode == rhs.zoom_mode && |
+ lhs.can_show_bubble == rhs.can_show_bubble; |
+} |
+ |
+class ZoomChangedWatcher : public ZoomObserver { |
+ public: |
+ ZoomChangedWatcher( |
+ content::WebContents* web_contents, |
+ const ZoomController::ZoomChangedEventData& expected_event_data) |
+ : web_contents_(web_contents), |
+ expected_event_data_(expected_event_data), |
+ message_loop_runner_(new content::MessageLoopRunner) { |
+ ZoomController::FromWebContents(web_contents)->AddObserver(this); |
+ } |
+ virtual ~ZoomChangedWatcher() {} |
+ |
+ void Wait() { |
+ message_loop_runner_->Run(); |
+ } |
+ virtual void OnZoomChanged( |
+ const ZoomController::ZoomChangedEventData& event_data) OVERRIDE { |
+ if (event_data == expected_event_data_) |
+ message_loop_runner_->Quit(); |
+ } |
+ |
+ private: |
+ content::WebContents* web_contents_; |
+ ZoomController::ZoomChangedEventData expected_event_data_; |
+ scoped_refptr<content::MessageLoopRunner> message_loop_runner_; |
+ |
+ DISALLOW_COPY_AND_ASSIGN(ZoomChangedWatcher); |
+}; |
+ |
+class TestZoomObserver : public ZoomObserver { |
+ public: |
+ MOCK_METHOD1(OnZoomChanged, |
+ void(const ZoomController::ZoomChangedEventData&)); |
+}; |
+ |
+class ZoomControllerBrowserTest: public InProcessBrowserTest { |
+ protected: |
+ TestZoomObserver zoom_observer_; |
+}; |
// TODO(wjmaclean): Enable this on Android when we can kill the process there. |
#if defined(OS_ANDROID) |
@@ -47,3 +98,50 @@ IN_PROC_BROWSER_TEST_F(ZoomControllerBrowserTest, |
zoom_controller->SetZoomLevel(new_zoom_level); |
EXPECT_FLOAT_EQ(old_zoom_level, zoom_controller->GetZoomLevel()); |
} |
+ |
+IN_PROC_BROWSER_TEST_F(ZoomControllerBrowserTest, OnPreferenceChanged) { |
+ content::WebContents* web_contents = |
+ browser()->tab_strip_model()->GetActiveWebContents(); |
+ double new_default_zoom_level = 1.0; |
+ // Since this page uses the default zoom level, the changes to the default |
+ // zoom level will change the zoom level for this web_contents. |
+ ZoomController::ZoomChangedEventData zoom_change_data( |
+ web_contents, |
+ new_default_zoom_level, |
+ new_default_zoom_level, |
+ ZoomController::ZOOM_MODE_DEFAULT, |
+ true); |
+ ZoomChangedWatcher zoom_change_watcher(web_contents, zoom_change_data); |
+ browser()->profile()->GetZoomLevelPrefs()->SetDouble(prefs::kDefaultZoomLevel, |
+ new_default_zoom_level); |
+ // Because this test relies on a round-trip IPC to/from the renderer process, |
+ // we need to wait for it to propagate. |
+ zoom_change_watcher.Wait(); |
+} |
+ |
+IN_PROC_BROWSER_TEST_F(ZoomControllerBrowserTest, Observe) { |
+ content::WebContents* web_contents = |
+ browser()->tab_strip_model()->GetActiveWebContents(); |
+ ZoomController* zoom_controller = |
+ ZoomController::FromWebContents(web_contents); |
+ zoom_controller->AddObserver(&zoom_observer_); |
+ |
+ double new_zoom_level = 1.0; |
+ // When the event is initiated from HostZoomMap, the old zoom level is not |
+ // available. |
+ ZoomController::ZoomChangedEventData zoom_change_data( |
+ web_contents, |
+ new_zoom_level, |
+ new_zoom_level, |
+ ZoomController::ZOOM_MODE_DEFAULT, |
+ true); // We have a non-empty host, so this will be 'true'. |
+ EXPECT_CALL(zoom_observer_, OnZoomChanged(zoom_change_data)).Times(1); |
+ |
+ content::HostZoomMap* host_zoom_map = |
+ content::HostZoomMap::GetDefaultForBrowserContext( |
+ web_contents->GetBrowserContext()); |
+ |
+ host_zoom_map->SetZoomLevelForHost("about:blank", new_zoom_level); |
+} |
+ |
+ |