| Index: chrome/browser/ui/browser_browsertest.cc
|
| diff --git a/chrome/browser/ui/browser_browsertest.cc b/chrome/browser/ui/browser_browsertest.cc
|
| index df5826073b2730ef0f1e6d7712ddb83688b814ed..8aebdc82cad128824e9b67c313294536f69554cf 100644
|
| --- a/chrome/browser/ui/browser_browsertest.cc
|
| +++ b/chrome/browser/ui/browser_browsertest.cc
|
| @@ -1589,12 +1589,12 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, MAYBE_PageZoom) {
|
| new content::MessageLoopRunner);
|
| content::HostZoomMap::ZoomLevelChangedCallback callback(
|
| base::Bind(&OnZoomLevelChanged, loop_runner->QuitClosure()));
|
| - content::HostZoomMap::GetForBrowserContext(
|
| - browser()->profile())->AddZoomLevelChangedCallback(callback);
|
| + scoped_ptr<content::HostZoomMap::Subscription> sub =
|
| + content::HostZoomMap::GetForBrowserContext(
|
| + browser()->profile())->AddZoomLevelChangedCallback(callback);
|
| chrome::Zoom(browser(), content::PAGE_ZOOM_IN);
|
| loop_runner->Run();
|
| - content::HostZoomMap::GetForBrowserContext(
|
| - browser()->profile())->RemoveZoomLevelChangedCallback(callback);
|
| + sub.reset();
|
| EXPECT_EQ(contents->GetZoomPercent(&enable_plus, &enable_minus), 110);
|
| EXPECT_TRUE(enable_plus);
|
| EXPECT_TRUE(enable_minus);
|
| @@ -1605,12 +1605,12 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, MAYBE_PageZoom) {
|
| new content::MessageLoopRunner);
|
| content::HostZoomMap::ZoomLevelChangedCallback callback(
|
| base::Bind(&OnZoomLevelChanged, loop_runner->QuitClosure()));
|
| - content::HostZoomMap::GetForBrowserContext(
|
| - browser()->profile())->AddZoomLevelChangedCallback(callback);
|
| + scoped_ptr<content::HostZoomMap::Subscription> sub =
|
| + content::HostZoomMap::GetForBrowserContext(
|
| + browser()->profile())->AddZoomLevelChangedCallback(callback);
|
| chrome::Zoom(browser(), content::PAGE_ZOOM_RESET);
|
| loop_runner->Run();
|
| - content::HostZoomMap::GetForBrowserContext(
|
| - browser()->profile())->RemoveZoomLevelChangedCallback(callback);
|
| + sub.reset();
|
| EXPECT_EQ(contents->GetZoomPercent(&enable_plus, &enable_minus), 100);
|
| EXPECT_TRUE(enable_plus);
|
| EXPECT_TRUE(enable_minus);
|
| @@ -1621,12 +1621,12 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, MAYBE_PageZoom) {
|
| new content::MessageLoopRunner);
|
| content::HostZoomMap::ZoomLevelChangedCallback callback(
|
| base::Bind(&OnZoomLevelChanged, loop_runner->QuitClosure()));
|
| - content::HostZoomMap::GetForBrowserContext(
|
| - browser()->profile())->AddZoomLevelChangedCallback(callback);
|
| + scoped_ptr<content::HostZoomMap::Subscription> sub =
|
| + content::HostZoomMap::GetForBrowserContext(
|
| + browser()->profile())->AddZoomLevelChangedCallback(callback);
|
| chrome::Zoom(browser(), content::PAGE_ZOOM_OUT);
|
| loop_runner->Run();
|
| - content::HostZoomMap::GetForBrowserContext(
|
| - browser()->profile())->RemoveZoomLevelChangedCallback(callback);
|
| + sub.reset();
|
| EXPECT_EQ(contents->GetZoomPercent(&enable_plus, &enable_minus), 90);
|
| EXPECT_TRUE(enable_plus);
|
| EXPECT_TRUE(enable_minus);
|
|
|