Make all_webkit depend on breakpad targets required to symbolize crashes

build/all.gyp

Index: build/all.gyp
diff --git a/build/all.gyp b/build/all.gyp
index 43ac284e51a2853fabc941f8afc32a35bd75d430..dc997af289a677b0177f67d2ae182f7a031ad4fc 100644
--- a/build/all.gyp
+++ b/build/all.gyp
@@ -369,6 +369,18 @@
             '../third_party/WebKit/public/all.gyp:all_blink',
             '../content/content_shell_and_tests.gyp:content_shell',
           ],
+          'conditions': [
+            ['OS!="win"', {

[Dirk Pranke, 2013/10/30 19:19:57]

Nit: I would probably flip this conditional, but it's not all that important ...

+              'dependencies': [
+                '../breakpad/breakpad.gyp:dump_syms',
+                '../breakpad/breakpad.gyp:minidump_stackwalk',

[Nico, 2013/11/27 01:06:30]

This isn't correct. dump_syms only exists if OS is mac, ios, linux, or android.
minidump_stackwalk doesn't exist on ios.

As a result, this breaks tools/gyp-explain.py.

The best fix is probably to change breakpad.gyp to define these targets
unconditionally and make them type: none and empty on platforms where they don't
currently exist. 

[jochen (gone - plz use gerrit), 2013/11/27 09:23:28]

all_webkit is defined within a OS!="ios" block, so not sure what you mean?

+              ],
+            }, {  # OS=="win"
+              'dependencies': [
+                '../content/content_shell_and_tests.gyp:content_shell_crash_service',
+              ],
+            }],
+          ],
         }, # target_name: all_webkit
         {
           'target_name': 'chromium_builder_nacl_win_integration',