Add trace event for complete network request

third_party/WebKit/Source/platform/tracing/NetworkInstrumentation.h

Index: third_party/WebKit/Source/platform/tracing/NetworkInstrumentation.h
diff --git a/third_party/WebKit/Source/platform/tracing/NetworkInstrumentation.h b/third_party/WebKit/Source/platform/tracing/NetworkInstrumentation.h
new file mode 100644
index 0000000000000000000000000000000000000000..affade18ffdd7a500df7e76601678aafb0d10758
--- /dev/null
+++ b/third_party/WebKit/Source/platform/tracing/NetworkInstrumentation.h
@@ -0,0 +1,21 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.

[caseq, 2016/10/24 22:47:46]

This rather needs to be somewhere under platorm/network -- tracing isn't
supposed to "know" anything about networking.

[dproy, 2016/11/03 15:13:37]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NetworkInstrumentation_h
+#define NetworkInstrumentation_h
+
+#include "platform/PlatformExport.h"
+
+namespace network_instrumentation {
+
+const char kNetInstrumentationCategory[] = "NetworkInstrumentation";

[chiniforooshan, 2016/10/24 19:25:37]

Lower case category names seem to be more popular. How about something like
"net.v2"?

[chiniforooshan, 2016/10/24 19:25:37]

What do you think about TRACE_DISABLED_BY_DEFAULT(...) for now, to not pollute
TV while we are in the work-in-progress phase, and maybe enable it by default
later when net instrumentation is in a good shape?

[dproy, 2016/11/03 15:13:37]

Changed the cat name. Disabled by default sounds good.

+
+void PLATFORM_EXPORT beginResourceLoad(unsigned long resourceID,
+                                       const char* url);
+
+void PLATFORM_EXPORT endResourceLoad(unsigned long resourceID);
+
+}  // namespace network_instrumentati
+
+#endif  // NetworkInstrumentation_h