summaryrefslogtreecommitdiffstats
path: root/chrome
diff options
context:
space:
mode:
authorananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-21 00:26:53 +0000
committerananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-21 00:26:53 +0000
commitda6970fe3c1c5901070f02379123fa5f5d2a3c11 (patch)
tree2d5d958b3460b5ee1c59a00ab3e5d6798bdde0a1 /chrome
parent26be478afc0752bbaa8381a6cf08d76384c46bdf (diff)
downloadchromium_src-da6970fe3c1c5901070f02379123fa5f5d2a3c11.zip
chromium_src-da6970fe3c1c5901070f02379123fa5f5d2a3c11.tar.gz
chromium_src-da6970fe3c1c5901070f02379123fa5f5d2a3c11.tar.bz2
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
Diffstat (limited to 'chrome')
-rw-r--r--chrome/browser/automation/url_request_automation_job.cc20
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;
+ }
}
}