Exclude duplicate gmock_mutant functor templates in WIN64

Exclude duplicate gmock_mutant functor templates in WIN64

On WIN64, there is only one calling convention.  Therefore,
implicit calling conventions and __stdcall are the same, hence
template redefinition errors occur in the previous gmock_mutant.h
on WIN64.

BUG=173697, 166496
TEST=media_unittests target builds without template redefinition errors.  With other build errors fixed in privates (ignore C4267 & 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=181348

[wolenetz, 2013-02-05 21:03:00]

Stoyan@: please review all
DaleCurtis@: please review relative to media_unittests usage & python
readability

Thanks,
Matt

[DaleCurtis, 2013-02-05 21:07:38]

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant.py
File testing/generate_gmock_mutant.py (right):

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:2: # Copyright (c) 2013 The Chromium Authors.
All rights reserved.
We don't update copyright date on changes anymore.

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:10: // Copyright (c) 2013 The Chromium Authors.
All rights reserved.
This one is probably okay.

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:423: print "#if defined (OS_WIN) and !defined
(ARCH_CPU_X86_64)"
s/and/&&/

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:440: print "#endif  // defined (OS_WIN) and
!defined (ARCH_CPU_X86_64)\n"
Ditto.

[wolenetz, 2013-02-05 21:38:46]

Please review PS2.  The items in "TEST=" have been confirmed.

Thanks,
Matt

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant.py
File testing/generate_gmock_mutant.py (right):

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:2: # Copyright (c) 2013 The Chromium Authors.
All rights reserved.
On 2013/02/05 21:07:38, DaleCurtis wrote:
> We don't update copyright date on changes anymore.

Done.

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:10: // Copyright (c) 2013 The Chromium Authors.
All rights reserved.
On 2013/02/05 21:07:38, DaleCurtis wrote:
> This one is probably okay.

Done.

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:423: print "#if defined (OS_WIN) and !defined
(ARCH_CPU_X86_64)"
On 2013/02/05 21:07:38, DaleCurtis wrote:
> s/and/&&/

Whoops!  Thanks for catching this.

https://codereview.chromium.org/12208028/diff/1/testing/generate_gmock_mutant...
testing/generate_gmock_mutant.py:440: print "#endif  // defined (OS_WIN) and
!defined (ARCH_CPU_X86_64)\n"
On 2013/02/05 21:07:38, DaleCurtis wrote:
> Ditto.

Done.

[DaleCurtis, 2013-02-05 21:40:24]

lgtm

[wolenetz, 2013-02-06 18:44:41]

On 2013/02/05 21:40:24, DaleCurtis wrote:
> lgtm

I received LGTM in email from stoyan@.  I removed stoyan@ from reviewers and
will use CQ to commit.

[commit-bot: I haz the power, 2013-02-06 18:45:19]

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

[wolenetz, 2013-02-07 00:05:38]

On 2013/02/06 18:45:19, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/wolenetz%40chromium.org/12208028/6001

I'm not sure why no new bots are being attempted in last couple hours.  I
cleared the commit checkbox and will attempt CQ again in a bit.

[commit-bot: I haz the power, 2013-02-07 02:03:59]

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

[commit-bot: I haz the power, 2013-02-07 17:06:00]

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

[commit-bot: I haz the power, 2013-02-07 19:20:22]

Change committed as 181348