[Android] Fix handling of asset files loading.

chrome/browser/android/android_protocol_adapter.h

Index: chrome/browser/android/android_protocol_adapter.h
diff --git a/chrome/browser/android/android_protocol_adapter.h b/chrome/browser/android/android_protocol_adapter.h
index 3cf11e801fdb5d48da38246fcc553d7b6b658bcd..15aeab135210364a4af65e5f3b26930c7b2e2446 100644
--- a/chrome/browser/android/android_protocol_adapter.h
+++ b/chrome/browser/android/android_protocol_adapter.h
@@ -9,13 +9,30 @@
 
 #include "net/url_request/url_request.h"
 
+namespace net {
+
+class URLRequestContextGetter;
+
+}
+
+// This class adds support for Android WebView-specific protocol schemes:
+//
+//  - "content" scheme is used for accessing data from Android content

[joth, 2012/08/24 17:28:28]

Thanks, great docs.


nit: "content:" scheme.

(aids grepability)

[mnaganov (inactive), 2012/08/24 18:17:22]

OK. But, technically, the colon isn't a part of the scheme name.

+//    providers, see http://developer.android.com/guide/topics/providers/
+//      content-provider-basics.html#ContentURIs
+//
+//  - "file" scheme extension for accessing application assets and resources,

[joth, 2012/08/24 17:28:28]

ditto, android_assets and android_resources. (right?)

[mnaganov (inactive), 2012/08/24 18:17:22]

Done.

+//    see http://developer.android.com/reference/android/webkit/
+//      WebSettings.html#setAllowFileAccess(boolean)
+//
 class AndroidProtocolAdapter {
  public:
   static net::URLRequest::ProtocolFactory Factory;
 
   // Register the protocol factories for all supported Android protocol
   // schemes.
-  static bool RegisterProtocols(JNIEnv* env);
+  static void RegisterProtocols(
+      JNIEnv* env, net::URLRequestContextGetter* context);
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(AndroidProtocolAdapter);