From dde45d1634f0e3a3ae62259ca6d6647fc86dfa61 Mon Sep 17 00:00:00 2001 From: "asanka@chromium.org" Date: Fri, 4 Mar 2011 13:27:44 +0000 Subject: Mention that HttpAuthCache::Entry::HasEnclosingPath() can return 0 as a path len and fix typo. (Comment only change. Cleanup for 76539) BUG=none TEST=none Review URL: http://codereview.chromium.org/6612026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76911 0039d316-1c4b-4281-b951-d872f2087c98 --- net/http/http_auth_cache.h | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'net/http/http_auth_cache.h') diff --git a/net/http/http_auth_cache.h b/net/http/http_auth_cache.h index 2895401..965cff7 100644 --- a/net/http/http_auth_cache.h +++ b/net/http/http_auth_cache.h @@ -163,7 +163,11 @@ class HttpAuthCache::Entry { // Returns true if |dir| is contained within the realm's protection // space. |*path_len| is set to the length of the enclosing path if // such a path exists and |path_len| is non-NULL. If no enclosing - // path is found, |path_len| is left unmodified. + // path is found, |*path_len| is left unmodified. + // + // Note that proxy auth cache entries are associated with empty + // paths. Therefore it is possible for HasEnclosingPath() to return + // true and set |*path_len| to 0. bool HasEnclosingPath(const std::string& dir, size_t* path_len); // |origin_| contains the {protocol, host, port} of the server. -- cgit v1.1