Cleanup: Combine some identical conditionals in chrome gyp targets.

chrome/chrome_tests_unit.gypi

 # Copyright 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.
 {
   'targets': [
     {
       # This target contains mocks and test utilities that don't belong in
       # production libraries but are used by more than one test executable.
       'target_name': 'test_support_common',
       'type': 'static_library',
@@ skipping 2165 matching lines @@
             # be fixed by creating a separate unit test target, or by deleting
             # the test.
             'browser/ui/libgtk2ui/x11_input_method_context_impl_gtk2_unittest.cc'
           ],
         }],
         ['enable_task_manager==0', {
           'sources/': [
             ['exclude', '^browser/task_manager/'],
           ],
         }],
-        ['chromeos==0', {
-          'sources!': [
-            # TODO(zturner): Enable this on Windows.  See
-            # BrowserWithTestWindowTest::SetUp() for a comment explaining why
-            # this is broken.
-            'browser/ui/views/frame/immersive_mode_controller_ash_unittest.cc',
-            'browser/ui/views/select_file_dialog_extension_unittest.cc',
-          ],
-        }],
         ['enable_mdns==1', {
           'sources': [
             'browser/local_discovery/privet_device_lister_unittest.cc',
             'browser/local_discovery/privet_local_printer_lister_unittest.cc',
             'browser/local_discovery/privet_notifications_unittest.cc',
             'common/local_discovery/local_domain_resolver_unittest.cc',
             'common/local_discovery/service_discovery_client_unittest.cc',
           ],
         }],
         ['enable_service_discovery==1', {
@@ skipping 71 matching lines @@
           'sources!': [
             'browser/component_updater/test/component_installers_unittest.cc',
             'browser/metrics/plugin_metrics_provider_unittest.cc',
             'common/pepper_permission_util_unittest.cc',
           ],
         }],
         ['enable_printing!=1', {
           'sources/': [
             ['exclude', '^browser/printing/'],
             ['exclude', '^browser/ui/webui/print_preview/'],
+            # No need for service process.
+            ['exclude', '^browser/service/'],
+            ['exclude', '^common/service_'],
             ['exclude', '^service/'],
           ],
           'sources!': [
             'browser/local_discovery/privet_http_unittest.cc',
           ],
         }],
         ['enable_captive_portal_detection!=1', {
           'sources/': [
             ['exclude', '^browser/captive_portal/'],
           ],
@@ skipping 48 matching lines @@
           'sources/': [
             ['exclude', '^browser/chromeos/'],
             ['exclude', '^browser/ui/webui/chromeos/imageburner/'],
             ['exclude', '^browser/ui/webui/chromeos/login'],
             ['exclude', '^browser/ui/webui/options/chromeos/'],
             ['exclude', '^browser/ui/webui/options/chromeos/'],
             ['exclude', '^browser/resources/chromeos/'],
           ],
           'sources!': [
             'browser/extensions/api/enterprise_platform_keys_private/enterprise_platform_keys_private_api_unittest.cc',
+            'browser/extensions/api/log_private/syslog_parser_unittest.cc',
             'browser/notifications/login_state_notification_blocker_chromeos_unittest.cc',
-            'browser/extensions/api/log_private/syslog_parser_unittest.cc',
+            # TODO(zturner): Enable this on Windows.  See
+            # BrowserWithTestWindowTest::SetUp() for a comment explaining why
+            # this is broken.
+            'browser/ui/views/frame/immersive_mode_controller_ash_unittest.cc',
+            'browser/ui/views/select_file_dialog_extension_unittest.cc',
           ],
         }],
         ['use_x11==1', {
           'dependencies': [
             '../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
           ],
         }],
         ['OS=="linux"', {
           'dependencies': [
             '../build/linux/system.gyp:ssl',
@@ skipping 36 matching lines @@
           ],
         }],
         ['OS=="mac"', {
           # The test fetches resources which means Mac need the app bundle to
           # exist on disk so it can pull from it.
           'dependencies': [
             '../third_party/google_toolbox_for_mac/google_toolbox_for_mac.gyp:google_toolbox_for_mac',
             '../third_party/ocmock/ocmock.gyp:ocmock',
             'chrome',
           ],
+          'sources': [
+            'browser/media_galleries/fileapi/iphoto_file_util_unittest.cc',
+            'utility/media_galleries/iphoto_library_parser_unittest.cc',
+          ],
           'sources!': [
             'browser/ui/tests/ui_gfx_image_unittest.cc',
             'tools/convert_dict/convert_dict_unittest.cc',
             'renderer/spellchecker/spellcheck_provider_hunspell_unittest.cc',
           ],
           # TODO(mark): We really want this for all non-static library targets,
           # but when we tried to pull it up to the common.gypi level, it broke
           # other things like the ui and startup tests. *shrug*
           'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-ObjC']},
         }],
