Commit 735586ec authored by battre@chromium.org's avatar battre@chromium.org

Fixed ExtraInfoSpec flags for webRequest.onAuthRequired


BUG=no
TEST=no


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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102182 0039d316-1c4b-4281-b951-d872f2087c98
parent cae2c540
...@@ -620,7 +620,7 @@ void ExtensionWebRequestEventRouter::OnAuthRequired( ...@@ -620,7 +620,7 @@ void ExtensionWebRequestEventRouter::OnAuthRequired(
challenger->SetInteger(keys::kPortKey, auth_info.challenger.port()); challenger->SetInteger(keys::kPortKey, auth_info.challenger.port());
dict->Set(keys::kChallengerKey, challenger); dict->Set(keys::kChallengerKey, challenger);
dict->SetDouble(keys::kTimeStampKey, time.ToDoubleT() * 1000); dict->SetDouble(keys::kTimeStampKey, time.ToDoubleT() * 1000);
if (extra_info_spec & ExtraInfoSpec::REQUEST_HEADERS) { if (extra_info_spec & ExtraInfoSpec::RESPONSE_HEADERS) {
dict->Set(keys::kResponseHeadersKey, dict->Set(keys::kResponseHeadersKey,
GetResponseHeadersList(request->response_headers())); GetResponseHeadersList(request->response_headers()));
} }
......
...@@ -227,7 +227,8 @@ runTests([ ...@@ -227,7 +227,8 @@ runTests([
isProxy: false, isProxy: false,
scheme: "basic", scheme: "basic",
realm: "testrealm", realm: "testrealm",
challenger: {host: testServer, port: testServerPort} challenger: {host: testServer, port: testServerPort},
responseHeadersExist: true
} }
}, },
{ label: "onResponseStarted", { label: "onResponseStarted",
...@@ -236,7 +237,8 @@ runTests([ ...@@ -236,7 +237,8 @@ runTests([
url: getURLAuthRequired(), url: getURLAuthRequired(),
fromCache: false, fromCache: false,
statusCode: 401, statusCode: 401,
ip: "127.0.0.1" ip: "127.0.0.1",
responseHeadersExist: true
} }
}, },
{ label: "onCompleted", { label: "onCompleted",
...@@ -245,7 +247,8 @@ runTests([ ...@@ -245,7 +247,8 @@ runTests([
url: getURLAuthRequired(), url: getURLAuthRequired(),
fromCache: false, fromCache: false,
statusCode: 401, statusCode: 401,
ip: "127.0.0.1" ip: "127.0.0.1",
responseHeadersExist: true
} }
}, },
], ],
...@@ -253,7 +256,7 @@ runTests([ ...@@ -253,7 +256,7 @@ runTests([
["onBeforeRequest", "onBeforeSendHeaders", "onSendHeaders", ["onBeforeRequest", "onBeforeSendHeaders", "onSendHeaders",
"onAuthRequired", "onResponseStarted", "onCompleted"] "onAuthRequired", "onResponseStarted", "onCompleted"]
], ],
{}, []); {}, ["responseHeaders"]);
navigateAndWait(getURLAuthRequired()); navigateAndWait(getURLAuthRequired());
}, },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment