[OBSOLETE] Reporting: Initial implementation.

net/reporting/reporting_controls.h

Index: net/reporting/reporting_controls.h
diff --git a/net/reporting/reporting_controls.h b/net/reporting/reporting_controls.h
new file mode 100644
index 0000000000000000000000000000000000000000..c1c7b44d1514b78e5e280a010849c62e3343dd00
--- /dev/null
+++ b/net/reporting/reporting_controls.h
@@ -0,0 +1,20 @@
+// 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_REPORTING_REPORTING_CONTROLS_H_
+#define NET_REPORTING_REPORTING_CONTROLS_H_
+
+#include "net/base/net_export.h"
+
+namespace net {
+
+// Returns whether Reporting should be enabled, based on command-line flags
+// (passed in, since they're in chrome/common), field trials, and a hard-coded
+// default.

[Ryan Sleevi, 2017/01/03 21:28:12]

1) //net should never mention layers above it (in comments or code), because
that's a layering violation. For example, it's impossible to evaluate what this
code would do in the case of Chrome for iOS or in the case of Cronet - both of
which have //net part of them.

2) From this header, it's not clear why a simple use of base::Feature wasn't
used here?

[Julia Tuttle, 2017/01/25 20:27:45]

Obsolete; I'm now using base::Feature.

+NET_EXPORT bool GetReportingEnabled(const char* disable_switch_name,
+                                    const char* enable_switch_name);
+
+}  // namespace net
+
+#endif  // NET_REPORTING_REPORTING_CONTROLS_H_