Make the gyp files more cross-platform across the Unices

chrome/chrome_common.gypi

 # 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.
 
 {
   'target_defaults': {
     'variables': {
       'chrome_common_target': 0,
     },
     'target_conditions': [
@@ skipping 238 matching lines @@
         'common/web_apps.cc',
         'common/web_apps.h',
         'common/web_resource/web_resource_unpacker.cc',
         'common/web_resource/web_resource_unpacker.h',
         'common/worker_thread_ticker.cc',
         'common/worker_thread_ticker.h',
         'common/zip.cc',  # Requires zlib directly.
         'common/zip.h',
       ],
       'conditions': [
-        ['OS=="linux" or OS=="freebsd" or OS=="openbsd"', {
+        ['toolkit_uses_gtk==1', {

[Evan Stade, 2011/05/17 00:46:12]

wow, we are really inconsistent about putting spaces around operators in gyp
files. But I think we do want them as per the python style guidelines.

           'dependencies': [
             '../build/linux/system.gyp:gtk',
           ],
           'export_dependent_settings': [
             '../third_party/sqlite/sqlite.gyp:sqlite',
           ],
           'link_settings': {
             'libraries': [
               '-lX11',
               '-lXrender',
               '-lXss',
               '-lXext',
             ],
           },
         },],
-        [ 'OS == "linux" or OS == "freebsd" or OS == "openbsd" or OS == "solaris"', {
+        [ 'os_posix==1 and OS!="mac"', {
           'include_dirs': [
             '<(SHARED_INTERMEDIATE_DIR)',
           ],
           # Because posix_version generates a header, we must set the
           # hard_dependency flag.
           'hard_dependency': 1,
           'actions': [
             {
               'action_name': 'posix_version',
               'variables': {
@@ skipping 94 matching lines @@
         'chrome_strings',
         '../app/app.gyp:app_base',
         '../base/base.gyp:base',
         '../gpu/gpu.gyp:gpu_ipc',
         '../net/net.gyp:net_resources',
         '../net/net.gyp:net',
         '../third_party/icu/icu.gyp:icui18n',
         '../third_party/icu/icu.gyp:icuuc',
       ],
       'conditions': [
-        [ 'OS == "linux" or OS == "freebsd" or OS == "openbsd"', {
+        [ 'os_posix==1 and OS!="mac"', {

[Evan Stade, 2011/05/17 00:46:12]

I guess to be consistent there should be no space between [ and ' (here and
elsewhere)

             'conditions': [
               ['use_openssl==1', {
                  'dependencies': [
                    '../third_party/openssl/openssl.gyp:openssl',
                  ],
                },
                { # else !use_openssl
                 'dependencies': [
                   '../build/linux/system.gyp:nss',
                 ],
@@ skipping 133 matching lines @@
           'configurations': {
             'Common_Base': {
               'msvs_target_platform': 'x64',
             },
           },
         },
       ],
     }],
   ],
 }