Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(268)

Unified Diff: ios/public/test/test_chrome_provider_initializer.cc

Issue 826803003: Cleanup in ios/chrome and ios/public/test. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/public/test/test_chrome_provider_initializer.cc
diff --git a/ios/public/test/test_chrome_provider_initializer.cc b/ios/public/test/test_chrome_provider_initializer.cc
index 082a0a73bd2772bad672c0db717c75269d6d811d..cd1419b3558297cb596406cb330236622b79ffff 100644
--- a/ios/public/test/test_chrome_provider_initializer.cc
+++ b/ios/public/test/test_chrome_provider_initializer.cc
@@ -10,14 +10,13 @@
namespace ios {
TestChromeProviderInitializer::TestChromeProviderInitializer() {
- chrome_browser_provider_ = new TestChromeBrowserProvider();
- ios::SetChromeBrowserProvider(chrome_browser_provider_);
+ chrome_browser_provider_.reset(new TestChromeBrowserProvider());
+ ios::SetChromeBrowserProvider(chrome_browser_provider_.get());
}
TestChromeProviderInitializer::~TestChromeProviderInitializer() {
- EXPECT_EQ(chrome_browser_provider_, ios::GetChromeBrowserProvider());
+ EXPECT_EQ(chrome_browser_provider_.get(), ios::GetChromeBrowserProvider());
ios::SetChromeBrowserProvider(NULL);
- delete chrome_browser_provider_;
}
} // namespace ios

Powered by Google App Engine
This is Rietveld 408576698