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

Unified Diff: net/base/capturing_net_log.h

Issue 4067002: First pass at adding http/backend cache to NetLog (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Minor cleanup Created 10 years 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/base/capturing_net_log.h
===================================================================
--- net/base/capturing_net_log.h (revision 68857)
+++ net/base/capturing_net_log.h (working copy)
@@ -61,6 +61,10 @@
void Clear();
+ // This function is not thread safe, so must only be called when the
eroman 2011/01/05 02:39:06 I suggest making this threadsafe -- simply read-wr
mmenke 2011/01/05 16:32:25 Done, and renamed to SetLogLevel.
+ // CapturingNetLog is not being actively used.
+ void set_log_level(NetLog::LogLevel log_level);
+
private:
// Needs to be "mutable" so can use it in GetEntries().
mutable Lock lock_;
@@ -71,6 +75,8 @@
size_t max_num_entries_;
EntryList entries_;
+ NetLog::LogLevel log_level_;
+
DISALLOW_COPY_AND_ASSIGN(CapturingNetLog);
};
@@ -97,6 +103,11 @@
void Clear();
+ // This function is not thread safe, so must only be called when the
eroman 2011/01/05 02:39:06 Ditto to comment above. (Also note that if you do
mmenke 2011/01/05 16:32:25 Done.
+ // CapturingBoundNetLog and underlying CapturingNetLog are not actively being
+ // used.
+ void set_log_level(NetLog::LogLevel log_level);
+
private:
NetLog::Source source_;
scoped_ptr<CapturingNetLog> capturing_net_log_;

Powered by Google App Engine
This is Rietveld 408576698