Increase warning level for windows

Some of the warnings that are disabled on windows are not actually trigger any failures and can be simply enabled without changing any source code.

It is our intent to keep the warning level in blink close to the to the default chromium warning level, so we should re-enable these warnings for windows.

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

[lgombos, 2013-11-06 21:49:19]

the failures on the try seems to be unrelated..

[dglazkov, 2013-11-06 21:54:30]

lgtm, I believe you.

[Nico, 2013-11-06 22:06:50]

Some of these might be needed for 2012 / 2013 (which the bots don't cover at
this point I believe). Please wait for scottmg's ok before you land.

[lgombos, 2013-11-06 22:24:12]

On 2013/11/06 22:06:50, Nico (ooo until Nov 12) wrote:
> Some of these might be needed for 2012 / 2013 (which the bots don't cover at
> this point I believe). Please wait for scottmg's ok before you land.

Got it.

[scottmg, 2013-11-06 23:08:05]

On 2013/11/06 22:24:12, lgombos wrote:
> On 2013/11/06 22:06:50, Nico (ooo until Nov 12) wrote:
> > Some of these might be needed for 2012 / 2013 (which the bots don't cover at
> > this point I believe). Please wait for scottmg's ok before you land.
> 
> Got it.

Possibly, but since we don't have bots it's too annoying to not know. Go ahead
and we can add any back in later if necessary.

[lgombos, 2013-11-06 23:11:53]

On 2013/11/06 23:08:05, scottmg wrote:
> On 2013/11/06 22:24:12, lgombos wrote:
> > On 2013/11/06 22:06:50, Nico (ooo until Nov 12) wrote:
> > > Some of these might be needed for 2012 / 2013 (which the bots don't cover
at
> > > this point I believe). Please wait for scottmg's ok before you land.
> > 
> > Got it.
> 
> Possibly, but since we don't have bots it's too annoying to not know. Go ahead
> and we can add any back in later if necessary.

Is there a particularly good time to land this ? Are we right after the branch
point now ?

[Nico, 2013-11-06 23:18:26]

Now is fine.


On Wed, Nov 6, 2013 at 3:11 PM, <l.gombos@samsung.com> wrote:

> On 2013/11/06 23:08:05, scottmg wrote:
>
>> On 2013/11/06 22:24:12, lgombos wrote:
>> > On 2013/11/06 22:06:50, Nico (ooo until Nov 12) wrote:
>> > > Some of these might be needed for 2012 / 2013 (which the bots don't
>> cover
>>
> at
>
>> > > this point I believe). Please wait for scottmg's ok before you land.
>> >
>> > Got it.
>>
>
>  Possibly, but since we don't have bots it's too annoying to not know. Go
>> ahead
>> and we can add any back in later if necessary.
>>
>
> Is there a particularly good time to land this ? Are we right after the
> branch
> point now ?
>
> https://codereview.chromium.org/63073002/
>

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-11-06 23:22:19]

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

[commit-bot: I haz the power, 2013-11-07 01:50:31]

Retried try job too often on win_blink_rel for step(s) blink_platform_unittests,
webkit_tests, webkit_unit_tests, weborigin_unittests, wtf_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[lgombos, 2013-11-07 02:30:00]

Dependent patch reverted.. will need to wait with this a bit more..
https://codereview.chromium.org/61733005

[commit-bot: I haz the power, 2013-11-09 06:17:52]

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

[commit-bot: I haz the power, 2013-11-09 07:35:02]

Change committed as 161685