diff options
| author | cbentzel@chromium.org <cbentzel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-20 16:52:15 +0000 |
|---|---|---|
| committer | cbentzel@chromium.org <cbentzel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-20 16:52:15 +0000 |
| commit | d66f881f539b0ed7a1c443c783d48973e271c43a (patch) | |
| tree | 79230c3b1deec0695ee6d50fbdae67c16beaf8bd /net/http/http_auth.cc | |
| parent | 72df2c8d3c0356db16cc37a22c66dd5dfc56ab83 (diff) | |
| download | chromium_src-d66f881f539b0ed7a1c443c783d48973e271c43a.zip chromium_src-d66f881f539b0ed7a1c443c783d48973e271c43a.tar.gz chromium_src-d66f881f539b0ed7a1c443c783d48973e271c43a.tar.bz2 | |
Add more tests cases for HttpAuth::HandleChallengeResponse.
This also improves the documentation for HandleChallengeResponse and clears the challenge_response arugment.
BUG=None
TEST=net_unittests --gtest_filter="*HandleChallengeResponse*"
Review URL: http://codereview.chromium.org/3752003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63227 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_auth.cc')
| -rw-r--r-- | net/http/http_auth.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/http/http_auth.cc b/net/http/http_auth.cc index ddd099c..e7002cf 100644 --- a/net/http/http_auth.cc +++ b/net/http/http_auth.cc @@ -60,7 +60,10 @@ HttpAuth::AuthorizationResult HttpAuth::HandleChallengeResponse( Target target, const std::set<std::string>& disabled_schemes, std::string* challenge_used) { + DCHECK(handler); + DCHECK(headers); DCHECK(challenge_used); + challenge_used->clear(); const std::string& current_scheme = handler->scheme(); if (disabled_schemes.find(current_scheme) != disabled_schemes.end()) return HttpAuth::AUTHORIZATION_RESULT_REJECT; |