Only copy libffmpegsumo if build_ffmpegsumo is 1.

Only copy libffmpegsumo if build_ffmpegsumo is 1.

BUG=none
TEST=Build with build_libffmpegsumo=0 in ~/.gyp/include.gypi. Build should give no errors.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=49982
Reverted

[awong, 2010-06-16 18:07:33]

LGTM

[Mark Mentovai, 2010-06-16 18:25:13]

http://codereview.chromium.org/2839006/diff/1/2
File chrome/chrome_dll.gypi (right):

http://codereview.chromium.org/2839006/diff/1/2#newcode527
chrome/chrome_dll.gypi:527: ['build_ffmpegsumo', {
Fix CL description: build_ffmpegsumo vs. build_libffmpegsumo.

http://codereview.chromium.org/2839006/diff/1/2#newcode527
chrome/chrome_dll.gypi:527: ['build_ffmpegsumo', {
Where does the default value come from?

[Nico, 2010-06-16 18:29:42]

http://codereview.chromium.org/2839006/diff/1/2
File chrome/chrome_dll.gypi (right):

http://codereview.chromium.org/2839006/diff/1/2#newcode527
chrome/chrome_dll.gypi:527: ['build_ffmpegsumo', {
On 2010/06/16 18:25:13, Mark Mentovai wrote:
> Fix CL description: build_ffmpegsumo vs. build_libffmpegsumo.

Too late :-/

http://codereview.chromium.org/2839006/diff/1/2#newcode527
chrome/chrome_dll.gypi:527: ['build_ffmpegsumo', {
On 2010/06/16 18:25:13, Mark Mentovai wrote:
> Where does the default value come from?

third_party/ffmpeg/ffmpeg.gyp

[Nico, 2010-06-16 19:35:19]

http://codereview.chromium.org/2839006/diff/1/2
File chrome/chrome_dll.gypi (right):

http://codereview.chromium.org/2839006/diff/1/2#newcode527
chrome/chrome_dll.gypi:527: ['build_ffmpegsumo', {
I used `git try`, but due to
http://code.google.com/p/chromium/issues/detail?id=46697 it didn't send what i
wanted it to, and i attributed the "update failed" to that. What could go wrong,
right? (even more than even you expected!)

On 2010/06/16 18:25:13, Mark Mentovai wrote:
> Where does the default value come from?

Is there a good way to do check build_ffmpegsumo then?