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

Unified Diff: chrome/browser/content_settings/cookie_settings_unittest.cc

Issue 15984016: Call scoped_refptr<T>::get() rather than relying on implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 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/content_settings/cookie_settings_unittest.cc
diff --git a/chrome/browser/content_settings/cookie_settings_unittest.cc b/chrome/browser/content_settings/cookie_settings_unittest.cc
index 59d2d55c4686c6588349cf006fb48ff7021da2ef..59f4570f802bedb8e05d9b37562f8704ee7cae18 100644
--- a/chrome/browser/content_settings/cookie_settings_unittest.cc
+++ b/chrome/browser/content_settings/cookie_settings_unittest.cc
@@ -22,17 +22,18 @@ namespace {
class CookieSettingsTest : public testing::Test {
public:
- CookieSettingsTest() :
- ui_thread_(BrowserThread::UI, &message_loop_),
- cookie_settings_(CookieSettings::Factory::GetForProfile(&profile_)),
- kBlockedSite("http://ads.thirdparty.com"),
- kAllowedSite("http://good.allays.com"),
- kFirstPartySite("http://cool.things.com"),
- kBlockedFirstPartySite("http://no.thirdparties.com"),
- kExtensionURL("chrome-extension://deadbeef"),
- kHttpsSite("https://example.com"),
- kAllHttpsSitesPattern(ContentSettingsPattern::FromString("https://*")) {
-}
+ CookieSettingsTest()
+ : ui_thread_(BrowserThread::UI, &message_loop_),
+ cookie_settings_(CookieSettings::Factory::GetForProfile(&profile_)
+ .get()),
+ kBlockedSite("http://ads.thirdparty.com"),
+ kAllowedSite("http://good.allays.com"),
+ kFirstPartySite("http://cool.things.com"),
+ kBlockedFirstPartySite("http://no.thirdparties.com"),
+ kExtensionURL("chrome-extension://deadbeef"),
+ kHttpsSite("https://example.com"),
+ kAllHttpsSitesPattern(ContentSettingsPattern::FromString("https://*")) {
+ }
protected:
base::MessageLoop message_loop_;

Powered by Google App Engine
This is Rietveld 408576698