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); |