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

Unified Diff: content/browser/renderer_host/render_widget_host_view_android.cc

Issue 780653007: Added GL_CHROMIUM_trace_marker feature as well as gpu_toplevel markers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed endtrace calls Created 6 years 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_view_android.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc
index 574c88fb9f24b07bc52da3014d167cb083a98974..154506aa1a1f65a2074ebbab01cf656168559622 100644
--- a/content/browser/renderer_host/render_widget_host_view_android.cc
+++ b/content/browser/renderer_host/render_widget_host_view_android.cc
@@ -135,6 +135,9 @@ GLHelperHolder::GLHelperHolder() {
}
GLHelperHolder::~GLHelperHolder() {
+ if (context_) {
+ context_->traceEndCHROMIUM();
David Yen 2014/12/18 21:26:00 Should this one be removed as well?
vmiura 2014/12/18 21:49:39 Yeah, let's remove this one as well.
David Yen 2014/12/18 21:54:27 Done.
+ }
}
void GLHelperHolder::Initialize() {
@@ -191,7 +194,8 @@ GLHelperHolder::CreateContext3D() {
url, gpu_channel_host.get(), attrs, lose_context_when_out_of_memory,
limits, nullptr));
if (context->InitializeOnCurrentThread()) {
- context->pushGroupMarkerEXT(
+ context->traceBeginCHROMIUM(
+ "gpu_toplevel",
base::StringPrintf("CmdBufferImageTransportFactory-%p",
context.get()).c_str());
} else {

Powered by Google App Engine
This is Rietveld 408576698