Change name of NativeTheme::instance() to make it clear it's web only.

ui/native_theme/native_theme.h

Index: ui/native_theme/native_theme.h
diff --git a/ui/native_theme/native_theme.h b/ui/native_theme/native_theme.h
index 714b4b09fba14a0dde29ad50a15a082283a5dbf4..15bc92197e7af97c1969a31e9a54259a5cf27c94 100644
--- a/ui/native_theme/native_theme.h
+++ b/ui/native_theme/native_theme.h
@@ -344,12 +344,13 @@ class NATIVE_THEME_EXPORT NativeTheme {
   // Return a color from the system theme.
   virtual SkColor GetSystemColor(ColorId color_id) const = 0;
 
-  // Returns a shared instance of the native theme.
-  // The returned object should not be deleted by the caller.  This function
-  // is not thread safe and should only be called from the UI thread.
-  // Each port of NativeTheme should provide its own implementation of this
-  // function, returning the port's subclass.
-  static NativeTheme* instance();
+  // Returns a shared instance of the native theme that should be used for web
+  // rendering. Do not use it in a normal application context (i.e. browser).
+  // The returned object should not be deleted by the caller. This function is
+  // not thread safe and should only be called from the UI thread. Each port of
+  // NativeTheme should provide its own implementation of this function,
+  // returning the port's subclass.
+  static NativeTheme* GetInstanceForWeb();

[sky, 2015/11/20 22:09:04]

Is it possible to remove this function entirely? Can't the blink side, or
wherever, figure out the right one to call?

[Evan Stade, 2015/11/20 22:23:45]

I would like to do that, but it's referenced from both content/ and components/
so I don't know of a better place to put this.

 
   // Add or remove observers to be notified when the native theme changes.
   void AddObserver(NativeThemeObserver* observer);