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

Unified Diff: chrome/common/extensions/manifest_tests/extension_manifest_test.cc

Issue 15836003: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 7 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: chrome/common/extensions/manifest_tests/extension_manifest_test.cc
diff --git a/chrome/common/extensions/manifest_tests/extension_manifest_test.cc b/chrome/common/extensions/manifest_tests/extension_manifest_test.cc
index 61d1c2ee949ee1679a9c1938107b94b097220035..e37eb6486ef0b5b31f3190610a3deaf575e8dde5 100644
--- a/chrome/common/extensions/manifest_tests/extension_manifest_test.cc
+++ b/chrome/common/extensions/manifest_tests/extension_manifest_test.cc
@@ -122,7 +122,7 @@ scoped_refptr<Extension> ExtensionManifestTest::LoadAndExpectSuccess(
std::string error;
scoped_refptr<Extension> extension =
LoadExtension(manifest, &error, location, flags);
- EXPECT_TRUE(extension) << manifest.name();
+ EXPECT_TRUE(extension.get()) << manifest.name();
EXPECT_EQ("", error) << manifest.name();
return extension;
}
@@ -142,7 +142,7 @@ scoped_refptr<Extension> ExtensionManifestTest::LoadAndExpectWarning(
std::string error;
scoped_refptr<Extension> extension =
LoadExtension(manifest, &error, location, flags);
- EXPECT_TRUE(extension) << manifest.name();
+ EXPECT_TRUE(extension.get()) << manifest.name();
EXPECT_EQ("", error) << manifest.name();
EXPECT_EQ(1u, extension->install_warnings().size());
EXPECT_EQ(expected_warning, extension->install_warnings()[0].message);

Powered by Google App Engine
This is Rietveld 408576698