Extracting NetLog inner classes into their own classes.

net/log/net_log_parameters_callback_typedef.h

Index: net/log/net_log_parameters_callback_typedef.h
diff --git a/net/log/net_log_parameters_callback_typedef.h b/net/log/net_log_parameters_callback_typedef.h
new file mode 100644
index 0000000000000000000000000000000000000000..07353277b79f86059ce3d8aa36e00274aabd2e31
--- /dev/null
+++ b/net/log/net_log_parameters_callback_typedef.h
@@ -0,0 +1,28 @@
+// Copyright 2016 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_PARAMETERS_CALLBACK_TYPEDEF_H_

[eroman, 2016/10/03 21:40:50]

nit on naming: I suggest dropping "_typedef" from the filename.
"typedef" reflects the implementation, which could just as easily be "using"
nowadays.

Feel free to leave as is and we can always discuss this change as a follow-up.

[mikecirone, 2016/10/03 23:38:25]

"net_log_parameters_callback_typedef.h" is now "net_log_parameters_callback.h"
and codebase refactored to this filename change. Change is in patch 8!

+#define NET_LOG_NET_LOG_PARAMETERS_CALLBACK_TYPEDEF_H_
+
+#include <memory>
+
+#include "base/callback_forward.h"
+#include "net/log/net_log_capture_mode.h"
+
+namespace base {
+class Value;
+}
+
+namespace net {
+
+// A callback that returns a Value representation of the parameters
+// associated with an event.  If called, it will be called synchronously,
+// so it need not have owning references.  May be called more than once, or
+// not at all.  May return NULL.

[eroman, 2016/10/03 21:40:50]

optional: Can you change NULL --> nullptr while editing this?

[mikecirone, 2016/10/03 23:38:25]

Done.

+typedef base::Callback<std::unique_ptr<base::Value>(NetLogCaptureMode)>
+    NetLogParametersCallback;
+
+}  // namespace net
+
+#endif  // NET_LOG_NET_LOG_PARAMETERS_CALLBACK_TYPEDEF_H_