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

Side by Side Diff: content/common/android/trace_event_binding.cc

Issue 12252058: Add a |scope| argument to TRACE_EVENT_INSTANT* and require its presence. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix builds 2 Created 7 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "content/common/android/trace_event_binding.h" 5 #include "content/common/android/trace_event_binding.h"
6 6
7 #include <jni.h> 7 #include <jni.h>
8 8
9 #include <set> 9 #include <set>
10 10
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 82
83 static void StopATrace(JNIEnv* env, jclass clazz) { 83 static void StopATrace(JNIEnv* env, jclass clazz) {
84 base::debug::TraceLog::GetInstance()->StopATrace(); 84 base::debug::TraceLog::GetInstance()->StopATrace();
85 } 85 }
86 86
87 static void Instant(JNIEnv* env, jclass clazz, 87 static void Instant(JNIEnv* env, jclass clazz,
88 jstring jname, jstring jarg) { 88 jstring jname, jstring jarg) {
89 TraceEventDataConverter converter(env, jname, jarg); 89 TraceEventDataConverter converter(env, jname, jarg);
90 if (converter.arg()) { 90 if (converter.arg()) {
91 TRACE_EVENT_COPY_INSTANT1(kJavaCategory, converter.name(), 91 TRACE_EVENT_COPY_INSTANT1(kJavaCategory, converter.name(),
92 TRACE_EVENT_SCOPE_THREAD,
92 converter.arg_name(), converter.arg()); 93 converter.arg_name(), converter.arg());
93 } else { 94 } else {
94 TRACE_EVENT_COPY_INSTANT0(kJavaCategory, converter.name()); 95 TRACE_EVENT_COPY_INSTANT0(kJavaCategory, converter.name(),
96 TRACE_EVENT_SCOPE_THREAD);
95 } 97 }
96 } 98 }
97 99
98 static void Begin(JNIEnv* env, jclass clazz, 100 static void Begin(JNIEnv* env, jclass clazz,
99 jstring jname, jstring jarg) { 101 jstring jname, jstring jarg) {
100 TraceEventDataConverter converter(env, jname, jarg); 102 TraceEventDataConverter converter(env, jname, jarg);
101 if (converter.arg()) { 103 if (converter.arg()) {
102 TRACE_EVENT_COPY_BEGIN1(kJavaCategory, converter.name(), 104 TRACE_EVENT_COPY_BEGIN1(kJavaCategory, converter.name(),
103 converter.arg_name(), converter.arg()); 105 converter.arg_name(), converter.arg());
104 } else { 106 } else {
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 } else { 147 } else {
146 TRACE_EVENT_COPY_ASYNC_END0(kJavaCategory, 148 TRACE_EVENT_COPY_ASYNC_END0(kJavaCategory,
147 converter.name(), 149 converter.name(),
148 converter.id()); 150 converter.id());
149 } 151 }
150 } 152 }
151 153
152 bool RegisterTraceEvent(JNIEnv* env) { 154 bool RegisterTraceEvent(JNIEnv* env) {
153 return RegisterNativesImpl(env); 155 return RegisterNativesImpl(env);
154 } 156 }
OLDNEW
« no previous file with comments | « content/browser/renderer_host/render_widget_host_impl.cc ('k') | content/common/gpu/image_transport_surface.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698