Chromium Code Reviews| Index: content/renderer/devtools/v8_sampling_profiler.cc |
| diff --git a/content/renderer/devtools/v8_sampling_profiler.cc b/content/renderer/devtools/v8_sampling_profiler.cc |
| index cdad93b20063b65d286b637f5911fe185965bd60..b869027101a25272d5032ac84c401b3655d53583 100644 |
| --- a/content/renderer/devtools/v8_sampling_profiler.cc |
| +++ b/content/renderer/devtools/v8_sampling_profiler.cc |
| @@ -327,23 +327,23 @@ void Sampler::HandleJitCodeEvent(const v8::JitCodeEvent* event) { |
| return; |
| switch (event->type) { |
| case v8::JitCodeEvent::CODE_ADDED: |
| - TRACE_EVENT_INSTANT1(TRACE_DISABLED_BY_DEFAULT("v8.cpu_profile"), |
| - "JitCodeAdded", TRACE_EVENT_SCOPE_THREAD, "data", |
| - JitCodeEventToTraceFormat(event)); |
| + TRACE_EVENT_METADATA1(TRACE_DISABLED_BY_DEFAULT("v8.cpu_profile"), |
| + "JitCodeAdded", "data", |
| + JitCodeEventToTraceFormat(event)); |
| base::subtle::NoBarrier_AtomicIncrement( |
| &sampler->code_added_events_count_, 1); |
| break; |
| case v8::JitCodeEvent::CODE_MOVED: |
| - TRACE_EVENT_INSTANT1(TRACE_DISABLED_BY_DEFAULT("v8.cpu_profile"), |
| - "JitCodeMoved", TRACE_EVENT_SCOPE_THREAD, "data", |
| - JitCodeEventToTraceFormat(event)); |
| + TRACE_EVENT_METADATA1(TRACE_DISABLED_BY_DEFAULT("v8.cpu_profile"), |
| + "JitCodeMoved", "data", |
| + JitCodeEventToTraceFormat(event)); |
| break; |
| case v8::JitCodeEvent::CODE_REMOVED: |
|
dsinclair
2015/07/02 14:17:05
So, this isn't usually emitted, but it can be?
alph
2015/07/02 14:59:55
Looks like this part is not implemented. https://c
|
| - TRACE_EVENT_INSTANT1(TRACE_DISABLED_BY_DEFAULT("v8.cpu_profile"), |
| - "JitCodeRemoved", TRACE_EVENT_SCOPE_THREAD, "data", |
| - JitCodeEventToTraceFormat(event)); |
| + TRACE_EVENT_METADATA1(TRACE_DISABLED_BY_DEFAULT("v8.cpu_profile"), |
| + "JitCodeRemoved", "data", |
| + JitCodeEventToTraceFormat(event)); |
| break; |
| case v8::JitCodeEvent::CODE_ADD_LINE_POS_INFO: |