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

content/content.gyp

Index: content/content.gyp
diff --git a/content/content.gyp b/content/content.gyp
index be3f85c6732549852aff9865435fbe45d12feb6e..effe548473646ea390c583c1b039a4223a4098b2 100644
--- a/content/content.gyp
+++ b/content/content.gyp
@@ -27,6 +27,7 @@
      'targets': [
       {'target_name': 'content',
        'type': 'none',
+       'variables': { 'enable_wexit_time_destructors': 1, },

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

Not strictly necessary for a none-type target, but see the next comment.

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

Removed.

        'dependencies': [
         'content_app',
         'content_browser',
@@ -41,6 +42,7 @@
       },
       {'target_name': 'content_app',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },

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

Why not put this in the variables block at the top of this file next to
chromium_code? It looks like all of the targets in this file want to
participate, and the new GYP variable functions the same as chromium_code, so
that seems like a logical place for it.

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

I believe the included gypi files get it too in that case, and the tests / the
shell aren't clean.

        'includes': [
           'content_app.gypi',
         ],
@@ -50,6 +52,7 @@
       },
       {'target_name': 'content_browser',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_browser.gypi',
         ],
@@ -59,12 +62,14 @@
       },
       {'target_name': 'content_common',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_common.gypi',
         ],
       },
       {'target_name': 'content_gpu',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_gpu.gypi',
         ],
@@ -74,6 +79,7 @@
       },
       {'target_name': 'content_plugin',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_plugin.gypi',
         ],
@@ -83,12 +89,14 @@
       },
       {'target_name': 'content_ppapi_plugin',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_ppapi_plugin.gypi',
         ],
       },
       {'target_name': 'content_renderer',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_renderer.gypi',
         ],
@@ -98,6 +106,7 @@
       },
       {'target_name': 'content_utility',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_utility.gypi',
         ],
@@ -107,6 +116,7 @@
       },
       {'target_name': 'content_worker',
        'type': 'static_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
           'content_worker.gypi',
        ],
@@ -185,6 +195,7 @@
      'targets': [
       {'target_name': 'content',
        'type': 'shared_library',
+       'variables': { 'enable_wexit_time_destructors': 1, },
        'includes': [
         'content_app.gypi',
         'content_browser.gypi',