Add trace event for complete network request

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

Index: third_party/WebKit/Source/platform/network/NetworkInstrumentation.h
diff --git a/third_party/WebKit/Source/platform/network/NetworkInstrumentation.h b/third_party/WebKit/Source/platform/network/NetworkInstrumentation.h
new file mode 100644
index 0000000000000000000000000000000000000000..50a70bbf5b460ba1eabacd8d49b19190eb958770
--- /dev/null
+++ b/third_party/WebKit/Source/platform/network/NetworkInstrumentation.h
@@ -0,0 +1,42 @@
+// 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 NetworkInstrumentation_h
+#define NetworkInstrumentation_h
+
+#include "platform/PlatformExport.h"
+#include "platform/network/ResourceLoadPriority.h"
+#include "platform/network/ResourceRequest.h"

[caseq, 2016/11/08 18:24:32]

please forward declare  here instead.

[dproy, 2016/11/09 20:44:00]

Done.

+
+namespace network_instrumentation {
+
+enum RequestOutcome { Success, Fail };
+
+class PLATFORM_EXPORT ScopedResourceLoadTracker {
+ public:
+  ScopedResourceLoadTracker(unsigned long resourceID,
+                            const blink::ResourceRequest&);
+  ~ScopedResourceLoadTracker();
+  void doNotCloseSliceAtEndOfScope();
+
+ private:
+  bool closeSliceAtEndOfScope;

[caseq, 2016/11/08 18:24:32]

use m_ prefix. Also, how about a more specific name, e.g.
m_resourceLoadingCancelled?

[dproy, 2016/11/09 20:44:00]

Done. I changed it to m_resourceLoadContinuesBeyondScope, because otherwise
m_resourceLoadingCancelled = true by default and that was confusing.

+  const unsigned long resourceID;

[caseq, 2016/11/08 18:24:32]

m_resourceId

[dproy, 2016/11/09 20:44:00]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(ScopedResourceLoadTracker);
+};
+
+void PLATFORM_EXPORT initialResourcePrioritySet(unsigned long resourceID,
+                                                blink::ResourceLoadPriority);
+
+void PLATFORM_EXPORT resourcePriorityChanged(unsigned long resourceID,
+                                             blink::ResourceLoadPriority);
+
+void PLATFORM_EXPORT endResourceLoad(unsigned long resourceID, RequestOutcome);
+
+const char* RequestOutcomeToString(RequestOutcome);

[caseq, 2016/11/08 18:24:32]

do we need to expose this one?

[dproy, 2016/11/09 20:44:00]

Nope - thanks for pointing this out.

+
+}  // namespace network_instrumentation
+
+#endif  // NetworkInstrumentation_h