Ignore C4267 size_t to int truncations in win64 media_unittests

Ignore C4267 size_t to int truncations in win64 media_unittests


BUG=173697, 166496
TEST=media_unittests target builds without C4267 errors. With other build errors fixed in privates (fix win64 gmock_mutant template functor redefinitions & lift MEDIA...INTRINSICS... definition into yuv_convert.h and use it to not link in methods that use MMX intrinsics), media_unittests executes


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

[wolenetz, 2013-02-05 22:46:43]

Please review.

Thanks,
Matt

[DaleCurtis, 2013-02-05 22:51:31]

Are there too many of these to fix right away?

[wolenetz, 2013-02-05 23:19:25]

On 2013/02/05 22:51:31, DaleCurtis wrote:
> Are there too many of these to fix right away?

There are 27 instances of C4267 in media_unittests without suppressing it for
win64.  With this many, I would prefer to follow the route of standing up a
build/bots first, then fixing regressions (and the root cause of the size_t/int
truncations) once that data is coming in.

[DaleCurtis, 2013-02-05 23:33:28]

Since you've already got one suppression in, this lgtm, but 27 seems small
enough to fix relatively quickly.

[wolenetz, 2013-02-05 23:53:02]

On 2013/02/05 23:33:28, DaleCurtis wrote:
> Since you've already got one suppression in, this lgtm, but 27 seems small
> enough to fix relatively quickly.

Thanks Dale.  I'll prioritize the real fix (171009), but get this checked in now
to unblock standing up bots.

[commit-bot: I haz the power, 2013-02-05 23:55:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wolenetz@chromium.org/12213027/1

[commit-bot: I haz the power, 2013-02-06 02:46:09]

Change committed as 180874