Improve some naming

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 37 matching lines @@
 #if defined(OS_WIN)
 const base::FilePath::CharType kExtensionFilePath[] =
     FILE_PATH_LITERAL("c:\\foo");
 #elif defined(OS_POSIX)
 const base::FilePath::CharType kExtensionFilePath[] = FILE_PATH_LITERAL("/oo");
 #endif
 
 class FakeThemeService : public ThemeService {
  public:
   FakeThemeService() :
-    using_native_theme_(false),
+    using_system_theme_(false),
     using_default_theme_(false),
     theme_extension_(NULL),
     is_dirty_(false) {}
 
   // ThemeService implementation
   virtual void SetTheme(const extensions::Extension* extension) OVERRIDE {
     is_dirty_ = true;
     theme_extension_ = extension;
-    using_native_theme_ = false;
+    using_system_theme_ = false;
     using_default_theme_ = false;
   }
 
   virtual void UseDefaultTheme() OVERRIDE {
     is_dirty_ = true;
     using_default_theme_ = true;
-    using_native_theme_ = false;
+    using_system_theme_ = false;
     theme_extension_ = NULL;
   }
 
-  virtual void SetNativeTheme() OVERRIDE {
+  virtual void UseSystemTheme() OVERRIDE {
     is_dirty_ = true;
-    using_native_theme_ = true;
+    using_system_theme_ = true;
     using_default_theme_ = false;
     theme_extension_ = NULL;
   }
 
   virtual bool UsingDefaultTheme() const OVERRIDE {
     return using_default_theme_;
   }
 
-  virtual bool UsingNativeTheme() const OVERRIDE {
-    return using_native_theme_;
+  virtual bool UsingSystemTheme() const OVERRIDE {
+    return using_system_theme_;
   }
 
   virtual string GetThemeID() const OVERRIDE {
     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 {
     return is_dirty_;
   }
 
   void MarkClean() {
     is_dirty_ = false;
   }
 
  private:
-  bool using_native_theme_;
+  bool using_system_theme_;
   bool using_default_theme_;
   scoped_refptr<const extensions::Extension> theme_extension_;
   bool is_dirty_;
 };
 
 KeyedService* BuildMockThemeService(content::BrowserContext* profile) {
   return new FakeThemeService;
 }
 
 scoped_refptr<extensions::Extension> MakeThemeExtension(
@@ skipping 196 matching lines @@
           ->MergeDataAndStartSyncing(
                 syncer::THEMES,
                 MakeThemeDataList(theme_specifics),
                 scoped_ptr<syncer::SyncChangeProcessor>(
                     new syncer::SyncChangeProcessorWrapperForTest(
                         fake_change_processor_.get())),
                 scoped_ptr<syncer::SyncErrorFactory>(
                     new syncer::SyncErrorFactoryMock()))
           .error();
   EXPECT_FALSE(error.IsSet()) << error.message();
-  EXPECT_TRUE(fake_theme_service_->UsingNativeTheme());
+  EXPECT_TRUE(fake_theme_service_->UsingSystemTheme());
 }
 
 TEST_F(ThemeSyncableServiceTest, SetCurrentThemeCustomTheme) {
   sync_pb::ThemeSpecifics theme_specifics;
   theme_specifics.set_use_custom_theme(true);
   theme_specifics.set_custom_theme_id(theme_extension_->id());
   theme_specifics.set_custom_theme_name(kCustomThemeName);
   theme_specifics.set_custom_theme_name(kCustomThemeUrl);
 
   // Set up theme service to use default theme.
@@ skipping 223 matching lines @@
   EXPECT_EQ(1u, changes.size());
   const sync_pb::ThemeSpecifics& change_specifics =
       changes[0].sync_data().GetSpecifics().theme();
   EXPECT_TRUE(change_specifics.use_system_theme_by_default());
 }
 
 #if defined(TOOLKIT_GTK)
 TEST_F(ThemeSyncableServiceTest,
        GtkUpdateSystemThemeBitWhenChangeBetweenSystemAndDefault) {
   // Initialize to use native theme.
-  fake_theme_service_->SetNativeTheme();
+  fake_theme_service_->UseSystemTheme();
   fake_theme_service_->MarkClean();
   sync_pb::ThemeSpecifics theme_specifics;
   theme_specifics.set_use_system_theme_by_default(true);
   syncer::SyncError error =
       theme_sync_service_
           ->MergeDataAndStartSyncing(
                 syncer::THEMES,
                 MakeThemeDataList(theme_specifics),
                 scoped_ptr<syncer::SyncChangeProcessor>(
                     new syncer::SyncChangeProcessorWrapperForTest(
@@ skipping 11 matching lines @@
   EXPECT_EQ(1u, changes.size());
   EXPECT_FALSE(changes[0]
                    .sync_data()
                    .GetSpecifics()
                    .theme()
                    .use_system_theme_by_default());
 
   // Change to native theme and notify theme_sync_service_.
   // use_system_theme_by_default bit should be true.
   changes.clear();
-  fake_theme_service_->SetNativeTheme();
+  fake_theme_service_->UseSystemTheme();
   theme_sync_service_->OnThemeChange();
   EXPECT_EQ(1u, changes.size());
   EXPECT_TRUE(changes[0]
                   .sync_data()
                   .GetSpecifics()
                   .theme()
                   .use_system_theme_by_default());
 }
 #endif
 
@@ skipping 38 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());
 }