Turn on -Wexit-time-destructors for ui and content.

ui/ui.gyp

 # Copyright (c) 2011 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.
-

[Mark Mentovai, 2011/11/09 17:44:44]

Don’t remove this line, most of our files seem to have it.

[Nico, 2011/11/09 18:16:02]

Fine, let's waste those bytes. Done.

 {
   'variables': {
     'chromium_code': 1,
   },
   'target_defaults': {
     'conditions': [
       ['touchui==0', {'sources/': [
         ['exclude', '_(touch)\\.cc$'],
       ]}],
     ],
   },
   'includes': [
     'ui_resources.gypi',
   ],
   'targets': [
     {
       'target_name': 'ui',
       'type': '<(component)',
+       'variables': { 'enable_wexit_time_destructors': 1, }, 

[Mark Mentovai, 2011/11/09 17:44:44]

Can this go into the top-level 'variables' in this file next to 'chromium_code'?

[Nico, 2011/11/09 18:16:02]

No, then it's enabled for tests as well, and they aren't clean yet.

       'dependencies': [
         '../base/base.gyp:base',
         '../base/base.gyp:base_i18n',
         '../base/base.gyp:base_static',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         '../build/temp_gyp/googleurl.gyp:googleurl',
         '../net/net.gyp:net',
         '../skia/skia.gyp:skia',
         '../third_party/icu/icu.gyp:icui18n',
         '../third_party/icu/icu.gyp:icuuc',
@@ skipping 576 matching lines @@
                 'shutil.copyfile(\'<(ui_copy_target)\', \'<(ui_copy_dest)\'); ' \
                 'os.chmod(\'<(ui_copy_dest)\', 0700)'
               ]
             }
           ],
         },
       ],
     }],
   ],
 }