Fix SF, FF2, and FF3 builds on osx in gears-scons build. OFFICIAL_BUILD=0 now

works as well.
Review URL: http://codereview.chromium.org/10923

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5504 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 65 insertions, 49 deletions

diff --git a/gears/SConscript.browser b/gears/SConscript.browser
index c24c870..090bab7 100644
--- a/gears/SConscript.browser
+++ b/gears/SConscript.browser
@@ -82,28 +82,6 @@ elif env['BROWSER'] in ['FF2', 'FF3']:
     env['GECKO_BASE'] = '$THIRD_PARTY_DIR/gecko_1.9'
 
   env.PrependENVPath('PATH', env.Dir('#/$GECKO_BIN').abspath)
-elif env['BROWSER'] == 'SF':
-  env.Replace(
-# Enable breakpad for Safari on MacOSX.
-      USING_BREAKPAD_OSX = '1')
-      
-  env.Append(
-      CPPDEFINES = [
-# Remove these - During development, it was convenient to have these defined in
-# the Safari port.  Before release we want to clean this up, and replace these
-# with a single BROWSER_SF symbol.
-# We also want to consolidate the include paths, so we don't have to add these
-# paths here.
-          'BROWSER_NPAPI',
-          'BROWSER_WEBKIT',
-          'BROWSER_SAFARI'
-      ],
-      CCFLAGS = [
-# These flags are needed so that instead of exporting all symbols defined in
-# the code, we just export those specifically marked, this reduces the output size.
-          '-fvisibility=hidden'
-      ],
-      CPPPATH = ['$THIRD_PARTY/spidermonkey/nspr/pr/include'])
 
 def MyIdlEmitter(target, source, env):
   """Produce the list of outputs generated by our IDL compiler.  Outputs
@@ -174,22 +152,36 @@ elif env['BROWSER'] == 'NPAPI' and env['OS'] == 'win32':
         'BROWSER_CHROME=1',
       ],
   )
+elif env['BROWSER'] == 'SF':
+  env.Replace(
+# Enable breakpad for Safari on MacOSX.
+      USING_BREAKPAD_OSX = '1')
+      
+  env.Append(
+      CPPDEFINES = [
+# Remove these - During development, it was convenient to have these defined in
+# the Safari port.  Before release we want to clean this up, and replace these
+# with a single BROWSER_SF symbol.
+# We also want to consolidate the include paths, so we don't have to add these
+# paths here.
+          'BROWSER_NPAPI',
+          'BROWSER_WEBKIT',
+          'BROWSER_SAFARI'
+      ],
+      CCFLAGS = [
+# These flags are needed so that instead of exporting all symbols defined in
+# the code, we just export those specifically marked, this reduces the output
+# size.
+          '-fvisibility=hidden'
+      ],
+      CPPPATH = ['$THIRD_PARTY/spidermonkey/nspr/pr/include']
+  )
 
 env.Append(
-    CPPPATH = [
-#        '$LIBPNG_DIR',
-#        '$SKIA_DIR/include',
-#        '$SKIA_DIR/include/corecg',
-#        '$SKIA_DIR/platform',
-    ],
     LIBS = [
-#        'base',
         'googleurl-gears',
-#        env['ICU_LIBS'],  # TODO(sgk):  '$ICU_LIBS' when scons is fixed
-#        'modp_b64',
         'png-gears',
         'sqlite-gears',
-#        'skia',
         'zlib-gears',
     ],
 )
@@ -197,7 +189,6 @@ env.Append(
 if env['OS'] == 'osx':
   env.Append(
       LINKFLAGS = [
-          '-mmacosx-version-min=10.4',
           '-bundle'
       ],
       LIBS = [
@@ -217,6 +208,13 @@ if env['OS'] == 'osx':
           'WebKit',
       ]
   )
+  if not env['OFFICIAL_BUILD']:
+    # For PortAudio:
+    env.Append(FRAMEWORKS = [
+        'CoreAudio',
+        'AudioToolbox',
+        'AudioUnit',
+    ])
 
 # Add in libraries for in-development APIs for non-official builds.
 if not env['OFFICIAL_BUILD']:
@@ -232,11 +230,6 @@ if not env['OFFICIAL_BUILD']:
       'gd',
       'jpeg-gears',
     ])
-  elif env['OS'] == 'osx':
-    env.FilterOut(LIBS = [
-      'gd',
-      'portaudio',
-    ])
 
 if env['BROWSER'] == 'IE':
   if env['OS'] == 'win32':
@@ -278,16 +271,18 @@ if env['BROWSER'] == 'IE':
         ],
     )
 elif env['BROWSER'] in ['FF2', 'FF3']:
-  env.Append(
-      LIBPATH = '$GECKO_LIB',
-      LIBS = [
-          'xpcom',
-          'xpcomglue_s',
-      ],
-  )
+  env.Append(LIBPATH = '$GECKO_LIB')
+
+  if env['BROWSER'] == 'FF2':
+    env.Append(LIBS = '$FF2_LIBS')
+  else:
+    env.Append(LIBS = '$FF3_LIBS')
+
   if env['OS'] == 'win32':
     env.Append(
         LIBS = [
+            'xpcom',
+            'xpcomglue_s',
             'nspr4.lib',
             'js3250.lib',
             'ole32.lib',
@@ -300,10 +295,31 @@ elif env['BROWSER'] in ['FF2', 'FF3']:
         ],
     )
   elif env['OS'] == 'linux':
-    # Although the 1.9 SDK contains libnspr4, it is better to link against
-    # libxul, which in turn depends on libnspr4. In Ubuntu 8.04, libnspr4 was
-    # not listed in /usr/lib, only libxul was.
-    env.Append(LIBS = (env['BROWSER'] == 'FF2' and 'nspr4' or 'xul'))
+    env.Append(
+        LIBS = [
+            'xpcom',
+            'xpcomglue_s',
+        ],
+# Although the 1.9 SDK contains libnspr4, it is better to link against
+# libxul, which in turn depends on libnspr4. In Ubuntu 8.04, libnspr4 was
+# not listed in /usr/lib, only libxul was.
+        FF2_LIBS = ['nspr4'],
+        FF3_LIBS = ['xul'],
+    )
+  elif env['OS'] == 'osx':
+    env.Append(
+        LIBS = [
+            'xpcom',
+            'mozjs',
+            'nspr4',
+            'plds4',
+            'plc4',
+        ],
+        FF2_LIBS = ['xpcom_core'],
+# TODO(mpcomplete): the Makefiles specify many more libs, but we link fine with
+# just this.  Why?
+        FF3_LIBS = ['xpcomglue_s'],
+    )
 elif env['BROWSER'] == 'NPAPI':
   env.Append(
       LIBS = [