Refactor NetLog::LogLevel --> NetLogCaptureMode.

net/log/net_log_capture_mode.h

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NET_LOG_NET_LOG_CAPTURE_MODE_H_
+#define NET_LOG_NET_LOG_CAPTURE_MODE_H_
+
+#include <string>
+
+#include "build/build_config.h"
+

[mmenke, 2015/04/20 15:54:38]

Remove blank line and alphabetize.  Actually, why do we need build_config?

[eroman, 2015/04/20 21:07:20]

Removed all these headers except for net_export.h.

Was a consequence of having duplicated an existing file as header file template,
and then failing to properly trim everything.

+#include "base/basictypes.h"
+#include "net/base/net_export.h"
+
+namespace net {
+
+// NetLogCaptureMode specifies the granularity of events that should be emitted
+// to the log. It is a simple wrapper around an integer, so it should be passed
+// to functions by value rather than by reference.
+class NET_EXPORT NetLogCaptureMode {
+ public:
+  // NOTE: Default assignment and copy constructor are OK

[mmenke, 2015/04/20 15:54:39]

+ period

[eroman, 2015/04/20 21:07:19]

Done.

+
+  // The default constructor creates an empty capture mode (equivalent to
+  // None()).
+  NetLogCaptureMode();
+
+  // If |enabled()| is true, then _something_ is being captured.
+  bool enabled() const;
+
+  // If |include_private_data()| is true , then it is OK to log events which
+  // contain cookies, credentials or other privacy sensitive data.
+  bool include_private_data() const;
+
+  // If |include_socket_bytes()| is true, then it is OK to output the actual
+  // bytes read/written from the network, even if it contains private data.
+  bool include_socket_bytes() const;
+
+  // Returns a capture mode that contains the maximal set of capabilities
+  // between |mode1| and |mode2|.
+  static NetLogCaptureMode Max(NetLogCaptureMode mode1,
+                               NetLogCaptureMode mode2);
+
+  // Constructs a capture mode which logs NOTHING.
+  //    enabled() --> false
+  //    include_private_data() --> false
+  //    include_socket_bytes() --> false
+  static NetLogCaptureMode None();
+
+  // Constructs a capture mode which logs basic events and event parameters.
+  //    enabled() --> true
+  //    include_private_data() --> false
+  //    include_socket_bytes() --> false
+  static NetLogCaptureMode Default();
+
+  // Constructs a capture mode which logs basic events, and additionally makes
+  // no effort to strip cookies and credentials.
+  //    enabled() --> true
+  //    include_private_data() --> true
+  //    include_socket_bytes() --> false
+  static NetLogCaptureMode IncludeCookiesAndCredentials();
+
+  // Constructs a capture mode which logs absolutely everything possible.
+  //    enabled() --> true
+  //    include_private_data() --> true
+  //    include_socket_bytes() --> true
+  static NetLogCaptureMode All();

[mmenke, 2015/04/20 15:54:38]

I suggest putting all these static methods just below the default constructor,
as they're basically constructors (And making Max the final one, as it's less
constructor-like).

[eroman, 2015/04/20 21:07:19]

Done

+
+  bool operator==(NetLogCaptureMode mode) const;
+
+  int32_t ToInternalValueForTesting() const { return ToInternalValue(); }
+
+ private:
+  // NetLog relies on the internal value of NetLogCaptureMode being an integer,
+  // so it can be read/written atomically across thread.
+  friend class NetLog;
+
+  explicit NetLogCaptureMode(uint32_t value);
+
+  static NetLogCaptureMode FromInternalValue(int32_t value) {
+    return NetLogCaptureMode(value);
+  }
+
+  int32_t ToInternalValue() const { return value_; }

[mmenke, 2015/04/20 15:54:38]

Hrm...These two probably shouldn't be inlined, given their names.  New style
guide isn't as clear on that as the old one, though...

[eroman, 2015/04/20 21:07:19]

Done.

+
+  int32_t value_;
+};
+
+}  // namespace net
+
+#endif  // NET_LOG_NET_LOG_CAPTURE_MODE_H_