Factor out a perf test result printer method.

chrome/chrome_browser.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': [
     {
       'target_name': 'browser',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
@@ skipping 2473 matching lines @@
             'common/extensions/api/api.gyp:api',
             'debugger',
             'installer_util',
             'sync_file_system_drive_proto',
             'sync_file_system_proto',
             '../cc/cc.gyp:cc',
             '../components/components.gyp:autofill_content_browser',
             '../components/components.gyp:navigation_interception',
             '../net/net.gyp:net_with_v8',
             '../printing/printing.gyp:printing',
+            '../testing/perf/perf_test.gyp:*',

[Nico, 2013/08/27 03:49:03]

This is a bit unfortunate. I don't think we want to have 'browser' depend on a
target in the testing/ folder. testing/ looks too much like test-only code, so
people will add stuff to this target, and then it ends up being compiled into
your shipping binary.

Is this dependency needed? What for?

[tonyg, 2013/08/27 04:08:31]

Oh jeez, this is ridiculous. I've just learned that apparently we link the page
cyclers into the chrome binary to support a --visit-urls chrome flag which runs
the page cyclers.

This never should have landed and I'd like to rip it out:
https://chromiumcodereview.appspot.com/10386134

For the purposes of this patch, I'm not making the dependency any worse. Would
you be okay if I landed this patch as-is then put together a subsequent patch to
rip the page cyclers out of the chrome binary.

[Nico, 2013/08/27 15:09:21]

Sounds good, please add a comment with a link to the bug you filed to this line
too (if it's still needed after your other CL).

[tonyg, 2013/08/27 15:23:59]

Done.

             '../third_party/adobe/flash/flash_player.gyp:flapper_version_h',
             '../third_party/expat/expat.gyp:expat',
             '../third_party/hunspell/hunspell.gyp:hunspell',
             '../third_party/leveldatabase/leveldatabase.gyp:leveldatabase',
             '../third_party/libphonenumber/libphonenumber.gyp:libphonenumber',
             '../third_party/libusb/libusb.gyp:libusb',
             '../third_party/libyuv/libyuv.gyp:libyuv',
             '../third_party/npapi/npapi.gyp:npapi',
             '../third_party/re2/re2.gyp:re2',
             '../third_party/webrtc/modules/modules.gyp:desktop_capture',
@@ skipping 998 matching lines @@
             'package_name': 'org/chromium/chrome/browser/ui/toolbar',
             'template_deps': ['browser/ui/toolbar/toolbar_model_security_level_list.h'],
           },
           'includes': [ '../build/android/java_cpp_template.gypi' ],
         },
       ],
      },
     ],
   ],
 }