Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(730)

Unified Diff: net/http/http_cache.h

Issue 2886483002: Adds a new class HttpCache::Writers for multiple cache transactions reading from the network. (Closed)
Patch Set: Fix test class memory leak Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/http/http_cache.h
diff --git a/net/http/http_cache.h b/net/http/http_cache.h
index e420c7244204874988b4908149bf51a2bd8697a3..6b148f156d4a56c385c210f3cd65a5d1e4b2ced5 100644
--- a/net/http/http_cache.h
+++ b/net/http/http_cache.h
@@ -68,6 +68,8 @@ class NET_EXPORT HttpCache : public HttpTransactionFactory {
DISABLE
};
+ // class WritersTest;
jkarlin 2017/06/28 15:50:08 What's this?
shivanisha 2017/06/28 19:38:19 Removed in a subsequent patch.
+
// A BackendFactory creates a backend object to be used by the HttpCache.
class NET_EXPORT BackendFactory {
public:
@@ -246,6 +248,9 @@ class NET_EXPORT HttpCache : public HttpTransactionFactory {
class QuicServerInfoFactoryAdaptor;
class Transaction;
class WorkItem;
+ class Writers;
+ friend class WritersTest; // To access ActiveEntry in the test class.
+ friend class WritersTest1; // To access ActiveEntry in the test class.
jkarlin 2017/06/28 15:50:08 What's this?
shivanisha 2017/06/28 19:38:19 Removed in a subsequent patch.
friend class Transaction;
friend class ViewCacheHelper;
struct PendingOp; // Info for an entry under construction.
« no previous file with comments | « net/BUILD.gn ('k') | net/http/http_cache_transaction.h » ('j') | net/http/http_cache_writers.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698