| 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 73e0690d446eb91a6b17ddb4c254715179645123..74adbe1b6ce31efbaf1354467650f81ec150c9b2 100644
|
| --- a/chrome/browser/ui/zoom/zoom_controller_browsertest.cc
|
| +++ b/chrome/browser/ui/zoom/zoom_controller_browsertest.cc
|
| @@ -2,7 +2,7 @@
|
| // Use of this source code is governed by a BSD-style license that can be
|
| // found in the LICENSE file.
|
|
|
| -#include "chrome/browser/ui/zoom/zoom_controller.h"
|
| +#include "components/ui/zoom/zoom_controller.h"
|
|
|
| #include "base/prefs/pref_service.h"
|
| #include "base/process/kill.h"
|
| @@ -20,6 +20,9 @@
|
| #include "content/public/test/browser_test_utils.h"
|
| #include "testing/gmock/include/gmock/gmock.h"
|
|
|
| +using ui_zoom::ZoomController;
|
| +using ui_zoom::ZoomObserver;
|
| +
|
| bool operator==(const ZoomController::ZoomChangedEventData& lhs,
|
| const ZoomController::ZoomChangedEventData& rhs) {
|
| return lhs.web_contents == rhs.web_contents &&
|
| @@ -57,16 +60,7 @@ class ZoomChangedWatcher : public ZoomObserver {
|
| 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_;
|
| -};
|
| +typedef InProcessBrowserTest ZoomControllerBrowserTest;
|
|
|
| #if defined(OS_ANDROID)
|
| #define MAYBE_CrashedTabsDoNotChangeZoom DISABLED_CrashedTabsDoNotChangeZoom
|
| @@ -145,9 +139,6 @@ IN_PROC_BROWSER_TEST_F(ZoomControllerBrowserTest, ErrorPagesCanZoom) {
|
| 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
|
| @@ -158,11 +149,12 @@ IN_PROC_BROWSER_TEST_F(ZoomControllerBrowserTest, Observe) {
|
| 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);
|
| + ZoomChangedWatcher zoom_change_watcher(web_contents, zoom_change_data);
|
|
|
| content::HostZoomMap* host_zoom_map =
|
| content::HostZoomMap::GetDefaultForBrowserContext(
|
| web_contents->GetBrowserContext());
|
|
|
| host_zoom_map->SetZoomLevelForHost("about:blank", new_zoom_level);
|
| + zoom_change_watcher.Wait();
|
| }
|
|
|