Automatically exclude *_win_browsertest.* from non-Windows builds.

Automatically exclude *_win_browsertest.* from non-Windows builds.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207151

[Nico, 2013-06-18 19:49:23]

lgtm, complimentary bikeshed below?

https://codereview.chromium.org/17009011/diff/1/build/filename_rules.gypi
File build/filename_rules.gypi (right):

https://codereview.chromium.org/17009011/diff/1/build/filename_rules.gypi#new...
build/filename_rules.gypi:13: ['exclude', '_win(_browsertest)?\\.(h|cc)$'],
No need for the ()? bit, since these are already excluded by the previous line.

Or you could change the previous line to _win(_(unit|browser)test)?.

[Nico, 2013-06-18 19:49:46]

s/below\?/below:/


On Tue, Jun 18, 2013 at 12:49 PM, <thakis@chromium.org> wrote:

> lgtm, complimentary bikeshed below?
>
>
> https://codereview.chromium.**org/17009011/diff/1/build/**
>
filename_rules.gypi<https://codereview.chromium.org/17009011/diff/1/build/filename_rules.gypi>
> File build/filename_rules.gypi (right):
>
> https://codereview.chromium.**org/17009011/diff/1/build/**
>
filename_rules.gypi#newcode13<https://codereview.chromium.org/17009011/diff/1/build/filename_rules.gypi#newcode13>
> build/filename_rules.gypi:13: ['exclude',
> '_win(_browsertest)?\\.(h|cc)$**'],
> No need for the ()? bit, since these are already excluded by the
> previous line.
>
> Or you could change the previous line to _win(_(unit|browser)test)?.
>
>
https://codereview.chromium.**org/17009011/<https://codereview.chromium.org/1...
>

[Lei Zhang, 2013-06-18 20:07:40]

I made all the rules consistent.

[commit-bot: I haz the power, 2013-06-18 20:08:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/17009011/6001

[commit-bot: I haz the power, 2013-06-19 03:21:16]

Change committed as 207151