Roll src/third_party/ffmpeg d4b1674:f5698b4

Roll src/third_party/ffmpeg d4b1674:f5698b4

Summary of changes available at:
https://chromium.googlesource.com/chromium/third_party/ffmpeg/+log/d4b1674..f5698b4

BUG=344767
Committed: https://crrev.com/176685b64deddd41f1b4ac40083ad24bc17af564
Cr-Commit-Position: refs/heads/master@{#317706}

[Dirk Pranke, 2015-02-23 23:06:08]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2015-02-23 23:08:22]

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

[sandersd (OOO until July 31), 2015-02-23 23:49:32]

sandersd@chromium.org changed reviewers:
+ sandersd@chromium.org

[sandersd (OOO until July 31), 2015-02-23 23:49:33]

I would prefer using the roll-deps tool to generate the CL description.

lgtm.

[Dirk Pranke, 2015-02-23 23:50:15]

The CQ bit was unchecked by dpranke@chromium.org

[Dirk Pranke, 2015-02-23 23:50:20]

On 2015/02/23 23:49:33, sandersd wrote:
> I would prefer using the roll-deps tool to generate the CL description.
> 
> lgtm.

Oh, sorry, I didn't know such a thing existed. Will re-do.

[sandersd (OOO until July 31), 2015-02-23 23:51:53]

On 2015/02/23 23:50:20, Dirk Pranke wrote:
> On 2015/02/23 23:49:33, sandersd wrote:
> > I would prefer using the roll-deps tool to generate the CL description.
> > 
> > lgtm.
> 
> Oh, sorry, I didn't know such a thing existed. Will re-do.

It's not important enough to be worth re-doing all the trybots, it just makes it
easy to find out what new patches are in the roll.

[ddorwin, 2015-02-23 23:57:53]

On 2015/02/23 23:51:53, sandersd wrote:
> On 2015/02/23 23:50:20, Dirk Pranke wrote:
> > On 2015/02/23 23:49:33, sandersd wrote:
> > > I would prefer using the roll-deps tool to generate the CL description.
> > > 
> > > lgtm.
> > 
> > Oh, sorry, I didn't know such a thing existed. Will re-do.
> 
> It's not important enough to be worth re-doing all the trybots, it just makes
it
> easy to find out what new patches are in the roll.

You could also run the script and just copy the results to the description,
assuming the DEPS change is the same.

[ddorwin, 2015-02-23 23:58:17]

ddorwin@chromium.org changed reviewers:
- ddorwin@chromium.org

[Dirk Pranke, 2015-02-23 23:58:23]

dpranke@chromium.org changed reviewers:
+ ddorwin@chromium.org

[ddorwin, 2015-02-23 23:58:34]

ddorwin@chromium.org changed reviewers:
- ddorwin@chromium.org

[Dirk Pranke, 2015-02-23 23:58:37]

dpranke@chromium.org changed reviewers:
+ ddorwin@chromium.org

[ddorwin, 2015-02-23 23:59:01]

ddorwin@chromium.org changed reviewers:
- ddorwin@chromium.org

[Dirk Pranke, 2015-02-23 23:59:02]

Done. Please take another look?

[sandersd (OOO until July 31), 2015-02-23 23:59:28]

On 2015/02/23 23:59:02, Dirk Pranke wrote:
> Done. Please take another look?

Perfect, lgtm.

[Dirk Pranke, 2015-02-23 23:59:41]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2015-02-24 00:01:31]

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

[commit-bot: I haz the power, 2015-02-24 01:02:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-24 01:03:56]

Patchset 1 (id:??) landed as
https://crrev.com/176685b64deddd41f1b4ac40083ad24bc17af564
Cr-Commit-Position: refs/heads/master@{#317706}