Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(488)

Side by Side Diff: third_party/WebKit/Source/platform/network/NetworkInstrumentation.cpp

Issue 2868923002: Blink rename follow-up: Use Chromium style in network_instrumentation namespace. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "platform/network/NetworkInstrumentation.h" 5 #include "platform/network/NetworkInstrumentation.h"
6 6
7 #include "base/trace_event/trace_event.h" 7 #include "base/trace_event/trace_event.h"
8 #include "platform/instrumentation/tracing/TracedValue.h" 8 #include "platform/instrumentation/tracing/TracedValue.h"
9 #include "platform/loader/fetch/ResourceLoadPriority.h" 9 #include "platform/loader/fetch/ResourceLoadPriority.h"
10 #include "platform/loader/fetch/ResourceRequest.h" 10 #include "platform/loader/fetch/ResourceRequest.h"
11 11
12 namespace network_instrumentation { 12 namespace network_instrumentation {
13 13
14 using network_instrumentation::RequestOutcome; 14 using network_instrumentation::RequestOutcome;
15 using blink::TracedValue; 15 using blink::TracedValue;
16 16
17 const char kBlinkResourceID[] = "BlinkResourceID"; 17 const char kBlinkResourceID[] = "BlinkResourceID";
18 const char kResourceLoadTitle[] = "ResourceLoad"; 18 const char kResourceLoadTitle[] = "ResourceLoad";
19 const char kResourcePrioritySetTitle[] = "ResourcePrioritySet"; 19 const char kResourcePrioritySetTitle[] = "ResourcePrioritySet";
20 const char kNetInstrumentationCategory[] = TRACE_DISABLED_BY_DEFAULT("network"); 20 const char kNetInstrumentationCategory[] = TRACE_DISABLED_BY_DEFAULT("network");
21 21
22 const char* RequestOutcomeToString(RequestOutcome outcome) { 22 const char* RequestOutcomeToString(RequestOutcome outcome) {
23 switch (outcome) { 23 switch (outcome) {
24 case RequestOutcome::Success: 24 case RequestOutcome::kSuccess:
25 return "Success"; 25 return "Success";
26 case RequestOutcome::Fail: 26 case RequestOutcome::kFail:
27 return "Fail"; 27 return "Fail";
28 default: 28 default:
29 NOTREACHED(); 29 NOTREACHED();
30 // We need to return something to avoid compiler warning. 30 // We need to return something to avoid compiler warning.
31 return "This should never happen"; 31 return "This should never happen";
32 } 32 }
33 } 33 }
34 34
35 // Note: network_instrumentation code should do as much work as possible inside 35 // Note: network_instrumentation code should do as much work as possible inside
36 // the arguments of trace macros so that very little instrumentation overhead is 36 // the arguments of trace macros so that very little instrumentation overhead is
37 // incurred if the trace category is disabled. See https://crbug.com/669666. 37 // incurred if the trace category is disabled. See https://crbug.com/669666.
38 38
39 namespace { 39 namespace {
40 40
41 std::unique_ptr<TracedValue> scopedResourceTrackerBeginData( 41 std::unique_ptr<TracedValue> ScopedResourceTrackerBeginData(
42 const blink::ResourceRequest& request) { 42 const blink::ResourceRequest& request) {
43 std::unique_ptr<TracedValue> data = TracedValue::Create(); 43 std::unique_ptr<TracedValue> data = TracedValue::Create();
44 data->SetString("url", request.Url().GetString()); 44 data->SetString("url", request.Url().GetString());
45 return data; 45 return data;
46 } 46 }
47 47
48 std::unique_ptr<TracedValue> resourcePrioritySetData( 48 std::unique_ptr<TracedValue> ResourcePrioritySetData(
49 blink::ResourceLoadPriority priority) { 49 blink::ResourceLoadPriority priority) {
50 std::unique_ptr<TracedValue> data = TracedValue::Create(); 50 std::unique_ptr<TracedValue> data = TracedValue::Create();
51 data->SetInteger("priority", priority); 51 data->SetInteger("priority", priority);
52 return data; 52 return data;
53 } 53 }
54 54
55 std::unique_ptr<TracedValue> endResourceLoadData(RequestOutcome outcome) { 55 std::unique_ptr<TracedValue> EndResourceLoadData(RequestOutcome outcome) {
56 std::unique_ptr<TracedValue> data = TracedValue::Create(); 56 std::unique_ptr<TracedValue> data = TracedValue::Create();
57 data->SetString("outcome", RequestOutcomeToString(outcome)); 57 data->SetString("outcome", RequestOutcomeToString(outcome));
58 return data; 58 return data;
59 } 59 }
60 60
61 } // namespace 61 } // namespace
62 62
63 ScopedResourceLoadTracker::ScopedResourceLoadTracker( 63 ScopedResourceLoadTracker::ScopedResourceLoadTracker(
64 unsigned long resourceID, 64 unsigned long resource_id,
65 const blink::ResourceRequest& request) 65 const blink::ResourceRequest& request)
66 : m_resourceLoadContinuesBeyondScope(false), m_resourceID(resourceID) { 66 : resource_load_continues_beyond_scope_(false), resource_id_(resource_id) {
67 TRACE_EVENT_NESTABLE_ASYNC_BEGIN1( 67 TRACE_EVENT_NESTABLE_ASYNC_BEGIN1(
68 kNetInstrumentationCategory, kResourceLoadTitle, 68 kNetInstrumentationCategory, kResourceLoadTitle,
69 TRACE_ID_WITH_SCOPE(kBlinkResourceID, TRACE_ID_LOCAL(resourceID)), 69 TRACE_ID_WITH_SCOPE(kBlinkResourceID, TRACE_ID_LOCAL(resource_id)),
70 "beginData", scopedResourceTrackerBeginData(request)); 70 "beginData", ScopedResourceTrackerBeginData(request));
71 } 71 }
72 72
73 ScopedResourceLoadTracker::~ScopedResourceLoadTracker() { 73 ScopedResourceLoadTracker::~ScopedResourceLoadTracker() {
74 if (!m_resourceLoadContinuesBeyondScope) 74 if (!resource_load_continues_beyond_scope_)
75 endResourceLoad(m_resourceID, RequestOutcome::Fail); 75 EndResourceLoad(resource_id_, RequestOutcome::kFail);
76 } 76 }
77 77
78 void ScopedResourceLoadTracker::resourceLoadContinuesBeyondScope() { 78 void ScopedResourceLoadTracker::ResourceLoadContinuesBeyondScope() {
79 m_resourceLoadContinuesBeyondScope = true; 79 resource_load_continues_beyond_scope_ = true;
80 } 80 }
81 81
82 void resourcePrioritySet(unsigned long resourceID, 82 void ResourcePrioritySet(unsigned long resource_id,
83 blink::ResourceLoadPriority priority) { 83 blink::ResourceLoadPriority priority) {
84 TRACE_EVENT_NESTABLE_ASYNC_INSTANT1( 84 TRACE_EVENT_NESTABLE_ASYNC_INSTANT1(
85 kNetInstrumentationCategory, kResourcePrioritySetTitle, 85 kNetInstrumentationCategory, kResourcePrioritySetTitle,
86 TRACE_ID_WITH_SCOPE(kBlinkResourceID, TRACE_ID_LOCAL(resourceID)), "data", 86 TRACE_ID_WITH_SCOPE(kBlinkResourceID, TRACE_ID_LOCAL(resource_id)),
87 resourcePrioritySetData(priority)); 87 "data", ResourcePrioritySetData(priority));
88 } 88 }
89 89
90 void endResourceLoad(unsigned long resourceID, RequestOutcome outcome) { 90 void EndResourceLoad(unsigned long resource_id, RequestOutcome outcome) {
91 TRACE_EVENT_NESTABLE_ASYNC_END1( 91 TRACE_EVENT_NESTABLE_ASYNC_END1(
92 kNetInstrumentationCategory, kResourceLoadTitle, 92 kNetInstrumentationCategory, kResourceLoadTitle,
93 TRACE_ID_WITH_SCOPE(kBlinkResourceID, TRACE_ID_LOCAL(resourceID)), 93 TRACE_ID_WITH_SCOPE(kBlinkResourceID, TRACE_ID_LOCAL(resource_id)),
94 "endData", endResourceLoadData(outcome)); 94 "endData", EndResourceLoadData(outcome));
95 } 95 }
96 96
97 } // namespace network_instrumentation 97 } // namespace network_instrumentation
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698