Roll FFmpeg DEPS.

Merge ffmpeg for M39 from upstream 5e3da256184e13502b3428f94e180888436d157b around 10 Sep 2014.

BUG=none
TEST=cq

Committed: https://crrev.com/eab98731cc2a2b23796a45ad06ed2496224e1897
Cr-Commit-Position: refs/heads/master@{#296314}

[jrummell, 2014-09-20 01:09:42]

jrummell@chromium.org changed reviewers:
+ dalecurtis@chromium.org, wolenetz@chromium.org

[jrummell, 2014-09-20 01:09:42]

PTAL.

[DaleCurtis, 2014-09-22 17:12:09]

Looks like you have some checklicenses failures on the bots, but all else looks
good.  It looks like its just a parse failure as the file is attributed with
"GNU Lesser General Public License" which is the same as the rest of FFmpeg.

I'd dig into checklicenses script and see if you need to white list this or fix
a parser.

[wolenetz, 2014-09-22 19:34:27]

looks good for trying merge-m39 on bots; landing will need to include updated
DEPS for the result of merging merge-m39 branch to master on chromium ffmpeg
git.

[jrummell, 2014-09-23 01:53:31]

jrummell@chromium.org changed reviewers:
+ phajdan.jr@chromium.org

[jrummell, 2014-09-23 01:53:32]

Updating to include checklicenses change. phajdan.jr@, please verify the change
in checklicenses.py.

[DaleCurtis, 2014-09-23 17:10:48]

Bots all look good, lets merge into master now.

[jrummell, 2014-09-23 17:50:21]

Updated to use ffmpeg hash from master branch.

[DaleCurtis, 2014-09-23 17:53:59]

lgtm

[jrummell, 2014-09-23 20:57:31]

jrummell@chromium.org changed reviewers:
+ thestig@chromium.org

[jrummell, 2014-09-23 20:57:32]

+thestig@ for OWNERS review of checklicenses.py

[Lei Zhang, 2014-09-23 21:48:07]

I'm not a lawyercat so I am not qualified to say whether ffmpeg with GPLv3 is
acceptable.

nit: Please change the commit message to include some version numbers. "Roll
FFmpeg DEPS" is too vague.

[DaleCurtis, 2014-09-23 21:49:18]

Is this really a lawyercat question? We already have many other third_party
libraries w/ GPLv3.

[Lei Zhang, 2014-09-23 21:52:13]

On 2014/09/23 21:49:18, DaleCurtis wrote:
> Is this really a lawyercat question? We already have many other third_party
> libraries w/ GPLv3.

There are a few, but it's not in the general WHITELISTED_LICENSES section. To
me, that seems like they are exceptions.

[DaleCurtis, 2014-09-23 21:55:44]

Hmm, I think for now we'll just delete the one file which is having issues here
then. We can resolve the license question at a later time when a branch cut
isn't looming.

FFmpeg itself has not changed and we are not passing --enable-version3 to
configure, so we are only using GPLv2 or  lower code.

[Lei Zhang, 2014-09-23 22:09:16]

On 2014/09/23 21:55:44, DaleCurtis wrote:
> Hmm, I think for now we'll just delete the one file which is having issues
here
> then. We can resolve the license question at a later time when a branch cut
> isn't looming.
> 
> FFmpeg itself has not changed and we are not passing --enable-version3 to
> configure, so we are only using GPLv2 or  lower code.

Ok, sounds good. Then presumably there's no checklicenses.py changes and you
won't need my review.

Please update the CL description once you figure out what commit you are rolling
to.

[DaleCurtis, 2014-09-23 22:18:37]

John can you update the CL? efa2e5d00c218551242fa50931cd81332359a509 is the new
hash.

[jrummell, 2014-09-23 22:52:20]

jrummell@chromium.org changed reviewers:
- phajdan.jr@chromium.org, thestig@chromium.org

[jrummell, 2014-09-23 22:52:21]

Updated to new hash that avoids new license requirement.

[jrummell, 2014-09-23 22:54:31]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-09-23 22:55:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/591783002/60001

[commit-bot: I haz the power, 2014-09-24 00:23:55]

Committed patchset #4 (id:60001) as d564b451e1e0284d42fffbf7c3726479f39e19f5

[commit-bot: I haz the power, 2014-09-24 00:24:29]

Patchset 4 (id:??) landed as
https://crrev.com/eab98731cc2a2b23796a45ad06ed2496224e1897
Cr-Commit-Position: refs/heads/master@{#296314}

[wolenetz, 2014-09-24 19:48:25]

belated lgtm.