We should only handle http and https in the URL job created to enable automation clients to

handle URL requests.

This fixes bug http://b/issue?id=2068725

Bug=2068725

Review URL: http://codereview.chromium.org/174197

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23925 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 11 insertions, 9 deletions

diff --git a/chrome/browser/automation/url_request_automation_job.cc b/chrome/browser/automation/url_request_automation_job.cc
index ee75f15..86f7ebb 100644
--- a/chrome/browser/automation/url_request_automation_job.cc
+++ b/chrome/browser/automation/url_request_automation_job.cc
@@ -63,15 +63,17 @@ class AutomationRequestInterceptor : public URLRequest::Interceptor {
 
 URLRequestJob* AutomationRequestInterceptor::MaybeIntercept(
     URLRequest* request) {
-  ResourceDispatcherHost::ExtraRequestInfo* request_info =
-      ResourceDispatcherHost::ExtraInfoForRequest(request);
-  if (request_info) {
-    AutomationResourceMessageFilter::AutomationDetails details;
-    if (AutomationResourceMessageFilter::LookupRegisteredRenderView(
-            request_info->process_id, request_info->route_id, &details)) {
-      URLRequestAutomationJob* job = new URLRequestAutomationJob(request,
-          details.tab_handle, details.filter);
-      return job;
+  if (request->url().SchemeIs("http") || request->url().SchemeIs("https")) {
+    ResourceDispatcherHost::ExtraRequestInfo* request_info =
+        ResourceDispatcherHost::ExtraInfoForRequest(request);
+    if (request_info) {
+      AutomationResourceMessageFilter::AutomationDetails details;
+      if (AutomationResourceMessageFilter::LookupRegisteredRenderView(
+              request_info->process_id, request_info->route_id, &details)) {
+        URLRequestAutomationJob* job = new URLRequestAutomationJob(request,
+            details.tab_handle, details.filter);
+        return job;
+      }
     }
   }