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

Unified Diff: chrome/browser/extensions/api/browsing_data/browsing_data_test.cc

Issue 16295003: 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/browser/extensions/api/browsing_data/browsing_data_test.cc
diff --git a/chrome/browser/extensions/api/browsing_data/browsing_data_test.cc b/chrome/browser/extensions/api/browsing_data/browsing_data_test.cc
index abb650e284fca3977c94622576e9eade2d2ae600..9d8b0c50fa2f65a9c08d00cee13292edac44f661 100644
--- a/chrome/browser/extensions/api/browsing_data/browsing_data_test.cc
+++ b/chrome/browser/extensions/api/browsing_data/browsing_data_test.cc
@@ -245,10 +245,10 @@ class ExtensionBrowsingDataTest : public InProcessBrowserTest,
EXPECT_EQ(NULL, RunFunctionAndReturnSingleResult(
function.get(), args, browser())) << " for " << args;
} else {
- EXPECT_TRUE(
- MatchPattern(RunFunctionAndReturnError(function, args, browser()),
- extension_browsing_data_api_constants::kDeleteProhibitedError)) <<
- " for " << args;
+ EXPECT_TRUE(MatchPattern(
+ RunFunctionAndReturnError(function.get(), args, browser()),
+ extension_browsing_data_api_constants::kDeleteProhibitedError))
+ << " for " << args;
}
}
@@ -263,11 +263,10 @@ IN_PROC_BROWSER_TEST_F(ExtensionBrowsingDataTest, OneAtATime) {
BrowsingDataRemover::set_removing(true);
scoped_refptr<RemoveBrowsingDataFunction> function =
new RemoveBrowsingDataFunction();
- EXPECT_TRUE(MatchPattern(
- RunFunctionAndReturnError(function,
- kRemoveEverythingArguments,
- browser()),
- extension_browsing_data_api_constants::kOneAtATimeError));
+ EXPECT_TRUE(
+ MatchPattern(RunFunctionAndReturnError(
+ function.get(), kRemoveEverythingArguments, browser()),
+ extension_browsing_data_api_constants::kOneAtATimeError));
BrowsingDataRemover::set_removing(false);
EXPECT_EQ(base::Time(), GetBeginTime());

Powered by Google App Engine
This is Rietveld 408576698