gyp: <(GENERATOR) is now always ninja, so simplify some conditionals.

breakpad/breakpad.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.
 
 {
   'includes': [
     'breakpad_sender.gypi',
     'breakpad_handler.gypi',
   ],
   'conditions': [
-    # minidump_stackwalk and minidump_dump are tool-type executables that do
-    # not build on iOS with Xcode (but do build on iOS with ninja.)
-    ['(OS!="ios" or "<(GENERATOR)"!="xcode" or "<(GENERATOR_FLAVOR)"=="ninja") and OS!="win"', {
+    ['OS!="win"', {
       'targets': [
         {
           # code shared by both {micro,mini}dump_stackwalk
           # GN version: //breakpad:stackwalk_common
           'target_name': 'stackwalk_common',
           'type': 'static_library',
           'includes': ['breakpad_tools.gypi'],
           'defines': ['BPLOG_MINIMUM_SEVERITY=SEVERITY_ERROR'],
           'sources': [
             'src/processor/basic_code_module.h',
@@ skipping 135 matching lines @@
             'src/processor/proc_maps_linux.cc',
           ],
           'conditions': [
             ['OS=="ios"', {
               'toolsets': ['host'],
             }],
           ],
         },
       ],
     }],
-    ['OS=="mac" or (OS=="ios" and ("<(GENERATOR)"!="xcode" or "<(GENERATOR_FLAVOR)"=="ninja"))', {
+    ['OS=="mac" or OS=="ios"', {
       'target_defaults': {
         'include_dirs': [
           'src',
         ],
         'configurations': {
           'Debug_Base': {
             'defines': [
               # This is needed for GTMLogger to work correctly.
               'DEBUG',
             ],
@@ skipping 709 matching lines @@
           'xcode_settings' : {
             'WARNING_CFLAGS': [
               # See https://bugs.chromium.org/p/google-breakpad/issues/detail?id=675.
               # TODO(crbug.com/569158): remove when fixed.
               '-Wno-deprecated-declarations',
             ],
           },
         }
       ]
     }],
-    ['OS=="ios" and "<(GENERATOR)"=="xcode" and "<(GENERATOR_FLAVOR)"!="ninja"', {
-      'variables': {
-        'ninja_output_dir': 'ninja-breakpad',
-        'ninja_product_dir':
-          '<(DEPTH)/xcodebuild/<(ninja_output_dir)/<(CONFIGURATION_NAME)',
-      },
-      # Generation is done via two actions: (1) compiling the executable with
-      # ninja, and (2) copying the executable into a location that is shared
-      # with other projects. These actions are separated into two targets in
-      # order to be able to specify that the second action should not run until
-      # the first action finishes (since the ordering of multiple actions in
-      # one target is defined only by inputs and outputs, and it's impossible
-      # to set correct inputs for the ninja build, so setting all the inputs
-      # and outputs isn't an option).
-      'targets': [
-        {
-          'target_name': 'compile_breakpad_utilities',
-          'type': 'none',
-          'variables': {
-            # Gyp to rerun
-            're_run_targets': [
-              'breakpad/breakpad.gyp',
-            ],
-          },
-          'includes': ['../build/ios/mac_build.gypi'],
-          'actions': [
-            {
-              'action_name': 'compile breakpad utilities',
-              'inputs': [],
-              'outputs': [],
-              'action': [
-                '<@(ninja_cmd)',
-                'dump_syms',
-                'symupload',
-              ],
-              'message': 'Generating the breakpad executables',
-            },
-          ],
-        },
-        {
-          'target_name': 'breakpad_utilities',
-          'type': 'none',
-          'dependencies': [
-            'compile_breakpad_utilities',
-          ],
-          'actions': [
-            {
-              'action_name': 'copy dump_syms',
-              'inputs': [
-                '<(ninja_product_dir)/dump_syms',
-              ],
-              'outputs': [
-                '<(PRODUCT_DIR)/dump_syms',
-              ],
-              'action': [
-                'cp',
-                '<(ninja_product_dir)/dump_syms',
-                '<(PRODUCT_DIR)/dump_syms',
-              ],
-            },
-            {
-              'action_name': 'copy symupload',
-              'inputs': [
-                '<(ninja_product_dir)/symupload',
-              ],
-              'outputs': [
-                '<(PRODUCT_DIR)/symupload',
-              ],
-              'action': [
-                'cp',
-                '<(ninja_product_dir)/symupload',
-                '<(PRODUCT_DIR)/symupload',
-              ],
-            },
-          ],
-        },
-        {
-          'target_name': 'dump_syms',
-          'type': 'none',
-          'dependencies': [
-            'breakpad_utilities',
-          ],
-        },
-        {
-          'target_name': 'symupload',
-          'type': 'none',
-          'dependencies': [
-            'breakpad_utilities',
-          ],
-        }
-      ],
-    }],
     ['OS=="android"', {
       'targets': [
         {
           'target_name': 'breakpad_unittests_stripped',
           'type': 'none',
           'dependencies': [ 'breakpad_unittests' ],
           'actions': [{
             'action_name': 'strip breakpad_unittests',
             'inputs': [ '<(PRODUCT_DIR)/breakpad_unittests' ],
             'outputs': [ '<(PRODUCT_DIR)/breakpad_unittests_stripped' ],
@@ skipping 35 matching lines @@
                   'breakpad_unittests_apk.isolate',
                 ],
               },
             ]
           }
         ],
       ],
     }],
   ],
 }