chrome: Remove perf_tests target.

chrome: Remove perf_tests target.

This is a followup to
https://chromium.googlesource.com/chromium/src/+/4ee38944c67548f1314f9fc1ef1363797b28bf02

BUG=None
TEST=trybots - GYP and GN builds still works.
R=thestig@chromium.org

Committed: https://crrev.com/3ea233235a9a882a3f2f1f0730b540124aba60db
Cr-Commit-Position: refs/heads/master@{#316907}

[tfarina, 2015-02-18 19:17:40]

https://codereview.chromium.org/940643002/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (left):

https://codereview.chromium.org/940643002/diff/1/chrome/chrome_tests.gypi#old...
chrome/chrome_tests.gypi:2896: 'test/perf/url_parse_perftest.cc',
I want to deal with url_parse_perftest.cc in a followup CL.

[Lei Zhang, 2015-02-18 20:25:28]

looking good, but...

https://codereview.chromium.org/940643002/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (left):

https://codereview.chromium.org/940643002/diff/1/chrome/chrome_tests.gypi#old...
chrome/chrome_tests.gypi:2896: 'test/perf/url_parse_perftest.cc',
On 2015/02/18 19:17:40, tfarina wrote:
> I want to deal with url_parse_perftest.cc in a followup CL.

Do you want to deal with it first? If we land this first, we'll have a file
that's not being built.

[tfarina, 2015-02-18 20:28:47]

https://codereview.chromium.org/940643002/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (left):

https://codereview.chromium.org/940643002/diff/1/chrome/chrome_tests.gypi#old...
chrome/chrome_tests.gypi:2896: 'test/perf/url_parse_perftest.cc',
On 2015/02/18 20:25:28, Lei Zhang (Soon to be OOO) wrote:
> On 2015/02/18 19:17:40, tfarina wrote:
> > I want to deal with url_parse_perftest.cc in a followup CL.
> 
> Do you want to deal with it first? If we land this first, we'll have a file
> that's not being built.

Maybe. The problem is that this file does nothing as it is guarded by a #if 0 on
top of it.

https://chromium.googlesource.com/chromium/src/+/master/chrome/test/perf/url_...

[Lei Zhang, 2015-02-18 20:37:19]

On 2015/02/18 20:28:47, tfarina wrote:
> Maybe. The problem is that this file does nothing as it is guarded by a #if 0
on
> top of it.
> 
>
https://chromium.googlesource.com/chromium/src/+/master/chrome/test/perf/url_...

Yes, I looked. Still, it would be nice to not leave a file dangling. For
instance, if we decide to delete it, then we can just squeeze it into this CL.

[tfarina, 2015-02-18 20:46:07]

Yup. Thanks for catching that. ptal.

[Lei Zhang, 2015-02-18 22:14:24]

lgtm

[tfarina, 2015-02-18 22:35:03]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-02-18 22:36:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/940643002/20001

[commit-bot: I haz the power, 2015-02-18 22:56:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-18 22:57:06]

Patchset 2 (id:??) landed as
https://crrev.com/3ea233235a9a882a3f2f1f0730b540124aba60db
Cr-Commit-Position: refs/heads/master@{#316907}