Improve some naming

chrome/browser/themes/theme_service_aurax11.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_THEMES_THEME_SERVICE_AURAX11_H_
 #define CHROME_BROWSER_THEMES_THEME_SERVICE_AURAX11_H_
 
 #include "chrome/browser/themes/theme_service.h"
-#include "ui/views/linux_ui/native_theme_change_observer.h"
+#include "ui/views/linux_ui/system_theme_change_observer.h"
 
 // A subclass of ThemeService that manages the CustomThemeSupplier which
 // provides the native X11 theme.
 class ThemeServiceAuraX11 : public ThemeService,
-                            public views::NativeThemeChangeObserver {
+                            public views::SystemThemeChangeObserver {

[Elliot Glaysher, 2014/04/21 23:31:47]

I think you're overshooting here, though this might be a collision with msw@'s
patch; NativeThemes can change for reasons other than being the system theme.

[Evan Stade, 2014/04/22 00:28:22]

Note that there's another method with the same name, View::OnNativeThemeChanged,
which is NOT an implementation of this interface method. (This CL hopes to make
that less confusing by giving the methods different names.)

[msw, 2014/04/22 18:02:13]

Hopefully ThemeServiceAuraX11 won't need to observe NativeTheme objects at all
after <https://codereview.chromium.org/239093007>. Let's coordinate.

  public:
   ThemeServiceAuraX11();
   virtual ~ThemeServiceAuraX11();
 
   // Overridden from ThemeService:
-  virtual bool ShouldInitWithNativeTheme() const OVERRIDE;
-  virtual void SetNativeTheme() OVERRIDE;
+  virtual bool ShouldInitWithSystemTheme() const OVERRIDE;
+  virtual void UseSystemTheme() OVERRIDE;
   virtual bool UsingDefaultTheme() const OVERRIDE;
-  virtual bool UsingNativeTheme() const OVERRIDE;
+  virtual bool UsingSystemTheme() const OVERRIDE;
 
-  // Overridden from views::NativeThemeChangeObserver:
-  virtual void OnNativeThemeChanged() OVERRIDE;
+  // Overridden from views::SystemThemeChangeObserver:
+  virtual void OnSystemThemeChanged() OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ThemeServiceAuraX11);
 };
 
 #endif  // CHROME_BROWSER_THEMES_THEME_SERVICE_AURAX11_H_