Minor refactorings:

- Change scheme from a (const char*) to a (std::string). This is in response to an earlier comment.
Background:
It used to be a (const char*) to avoid allocating a string object since all the callers initialize it with a string literal.

- Change the initial value of nonce_count_ from 1 to 0. This doesn't functionally change anything (we just change to a pre-increment rather than post-increment), but I prefer the the start-at-zero style.

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

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

1 files changed, 1 insertions, 1 deletions

diff --git a/net/http/http_auth_handler_digest.h b/net/http/http_auth_handler_digest.h
index 85d3f3b..26d67f8 100644
--- a/net/http/http_auth_handler_digest.h
+++ b/net/http/http_auth_handler_digest.h
@@ -23,7 +23,7 @@ class HttpAuthHandlerDigest : public HttpAuthHandler {
  protected:
   virtual bool Init(std::string::const_iterator challenge_begin,
                     std::string::const_iterator challenge_end) {
-    nonce_count_ = 1;
+    nonce_count_ = 0;
     return ParseChallenge(challenge_begin, challenge_end);
   }