Move ProxyConfigService construction onto the IO thread.

net/proxy/proxy_config_service_android.h

 // 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.
 
 #ifndef NET_PROXY_PROXY_CONFIG_SERVICE_ANDROID_H_
 #define NET_PROXY_PROXY_CONFIG_SERVICE_ANDROID_H_
 
 #include <string>
 
 #include "base/android/jni_android.h"
@@ skipping 10 matching lines @@
 
 namespace net {
 
 class ProxyConfig;
 
 class NET_EXPORT ProxyConfigServiceAndroid : public ProxyConfigService {
  public:
   // Callback that returns the value of the property identified by the provided
   // key. If it was not found, an empty string is returned. Note that this
   // interface does not let you distinguish an empty property from a
-  // non-existing property. This callback is invoked on the JNI thread.
+  // non-existing property. This callback is invoked on the network thread.
   typedef base::Callback<std::string (const std::string& property)>
       GetPropertyCallback;
 
   // Separate class whose instance is owned by the Delegate class implemented in
   // the .cc file.
   class JNIDelegate {
    public:
     virtual ~JNIDelegate() {}
 
-    // Called from Java (on JNI thread) to signal that the proxy settings have
-    // changed.
+    // Called from Java (on glib thread) to signal that the proxy settings

[Philippe, 2012/10/19 09:35:25]

Maybe you should keep JNI here and in the cc file. What do you think? We don't
use glib on Android.

+    // have changed.
     virtual void ProxySettingsChanged(JNIEnv*, jobject) = 0;
   };
 
-  ProxyConfigServiceAndroid(base::SequencedTaskRunner* network_task_runner,
-                            base::SequencedTaskRunner* jni_task_runner);
+  ProxyConfigServiceAndroid(base::SequencedTaskRunner* glib_task_runner,
+                            base::SequencedTaskRunner* network_task_runner);
 
   virtual ~ProxyConfigServiceAndroid();
 
   // Register JNI bindings.
   static bool Register(JNIEnv* env);
 
   // ProxyConfigService:
   // Called only on the network thread.
   virtual void AddObserver(Observer* observer) OVERRIDE;
   virtual void RemoveObserver(Observer* observer) OVERRIDE;
   virtual ConfigAvailability GetLatestProxyConfig(ProxyConfig* config) OVERRIDE;
+  // Called from glib thread.
+  virtual void StartTearDown() OVERRIDE;
 
  private:
   friend class ProxyConfigServiceAndroidTestBase;
   class Delegate;
 
 
   // For tests.
   ProxyConfigServiceAndroid(base::SequencedTaskRunner* network_task_runner,
-                            base::SequencedTaskRunner* jni_task_runner,
+                            base::SequencedTaskRunner* glib_task_runner,
                             GetPropertyCallback get_property_callback);
 
   // For tests.
   void ProxySettingsChanged();
 
   scoped_refptr<Delegate> delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(ProxyConfigServiceAndroid);
 };
 
 } // namespace net
 
 #endif // NET_PROXY_PROXY_CONFIG_SERVICE_ANDROID_H_