Clear HTTP auth data on clearing browsing data

net/http/http_auth_cache.h

Index: net/http/http_auth_cache.h
diff --git a/net/http/http_auth_cache.h b/net/http/http_auth_cache.h
index 9190e42c50899951eac307e3afa674cdb7175878..3f9afaf01a2ffb4fd3d4b87eea71c99da2b73c50 100644
--- a/net/http/http_auth_cache.h
+++ b/net/http/http_auth_cache.h
@@ -27,9 +27,9 @@ namespace net {
 //   - the last auth handler used (contains realm and authentication scheme)
 //   - the list of paths which used this realm
 // Entries can be looked up by either (origin, realm, scheme) or (origin, path).
-class NET_EXPORT_PRIVATE HttpAuthCache {
+class NET_EXPORT HttpAuthCache {
  public:
-  class NET_EXPORT_PRIVATE Entry {
+  class NET_EXPORT Entry {
    public:
     Entry(const Entry& other);
     ~Entry();
@@ -64,6 +64,7 @@ class NET_EXPORT_PRIVATE HttpAuthCache {
     friend class HttpAuthCache;
     FRIEND_TEST_ALL_PREFIXES(HttpAuthCacheTest, AddPath);
     FRIEND_TEST_ALL_PREFIXES(HttpAuthCacheTest, AddToExistingEntry);
+    FRIEND_TEST_ALL_PREFIXES(HttpAuthCacheTest, ClearEntriesAddedWithin);

[mmenke, 2016/08/31 14:48:32]

Suggest a public method set_creation_time_for_testing instead.  We have
presubmit checks about for_testing methods called in tests, and not-friending
limits the API that tests can depend on to the public API + for_testing methods,
as opposed all the guts of the class.

[Tomasz Moniuszko, 2016/09/01 08:40:00]

Done.

 
     typedef std::list<std::string> PathList;
 
@@ -162,8 +163,8 @@ class NET_EXPORT_PRIVATE HttpAuthCache {
               HttpAuth::Scheme scheme,
               const AuthCredentials& credentials);
 
-  // Clears the cache.
-  void Clear();
+  // Clears cache entries created within |duration| of base::TimeTicks::Now().
+  void ClearEntriesAddedWithin(base::TimeDelta duration);
 
   // Updates a stale digest entry on server |origin| for realm |realm| and
   // scheme |scheme|. The cached auth challenge is replaced with