Disble 4351 warnings on msvs compiler

Disble 4351 warnings on msvs compiler

Chromium is actually disabling additional warnings see:
http://src.chromium.org/viewvc/chrome/trunk/src/build/common.gypi?view=markup#l4447

do we want to disable any of these as well?

R=kmillikin@google.com

Committed: https://code.google.com/p/dart/source/detail?r=26864

[Kevin Millikin (Google), 2013-08-29 11:24:50]

LGTM, but I'd add a comment about what the warning is and especially why we
disabled it.

Suggested text:

C4351 warns MSVC follows the C++ specification regarding array initialization in
member initializers.  Code that expects the specified behavior should silence
this warning.

[ricow1, 2013-08-29 11:27:32]

On 2013/08/29 11:24:50, kmillikin wrote:
> LGTM, but I'd add a comment about what the warning is and especially why we
> disabled it.
> 
> Suggested text:
> 
> C4351 warns MSVC follows the C++ specification regarding array initialization
in
> member initializers.  Code that expects the specified behavior should silence
> this warning.

Done, also added this to the debug target

[ricow1, 2013-08-29 12:22:10]

Committed patchset #2 manually as r26864 (presubmit successful).