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

[digit1, 2012/10/19 10:32:46]

As philippe said, Android doesn't use Glibc at all, so these comments and
variable name changes are misleading. Can you fix this? Use "main thread" or
keep "jni thread" since this is under net/. Thanks.

+    // 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_