[mb] Remove redundant flags

infra/mb/mb_config.pyl

 # Copyright 2016 The V8 project authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   # This is a map of buildbot master names -> buildbot builder names ->
   # config names (where each config name is a key in the 'configs' dict,
   # below). MB uses this dict to look up which config to use for a given bot.
   # Bots are ordered by appearance on waterfall.
   'masters': {
@@ skipping 30 matching lines @@
       'gn', 'release_bot', 'swarming', 'x64'],
   },
 
   'mixins': {
     'dcheck_always_on': {
       'gn_args': 'dcheck_always_on=true',
       'gyp_defines': 'dcheck_always_on=1',
     },
 
     'debug': {
-      'gyp_defines': 'v8_enable_slow_dchecks=1 v8_optimized_debug=1',
       'gn_args': 'is_debug=true',
     },
 
     'debug_bot': {
       'mixins': [
         'debug', 'static', 'goma', 'v8_enable_slow_dchecks',

[Michael Achenbach, 2016/06/17 12:25:11]

Those flags are specified already for the debug_bot mixin. Now they appear twice
in prod.

For debug it would be too generic, as we e.g. have a bot setting optimized debug
to 0.

         'v8_optimized_debug'],
     },
 
     'debug_trybot': {
       'mixins': ['debug_bot', 'minimal_symbols'],
     },
 
     'gn': {'type': 'gn'},
 
     'goma': {
@@ skipping 68 matching lines @@
       'gn_args': 'target_cpu="x64"',
       'gyp_defines': 'target_arch=x64',
     },
 
     #'x86': {
     #  'gn_args': 'target_cpu="x86"',
     #  'gyp_defines': 'target_arch=ia32',
     #},
   },
 }