Add psutil build step to fix pyauto media issues. Upgrade psutil to 0.4.0.

chrome/chrome_tests.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.
 {
   'variables' : {
     'pyautolib_sources': [
       'app/chrome_command_ids.h',
       'app/chrome_dll_resource.h',
       'common/automation_constants.h',
       'common/pref_names.cc',
@@ skipping 425 matching lines @@
           'dependencies': [
             # Needed for the following #include chain:
             #   test/base/run_all_unittests.cc
             #   test/base/chrome_test_suite.h
             #   gtk/gtk.h
             '../build/linux/system.gyp:gtk',
           ],
         }],
       ],
     },
     {

[Nirnimesh, 2011/12/08 00:23:41]

I forgot to point out earlier: the gyp rule for psutil should be in third_party,
in its own file.
Then you can depend on it with: ../third_party/psutil/psutil.gyp:psutil

[DaleCurtis, 2011/12/08 22:04:38]

Hurm. So I'm having trouble getting this working without warnings and errors
being thrown.

I had to set 'type': 'none' and I'm still getting a "ninja: WARNING: multiple
rules generate psutil. build will not be correct; continuing anyway" error. Any
ideas?

[Nirnimesh, 2011/12/08 22:40:56]

Can you upload your latest CL?
You could take other packages in third_party as examples. say libjpeg

[DaleCurtis, 2011/12/08 23:03:36]

Done. I looked there, but psutil is a unique in that it generates objects that
aren't supposed to be linked with anything. With a type other than none gyp
tries to link the .so's against pyautolib.

+      'target_name': 'test_support_psutil',
+      'type': 'static_library',
+      'actions': [
+        {
+          'action_name': 'psutil_build',
+          'inputs': [
+            '../third_party/psutil/psutil/_psutil_bsd.c',
+            '../third_party/psutil/psutil/_psutil_common.c',
+            '../third_party/psutil/psutil/_psutil_linux.c',
+            '../third_party/psutil/psutil/_psutil_mswindows.c',
+            '../third_party/psutil/psutil/_psutil_osx.c',
+            '../third_party/psutil/psutil/_psutil_posix.c',
+            '../third_party/psutil/psutil/_psutil_bsd.h',
+            '../third_party/psutil/psutil/_psutil_common.h',
+            '../third_party/psutil/psutil/_psutil_linux.h',
+            '../third_party/psutil/psutil/_psutil_mswindows.h',
+            '../third_party/psutil/psutil/_psutil_osx.h',
+            '../third_party/psutil/psutil/_psutil_posix.h',

[Nirnimesh, 2011/12/08 00:23:41]

please include setup.py as well, since we want to re-build if it changes.

[DaleCurtis, 2011/12/08 22:04:38]

Done.

+          ],
+          'outputs': [
+            '<(PRODUCT_DIR)/psutil',
+            '<(PRODUCT_DIR)/psutil/_compat.py',
+            '<(PRODUCT_DIR)/psutil/_psosx.py',
+            '<(PRODUCT_DIR)/psutil/__init__.py',
+            '<(PRODUCT_DIR)/psutil/_pslinux.py',
+            '<(PRODUCT_DIR)/psutil/_common.py',
+            '<(PRODUCT_DIR)/psutil/error.py',
+            '<(PRODUCT_DIR)/psutil/_psmswindows.py',
+            '<(PRODUCT_DIR)/psutil/_psbsd.py',
+            '<(PRODUCT_DIR)/psutil/_psposix.py',
+          ],
+          'conditions': [
+            ['OS=="linux"', {
+              'outputs': [
+                '<(PRODUCT_DIR)/_psutil_linux.so',
+                '<(PRODUCT_DIR)/_psutil_posix.so',
+              ],
+            },],

[Nirnimesh, 2011/12/08 00:23:41]

nit: first , not necessary

[DaleCurtis, 2011/12/08 22:04:38]

Done.

+            ['OS=="mac"', {
+              'outputs': [
+                '<(PRODUCT_DIR)/_psutil_osx.so',
+                '<(PRODUCT_DIR)/_psutil_posix.so',
+              ],
+            },],
+            ['OS=="win"', {
+              'outputs': [
+                '<(PRODUCT_DIR)/_psutil_mswindows.pyd',
+              ],
+            },],
+          ],
+          'action': ['python',
+                     '../third_party/psutil/build.py',
+                     '<(PRODUCT_DIR)'
+          ],
+          'message': 'Generating psutil binary',

[Nirnimesh, 2011/12/08 00:23:41]

s/binary/binaries/

[DaleCurtis, 2011/12/08 22:04:38]

Done.

+        },
+      ],  # actions
+    },
+    {
       'target_name': 'automated_ui_tests',
       'type': 'executable',
       'dependencies': [
         'browser',
         'chrome_resources.gyp:theme_resources',
         'renderer',
         'test_support_common',
         'test_support_ui',
         '../base/base.gyp:base',
         '../skia/skia.gyp:skia',
@@ skipping 3565 matching lines @@
           'type': 'loadable_module',
           'product_prefix': '_',
           'dependencies': [
             'chrome',
             'chrome_resources.gyp:chrome_resources',
             'chrome_resources.gyp:chrome_strings',
             'chrome_resources.gyp:theme_resources',
             'debugger',
             'syncapi_core',
             'test_support_common',
+            'test_support_psutil',
             '../skia/skia.gyp:skia',
             '../testing/gtest.gyp:gtest',
           ],
           'export_dependent_settings': [
             'test_support_common',
           ],
           'include_dirs': [
             '..',
           ],
           'cflags': [
@@ skipping 269 matching lines @@
               # Use outputs of this action as inputs for the main target build.
               # Seems as a misnomer but makes this happy on Linux (scons).
               'process_outputs_as_sources': 1,
             },
           ],  # 'actions'
         },
       ]
     }],  # 'coverage!=0'
   ],  # 'conditions'
 }