collect minidump if it chrashes

gyp/tools.gyp

 # GYP file to build various tools.
 #
 # To build on Linux:
 #  ./gyp_skia tools.gyp && make tools
 #
 # Building on other platforms not tested yet.
 #
 {
   'includes': [
     'apptype_console.gypi',
@@ skipping 166 matching lines @@
         '../src/utils/',
       ],
       'dependencies': [
         'core.gyp:core',
         'effects.gyp:effects',
         'images.gyp:images',
         'pdf.gyp:pdf',
         'ports.gyp:ports',
         'tools.gyp:picture_utils',
       ],
+      'conditions': [
+        ['skia_win_debuggers_path and skia_os == "win"',
+          {
+            'dependencies': [
+              'tools.gyp:win_dbghelp',
+            ],
+          },
+        ],
+        # VS static libraries don't have a linker option. We must set a global
+        # project linker option, or add it to each executable.
+        ['skia_win_debuggers_path and skia_os == "win" and '
+            'skia_arch_width == 64',

[borenet, 2013/03/08 16:59:52]

Sorry for continuing to nitpick... can we make this a 4-space indent instead of
3?

[edisonn, 2013/03/08 17:05:39]

but it is a 4 space indent, I think it should be calculated from [

[borenet, 2013/03/08 17:11:21]

I disagree.  To me, the proper alignment for a second line which is not indented
further would be like this:

    ['blahblah......'
     'blahblah']

Maybe the above is what we really want here anyway.  Elliot, your thoughts?

[edisonn, 2013/03/08 17:13:12]

done; yeah, look better like this
On 2013/03/08 17:11:21, borenet wrote:

+          {
+            'msvs_settings': {
+              'VCLinkerTool': {
+                'AdditionalDependencies': [
+                  '<(skia_win_debuggers_path)/x64/DbgHelp.lib',
+                ],
+              },
+            },
+          },
+        ],
+        ['skia_win_debuggers_path and skia_os == "win" and '
+            'skia_arch_width == 32',

[borenet, 2013/03/08 16:59:52]

Same here.

[edisonn, 2013/03/08 17:05:39]

same

+          {
+            'msvs_settings': {
+              'VCLinkerTool': {
+                'AdditionalDependencies': [
+                  '<(skia_win_debuggers_path)/DbgHelp.lib',
+                ],
+              },
+            },
+          },
+        ],
+      ],
     },
     {
       'target_name': 'picture_utils',
       'type': 'static_library',
       'sources': [
         '../tools/picture_utils.cpp',
         '../tools/picture_utils.h',
       ],
       'dependencies': [
         'skia_base_libs.gyp:skia_base_libs',
@@ skipping 29 matching lines @@
         '../debugger/SkObjectParser.h',
         '../debugger/SkObjectParser.cpp',
       ],
       'dependencies': [
         'skia_base_libs.gyp:skia_base_libs',
         'effects.gyp:effects',
         'images.gyp:images',
         'tools.gyp:picture_utils',
       ],
     },
+    {
+      'target_name': 'win_dbghelp',
+      'type': 'static_library',
+      'defines': [
+        'SK_CDB_PATH="<(skia_win_debuggers_path)"',
+      ],
+      'sources': [
+        '../tools/win_dbghelp.h',
+        '../tools/win_dbghelp.cpp',
+      ],
+    },
   ],
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: