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

Unified Diff: chrome/test/perf/perf_ui_test_suite.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
« no previous file with comments | « chrome/test/perf/frame_rate/frame_rate_tests.cc ('k') | chrome/test/reliability/automated_ui_test_base.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/perf/perf_ui_test_suite.cc
diff --git a/chrome/test/perf/perf_ui_test_suite.cc b/chrome/test/perf/perf_ui_test_suite.cc
index 1f15015c23550d0127145d0240ea8b25df8f1a08..0c6312d18f87cb881dd5ba2416f0321dc9019a98 100644
--- a/chrome/test/perf/perf_ui_test_suite.cc
+++ b/chrome/test/perf/perf_ui_test_suite.cc
@@ -153,7 +153,7 @@ void PerfUITestSuite::BuildCachedThemePakIn(
*valid_value,
Extension::NO_FLAGS,
&error);
- if (!extension)
+ if (!extension.get())
LOG(FATAL) << "Error loading theme extension: " << error;
// Build the "Cached Theme.pak" file in the template. (It's not committed
@@ -166,8 +166,8 @@ void PerfUITestSuite::BuildCachedThemePakIn(
&message_loop_);
scoped_refptr<BrowserThemePack> theme(
- BrowserThemePack::BuildFromExtension(extension));
- if (!theme)
+ BrowserThemePack::BuildFromExtension(extension.get()));
+ if (!theme.get())
LOG(FATAL) << "Failed to load theme from extension";
theme->WriteToDisk(extension_base.AppendASCII("Cached Theme.pak"));
« no previous file with comments | « chrome/test/perf/frame_rate/frame_rate_tests.cc ('k') | chrome/test/reliability/automated_ui_test_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698