gpu: Add more context labelling.

cc/output/gl_renderer.cc

 // Copyright 2010 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.
 
 #include "cc/output/gl_renderer.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
 #include "build/build_config.h"
 #include "cc/base/math_util.h"
 #include "cc/layers/video_layer_impl.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/output/compositor_frame_metadata.h"
 #include "cc/output/context_provider.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/copy_output_result.h"
 #include "cc/output/geometry_binding.h"
 #include "cc/output/gl_frame_data.h"
@@ skipping 104 matching lines @@
       shared_geometry_quad_(gfx::RectF(-0.5f, -0.5f, 1.0f, 1.0f)),
       context_(output_surface->context3d()),
       is_backbuffer_discarded_(false),
       discard_backbuffer_when_not_visible_(false),
       is_using_bind_uniform_(false),
       visible_(true),
       is_scissor_enabled_(false),
       highp_threshold_min_(highp_threshold_min),
       highp_threshold_cache_(0),
       on_demand_tile_raster_resource_id_(0),
-      weak_factory_(this) {
+      weak_factory_(this),
+      offscreen_context_labelled_(false) {
   DCHECK(context_);
 }
 
 bool GLRenderer::Initialize() {
   if (!context_->makeContextCurrent())
     return false;
 
-  context_->pushGroupMarkerEXT(Settings().compositor_name.c_str());
+  std::string unique_context_name = base::StringPrintf(
+      "%s-%p",
+      Settings().compositor_name.c_str(),
+      context_);
+  context_->pushGroupMarkerEXT(unique_context_name.c_str());
 
   std::string extensions_string =
       UTF16ToASCII(context_->getString(GL_EXTENSIONS));
   std::vector<std::string> extensions_list;
   base::SplitString(extensions_string, ' ', &extensions_list);
   std::set<std::string> extensions(extensions_list.begin(),
                                    extensions_list.end());
 
   capabilities_.using_partial_swap =
       Settings().partial_swap_enabled &&
@@ skipping 295 matching lines @@
                                            source_texture_resource->id());
 
   // Flush the compositor context to ensure that textures there are available
   // in the shared context.  Do this after locking/creating the compositor
   // texture.
   renderer->resource_provider()->Flush();
 
   // Make sure skia uses the correct GL context.
   offscreen_contexts->Context3d()->makeContextCurrent();
 
+  // Lazily label this context.
+  renderer->LazyLabelOffscreenContext();
+
   SkBitmap source =
       RenderSurfaceFilters::Apply(filters,
                                   lock.texture_id(),
                                   source_texture_resource->size(),
                                   offscreen_contexts->GrContext());
 
   // Flush skia context so that all the rendered stuff appears on the
   // texture.
   offscreen_contexts->GrContext()->flush();
 
@@ skipping 21 matching lines @@
                                            source_texture_resource->id());
 
   // Flush the compositor context to ensure that textures there are available
   // in the shared context.  Do this after locking/creating the compositor
   // texture.
   renderer->resource_provider()->Flush();
 
   // Make sure skia uses the correct GL context.
   offscreen_contexts->Context3d()->makeContextCurrent();
 
+  // Lazily label this context.
+  renderer->LazyLabelOffscreenContext();
+
   // Wrap the source texture in a Ganesh platform texture.
   GrBackendTextureDesc backend_texture_description;
   backend_texture_description.fWidth = source_texture_resource->size().width();
   backend_texture_description.fHeight =
       source_texture_resource->size().height();
   backend_texture_description.fConfig = kSkia8888_GrPixelConfig;
   backend_texture_description.fTextureHandle = lock.texture_id();
   backend_texture_description.fOrigin = kBottomLeft_GrSurfaceOrigin;
   skia::RefPtr<GrTexture> texture =
       skia::AdoptRef(offscreen_contexts->GrContext()->wrapBackendTexture(
@@ skipping 2512 matching lines @@
 bool GLRenderer::CanUseSkiaGPUBackend() const {
   // The Skia GPU backend requires a stencil buffer.  See ReinitializeGrCanvas
   // implementation.
   return gr_context_ && context_->getContextAttributes().stencil;
 }
 
 bool GLRenderer::IsContextLost() {
   return (context_->getGraphicsResetStatusARB() != GL_NO_ERROR);
 }
 
+void GLRenderer::LazyLabelOffscreenContext() {
+  if (!offscreen_context_labelled_) {

[danakj, 2013/06/25 15:02:19]

nit: if (labelled) return;

+    offscreen_context_labelled_ = true;
+    std::string unique_context_name = base::StringPrintf(
+        "%s-Offscreen-%p",
+        Settings().compositor_name.c_str(),
+        context_);
+    resource_provider()->offscreen_context_provider()->Context3d()->
+        pushGroupMarkerEXT(unique_context_name.c_str());
+  }
+}
+
+
 }  // namespace cc