Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*"

chrome/browser/themes/theme_syncable_service_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/themes/theme_syncable_service.h"
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 89 matching lines @@
 
   virtual bool UsingDefaultTheme() const OVERRIDE {
     return using_default_theme_;
   }
 
   virtual bool UsingNativeTheme() const OVERRIDE {
     return using_native_theme_;
   }
 
   virtual string GetThemeID() const OVERRIDE {
-    if (theme_extension_)
+    if (theme_extension_.get())
       return theme_extension_->id();
     else
       return std::string();
   }
 
   const extensions::Extension* theme_extension() const {
     return theme_extension_.get();
   }
 
   bool is_dirty() const {
@@ skipping 24 matching lines @@
   DictionaryValue source;
   source.SetString(extension_manifest_keys::kName, name);
   source.Set(extension_manifest_keys::kTheme, new DictionaryValue());
   source.SetString(extension_manifest_keys::kUpdateURL, update_url);
   source.SetString(extension_manifest_keys::kVersion, "0.0.0.0");
   string error;
   scoped_refptr<extensions::Extension> extension =
       extensions::Extension::Create(
           extension_path, location, source,
           extensions::Extension::NO_FLAGS, &error);
-  EXPECT_TRUE(extension);
+  EXPECT_TRUE(extension.get());
   EXPECT_EQ("", error);
   return extension;
 }
 
 }  // namespace
 
 class ThemeSyncableServiceTest : public testing::Test {
  protected:
   ThemeSyncableServiceTest()
       : loop_(base::MessageLoop::TYPE_DEFAULT),
@@ skipping 33 matching lines @@
     theme_extension_ = MakeThemeExtension(base::FilePath(kExtensionFilePath),
                                           kCustomThemeName,
                                           GetThemeLocation(),
                                           kCustomThemeUrl);
     extensions::APIPermissionSet empty_set;
     extensions::URLPatternSet empty_extent;
     scoped_refptr<extensions::PermissionSet> permissions =
         new extensions::PermissionSet(empty_set, empty_extent, empty_extent);
     service->extension_prefs()->AddGrantedPermissions(
         theme_extension_->id(), permissions.get());
-    service->AddExtension(theme_extension_);
+    service->AddExtension(theme_extension_.get());
     ASSERT_EQ(1u, service->extensions()->size());
   }
 
   // Overridden in PolicyInstalledThemeTest below.
   virtual extensions::Manifest::Location GetThemeLocation() {
     return extensions::Manifest::INTERNAL;
   }
 
   FakeThemeService* BuildForProfile(Profile* profile) {
     return static_cast<FakeThemeService*>(
@@ skipping 162 matching lines @@
   ASSERT_EQ(1u, change_list.size());
   EXPECT_TRUE(change_list[0].IsValid());
   EXPECT_EQ(syncer::SyncChange::ACTION_ADD, change_list[0].change_type());
   EXPECT_EQ(syncer::THEMES, change_list[0].sync_data().GetDataType());
 
   const sync_pb::ThemeSpecifics& theme_specifics =
       change_list[0].sync_data().GetSpecifics().theme();
   EXPECT_TRUE(theme_specifics.use_custom_theme());
   EXPECT_EQ(theme_extension_->id(), theme_specifics.custom_theme_id());
   EXPECT_EQ(theme_extension_->name(), theme_specifics.custom_theme_name());
-  EXPECT_EQ(extensions::ManifestURL::GetUpdateURL(theme_extension_).spec(),
-            theme_specifics.custom_theme_update_url());
+  EXPECT_EQ(
+      extensions::ManifestURL::GetUpdateURL(theme_extension_.get()).spec(),
+      theme_specifics.custom_theme_update_url());
 }
 
 TEST_F(ThemeSyncableServiceTest, GetAllSyncData) {
   // Set up theme service to use custom theme.
   fake_theme_service_->SetTheme(theme_extension_.get());
 
   syncer::SyncDataList data_list =
       theme_sync_service_->GetAllSyncData(syncer::THEMES);
 
   ASSERT_EQ(1u, data_list.size());
   const sync_pb::ThemeSpecifics& theme_specifics =
       data_list[0].GetSpecifics().theme();
   EXPECT_TRUE(theme_specifics.use_custom_theme());
   EXPECT_EQ(theme_extension_->id(), theme_specifics.custom_theme_id());
   EXPECT_EQ(theme_extension_->name(), theme_specifics.custom_theme_name());
-  EXPECT_EQ(extensions::ManifestURL::GetUpdateURL(theme_extension_).spec(),
-            theme_specifics.custom_theme_update_url());
+  EXPECT_EQ(
+      extensions::ManifestURL::GetUpdateURL(theme_extension_.get()).spec(),
+      theme_specifics.custom_theme_update_url());
 }
 
 TEST_F(ThemeSyncableServiceTest, ProcessSyncThemeChange) {
   // Set up theme service to use default theme.
   fake_theme_service_->UseDefaultTheme();
   fake_theme_service_->MarkClean();
 
   // Start syncing.
   syncer::SyncError error = theme_sync_service_->MergeDataAndStartSyncing(
       syncer::THEMES, MakeThemeDataList(sync_pb::ThemeSpecifics()),
@@ skipping 41 matching lines @@
 
   // Change current theme to custom theme and notify theme_sync_service_.
   fake_theme_service_->SetTheme(theme_extension_.get());
   theme_sync_service_->OnThemeChange();
   EXPECT_EQ(1u, change_list.size());
   const sync_pb::ThemeSpecifics& change_specifics =
       change_list[0].sync_data().GetSpecifics().theme();
   EXPECT_TRUE(change_specifics.use_custom_theme());
   EXPECT_EQ(theme_extension_->id(), change_specifics.custom_theme_id());
   EXPECT_EQ(theme_extension_->name(), change_specifics.custom_theme_name());
-  EXPECT_EQ(extensions::ManifestURL::GetUpdateURL(theme_extension_).spec(),
-            change_specifics.custom_theme_update_url());
+  EXPECT_EQ(
+      extensions::ManifestURL::GetUpdateURL(theme_extension_.get()).spec(),
+      change_specifics.custom_theme_update_url());
 }
 
 TEST_F(ThemeSyncableServiceTest, StopSync) {
   syncer::SyncChangeList change_list;
   static_cast<FakeSyncChangeProcessor*>(fake_change_processor_.get())->
       SetChangeOutput(&change_list);
 
   // Set up theme service to use default theme.
   fake_theme_service_->UseDefaultTheme();
 
@@ skipping 124 matching lines @@
 
 TEST_F(PolicyInstalledThemeTest, InstallThemeByPolicy) {
   // Set up theme service to use custom theme that was installed by policy.
   fake_theme_service_->SetTheme(theme_extension_.get());
 
   syncer::SyncDataList data_list =
       theme_sync_service_->GetAllSyncData(syncer::THEMES);
 
   ASSERT_EQ(0u, data_list.size());
 }