Remove -DPREFIX from win64 yasm_compile.gypi to fix ffmpeg linking

Remove -DPREFIX from win64 yasm_compile.gypi to fix ffmpeg linking

BUG=171542, 171721, 166496
TEST=ffmpegsumo target builds on win64 with private fix to disable ffmpeg compiler warning 4267 on win64


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

[wolenetz, 2013-01-22 23:31:19]

I don't have a working libvpx build (or private), so I don't know if this
removal of -DPREFIX fixes ffmpeg but perhaps breaks libvpx on win64.

jschuh: please provide details on how libvpx can be privately built prior to
this change and review
fgalligan & scherkus: please review

[wolenetz, 2013-01-22 23:39:58]

https://codereview.chromium.org/12059002/diff/1/third_party/yasm/yasm_compile...
File third_party/yasm/yasm_compile.gypi (left):

https://codereview.chromium.org/12059002/diff/1/third_party/yasm/yasm_compile...
third_party/yasm/yasm_compile.gypi:73: '-DPREFIX',
try bots are failing update.  I'll sync my client, rebase the patch and submit
PS2 shortly.

[fgalligan1, 2013-01-23 18:24:47]

lgtm

[wolenetz, 2013-01-23 18:30:13]

jschuh & scherkus: please review

Thanks!

[scherkus (not reviewing), 2013-01-23 19:06:24]

rubber stamp lgtm (but I know nothing about yasm gyp!)

[jschuh, 2013-01-23 19:20:18]

On 2013/01/23 18:30:13, wolenetz wrote:
> jschuh & scherkus: please review
> 
> Thanks!

lgtm. If Andrew doesn't respond soon I doubt he'd object if you TBR him and CQ.

[jschuh, 2013-01-23 19:20:54]

On 2013/01/23 19:20:18, Justin Schuh wrote:
> lgtm. If Andrew doesn't respond soon I doubt he'd object if you TBR him and
CQ.

nm. took me 15 mins to click send.

[commit-bot: I haz the power, 2013-01-23 19:25:52]

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

[commit-bot: I haz the power, 2013-01-23 21:55:36]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-01-23 22:22:31]

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

[commit-bot: I haz the power, 2013-01-23 23:44:29]

Change committed as 178429