@@ skipping 14 matching lines @@
           'sources': [
             'browser/media_galleries/fileapi/itunes_file_util_unittest.cc',
             'browser/media_galleries/fileapi/picasa_file_util_unittest.cc',
             'common/extensions/api/networking_private/networking_private_crypto_unittest.cc',
             'utility/media_galleries/itunes_library_parser_unittest.cc',
             'utility/media_galleries/picasa_album_table_reader_unittest.cc',
             'utility/media_galleries/picasa_albums_indexer_unittest.cc',
             'utility/media_galleries/pmp_column_reader_unittest.cc',
           ],
         }],
-        ['OS=="mac"', {
-          'sources': [
-            'browser/media_galleries/fileapi/iphoto_file_util_unittest.cc',
-            'utility/media_galleries/iphoto_library_parser_unittest.cc',
-          ],
-        }],
         ['enable_rlz!=0', {
           'dependencies': [
             '../rlz/rlz.gyp:test_support_rlz',
           ],
         }, {  # enable_rlz==0
           'sources!': [
             'browser/rlz/rlz_unittest.cc',
           ],
         }],
         ['OS=="win" and component!="shared_library"', {
@@ skipping 167 matching lines @@
             # Android doesn't support download feedbacks.
             'browser/safe_browsing/download_feedback_unittest.cc',
             'browser/safe_browsing/download_feedback_service_unittest.cc',
             'browser/safe_browsing/download_protection_service_unittest.cc',
             'browser/safe_browsing/two_phase_uploader_unittest.cc',
           ],
           'sources/': [
             ['exclude', '^browser/captive_portal/'],
             ['exclude', '^browser/diagnostics/'],
             ['exclude', '^browser/first_run/'],
+            ['exclude', '^browser/importer/'],
             ['exclude', '^browser/lifetime/'],
             ['exclude', '^browser/media_galleries/'],
             ['exclude', '^browser/net/firefox_*'],
             ['exclude', '^browser/profile_resetter/'],
             ['exclude', '^browser/speech/'],
             ['exclude', '^browser/sync/glue/app_'],
             ['exclude', '^browser/sync/glue/extension_'],
             ['exclude', '^browser/themes/'],
             ['exclude', '^browser/ui/autofill/.*credit_card_bubble.*'],
             ['exclude', '^browser/ui/content_settings/'],
@@ skipping 22 matching lines @@
         ['OS!="android" and OS!="ios" and chromeos==0 and configuration_policy==1', {
           'sources': [
             'browser/net/disk_cache_dir_policy_handler_unittest.cc',
           ],
         }],
         ['OS!="android" and OS!="ios" and configuration_policy==1', {
           'sources': [
             'browser/download/download_dir_policy_handler_unittest.cc'
           ],
         }],
-        ['enable_printing!=1', {
-          'sources/': [
-            # No need for service process.
-            ['exclude', '^browser/service/'],
-            ['exclude', '^common/service_'],
-            ['exclude', '^service/'],
-          ],
-        }],
         ['enable_themes==0', {
           'sources!': [
             'browser/ui/webui/theme_source_unittest.cc',
           ],
           'sources/': [
             ['exclude', '^browser/themes/'],
           ],
         }],
         ['toolkit_views==1', {
           'dependencies': [
             '../components/components.gyp:web_modal_test_support',
             '../ui/views/views.gyp:views',
             '../ui/views/views.gyp:views_test_support',
           ],
         }, { # else: toolkit_views == 0
           'sources/': [
             ['exclude', '^browser/ui/views/'],
             ['exclude', '^../apps/ui/views/'],
             ['exclude', '^../ui/views/'],
           ],
         }],
         ['use_nss==0 and use_openssl_certs==0', {
           'sources!': [
             'common/net/x509_certificate_model_unittest.cc',
           ],
         }],
-        ['OS=="android"', {
-          'sources/': [
-            # Android doesn't build the importer.
-            ['exclude', '^browser/importer/'],
-          ],
-        }],
         ['enable_managed_users!=1', {
           'sources/': [
             ['exclude', '^browser/supervised_user/'],
           ],
         }],
         ['safe_browsing==1 and enable_extensions==1', {
           'sources': [
             'browser/extensions/blacklist_unittest.cc',
           ],
         }],
@@ skipping 156 matching lines @@
               'dependencies': [
                 '../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
               ],
             }],
           ],
         },
       ],
     }],
   ],  # 'conditions'
 }