[Android] Use resource throttle instead of HandleExternalProtocol.

content/browser/renderer_host/resource_dispatcher_host_impl.cc

Index: content/browser/renderer_host/resource_dispatcher_host_impl.cc
diff --git a/content/browser/renderer_host/resource_dispatcher_host_impl.cc b/content/browser/renderer_host/resource_dispatcher_host_impl.cc
index 60a3c9f89b0fa983d0011475fc1776a8f521ef91..f050138eb8224d4c4d7528b5315dbb83083b3ccf 100644
--- a/content/browser/renderer_host/resource_dispatcher_host_impl.cc
+++ b/content/browser/renderer_host/resource_dispatcher_host_impl.cc
@@ -659,6 +659,10 @@ bool ResourceDispatcherHostImpl::AcceptSSLClientCertificateRequest(
 
 bool ResourceDispatcherHostImpl::HandleExternalProtocol(ResourceLoader* loader,
                                                         const GURL& url) {
+#if defined OS(ANDROID)
+  return false;

[joth, 2012/10/24 18:47:55]

Did you mean this? you've changed delegate_->HandleExternalProtocol to return
bool instead -- shouldn't we just be returning that result below, no need for
ifdef here?

[mkosiba (inactive), 2012/10/25 09:02:12]

Do what Joth says ;)

[John Knottenbelt, 2012/10/25 16:17:27]

Yes, I agree. Sorry, this was accidentally left in from an experiment. 

On 2012/10/24 18:47:55, joth wrote:
ad -- shouldn't we just be returning that result below, no need for

+#endif
+
   if (!delegate_)
     return false;
 
@@ -674,6 +678,7 @@ bool ResourceDispatcherHostImpl::HandleExternalProtocol(ResourceLoader* loader,
 
   delegate_->HandleExternalProtocol(url, info->GetChildID(),
                                     info->GetRouteID());
+
   return true;
 }