Adding build-time option to enable the tracing of skia events in

skia/skia.gyp

 # Copyright (c) 2012 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.
 
 {
   'targets': [
     {
       'target_name': 'skia',
       'type': '<(component)',
       'variables': {
@@ skipping 703 matching lines @@
         'ext/platform_canvas_skia.cc',
         'ext/platform_canvas_win.cc',
         'ext/platform_device.cc',
         'ext/platform_device.h',
         'ext/platform_device_linux.cc',
         'ext/platform_device_mac.cc',
         'ext/platform_device_win.cc',
         'ext/SkMemory_new_handler.cpp',
         'ext/skia_sandbox_support_win.h',
         'ext/skia_sandbox_support_win.cc',
+        'ext/skia_trace_shim.h',
         'ext/skia_utils_mac.mm',
         'ext/skia_utils_mac.h',
         'ext/skia_utils_win.cc',
         'ext/skia_utils_win.h',
         'ext/vector_canvas.cc',
         'ext/vector_canvas.h',
         'ext/vector_platform_device_emf_win.cc',
         'ext/vector_platform_device_emf_win.h',
         'ext/vector_platform_device_skia.cc',
         'ext/vector_platform_device_skia.h',
@@ skipping 18 matching lines @@
         '$(SDKROOT)/System/Library/Frameworks/ApplicationServices.framework/Frameworks',
       ],
       'defines': [
         'SK_BUILD_NO_IMAGE_ENCODE',
         'GR_GL_CUSTOM_SETUP_HEADER="GrGLConfig_chrome.h"',
         'GR_STATIC_RECT_VB=1',
         'GR_AGGRESSIVE_SHADER_OPTS=1',
         'SK_DISABLE_FAST_AA_STROKE_RECT',
         'SK_DEFAULT_FONT_CACHE_LIMIT=(20*1024*1024)',
 
+        # tracing of skia events is disabled by default. To enable
+        # tracing for your build, comment-out the line
+        # #undef SK_USER_TRACE_INCLUDE_FILE
+        # in file config/SkUserConfig.h
+        'SK_USER_TRACE_INCLUDE_FILE="skia/ext/skia_trace_shim.h"',

[senorblanco, 2012/04/02 20:49:12]

I'd prefer to do this all in one place, not in two.  Having the SkUserConfig.h
#undef this after the gyp file sets it is kind of weird.  I'd rather it be in
the SkUserConfig.h, in the chrome-specific block at the bottom.  You can just
put the #define there, commented out.

+
         # temporary for landing Skia rev 3077 with minimal layout test breakage
         'SK_SIMPLE_TWOCOLOR_VERTICAL_GRADIENTS',
 
         # skia uses static initializers to initialize the serialization logic
         # of its "pictures" library. This is currently not used in chrome; if
         # it ever gets used the processes that use it need to call
         # SkGraphics::Init().
         'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=0',
 
         # Temporarily disable the Skia fix in
@@ skipping 477 matching lines @@
       ],
       'include_dirs': [
         '..',
       ],
       'sources': [
         'ext/image_operations_bench.cc',
       ],
     },
   ],
 }