Make follow-up changes per r26302 review comments that were apparently not made

in r26302.  Set Subversion properties, fix the O3D GYP invocation to work
properly, fix presubmit warnings, and do some additional cleanup.
Review URL: http://codereview.chromium.org/200144

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

1 files changed, 17 insertions, 8 deletions

diff --git a/build/gyp_chromium b/build/gyp_chromium
index 017d298..93687ce 100644..100755
--- a/build/gyp_chromium
+++ b/build/gyp_chromium
@@ -1,7 +1,11 @@
 #!/usr/bin/python
 
+# Copyright (c) 2009 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.
+
 # This script is wrapper for Chromium that adds some support for how GYP
-# is invoked by Chromium beyond what can be done it the gclient hooks.
+# is invoked by Chromium beyond what can be done in the gclient hooks.
 
 import glob
 import os
@@ -9,27 +13,32 @@ import shlex
 import sys
 
 print 'Updating projects from gyp files...'
+sys.stdout.flush()
+
+chrome_src = os.path.join(os.path.dirname(sys.argv[0]), os.pardir)
 
 try:
   import gyp
 except ImportError, e:
-  sys.path.append(os.path.join(os.path.dirname(sys.argv[0]), '../tools/gyp/pylib'))
+  sys.path.append(os.path.join(chrome_src, 'tools', 'gyp', 'pylib'))
   import gyp
 
 if __name__ == '__main__':
   args = sys.argv[1:]
 
   # If we didn't get a file, check an env var, and then fall back to
-  # assuming 'src/build/all.gyp'
+  # assuming 'src/build/all.gyp'.  This can't have any backslashes as path
+  # separators even on Windows due to the use of shlex.split.
+  default_gyp_file = 'src/build/all.gyp'
   if len(args) == 0:
     args += shlex.split(os.environ.get('CHROMIUM_GYP_FILE',
-                                       'src/build/all.gyp'))
+                                       default_gyp_file))
+
+  # Always include common.gypi
+  args += ['-I', os.path.join(chrome_src, 'build', 'common.gypi')]
 
-  # Always include gyp_chromium.gypi
-  args += ['-I', os.path.join(os.path.dirname(sys.argv[0]),'common.gypi')]
-  
   # Optionally add supplemental .gypi files if present.
-  supplements = glob.glob('src/*/supplement.gypi')
+  supplements = glob.glob(os.path.join(chrome_src, '*', 'supplement.gypi'))
   for supplement in supplements:
     args += ['-I', supplement]