| Index: chrome/browser/ui/cocoa/location_bar/zoom_decoration_browsertest.mm
|
| diff --git a/chrome/browser/ui/cocoa/location_bar/zoom_decoration_browsertest.mm b/chrome/browser/ui/cocoa/location_bar/zoom_decoration_browsertest.mm
|
| index eef9f857a7e47be351d5b01124490862542bb890..0e4d425eb3c4a9b4767f62c3282ebd4872417fd1 100644
|
| --- a/chrome/browser/ui/cocoa/location_bar/zoom_decoration_browsertest.mm
|
| +++ b/chrome/browser/ui/cocoa/location_bar/zoom_decoration_browsertest.mm
|
| @@ -14,6 +14,7 @@
|
| #include "chrome/browser/ui/toolbar/test_toolbar_model.h"
|
| #include "chrome/test/base/in_process_browser_test.h"
|
| #include "components/ui/zoom/page_zoom.h"
|
| +#include "components/ui/zoom/zoom_controller.h"
|
| #include "content/public/browser/host_zoom_map.h"
|
| #include "content/public/test/test_utils.h"
|
|
|
| @@ -78,6 +79,16 @@ class ZoomDecorationTest : public InProcessBrowserTest {
|
| IN_PROC_BROWSER_TEST_F(ZoomDecorationTest, BubbleAtDefaultZoom) {
|
| ZoomDecoration* zoom_decoration = GetZoomDecoration();
|
|
|
| + // TODO(wjmaclean): This shouldn't be necessary, but at present this test
|
| + // assumes the various Zoom() calls do not invoke a notification
|
| + // bubble, which prior to https://codereview.chromium.org/940673002/
|
| + // was accomplished by not showing the bubble for inactive windows.
|
| + // Since we now need to be able to show the zoom bubble as a notification
|
| + // on non-active pages, this test should be revised to account for
|
| + // these notifications.
|
| + ui_zoom::ZoomController::FromWebContents(
|
| + GetLocationBar()->GetWebContents())->SetShowsNotificationBubble(false);
|
| +
|
| // Zoom in and reset.
|
| EXPECT_FALSE(zoom_decoration->IsVisible());
|
| Zoom(content::PAGE_ZOOM_IN);
|
|
|