Enable -Wall on Linux as well.

Enable -Wall on Linux as well.

BUG=312287
TBR=dglazkov

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=160985

[lgombos, 2013-10-30 20:09:50]

All warnings are fixed for Linux as well. Enable -Wall for Linux just like for
MacOS.

[Nico, 2013-10-30 20:24:38]

Lgtm
On Oct 30, 2013 1:09 PM, <l.gombos@samsung.com> wrote:

> Reviewers: Nico, Dimitri Glazkov,
>
> Message:
> All warnings are fixed for Linux as well. Enable -Wall for Linux just like
> for
> MacOS.
>
>
> Description:
> Enable -Wall on Linux as well.
>
> BUG=312287
>
> Please review this at
https://codereview.chromium.**org/53223002/<https://codereview.chromium.org/5...
>
> SVN Base:
https://chromium.googlesource.**com/chromium/blink.git@master<https://chromiu...
>
> Affected files (+1, -1 lines):
>   M Source/config.gyp
>
>
> Index: Source/config.gyp
> diff --git a/Source/config.gyp b/Source/config.gyp
> index 7763dc6d6d65d7ce79abe046d07060**7e4faf1682..**
> 5ed67e1449607332e9f816108a8e40**01c1488a1b 100644
> --- a/Source/config.gyp
> +++ b/Source/config.gyp
> @@ -89,7 +89,7 @@
>            # timesNewRoman.unstatic.3258 and colorTransparent.unstatic.**
> 4879.
>            'cflags': ['-Wno-uninitialized'],
>          }],
> -        ['OS=="mac"', {
> +        ['OS=="mac" or OS=="linux"', {
>            # TODO: Turn this on on all platforms, http://crbug.com/312287
>            'variables': {
>              'chromium_code': 1,
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-10-30 20:34:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/53223002/1

[commit-bot: I haz the power, 2013-10-30 20:51:01]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[commit-bot: I haz the power, 2013-10-30 20:55:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/53223002/1

[commit-bot: I haz the power, 2013-10-30 23:11:14]

Change committed as 160985