Fix disabling FFMpeg video decoders on non-Android platforms

Fix disabling FFMpeg video decoders on non-Android platforms

If disable_ffmpeg_video_decoders is set to true on platfoms other
than Android ffmped_video_decoder.* is always included.
This patch removes those sources and fixes dependencies in code.

Review-Url: https://codereview.chromium.org/2808093008
Cr-Commit-Position: refs/heads/master@{#464394}
Committed: https://chromium.googlesource.com/chromium/src/+/ab15c62867108abf92582f3f6c41a766b10324c0

[rduszynski, 2017-04-12 14:40:04]

Description was changed from

==========
Fix disabling FFMpeg video decoders on non-Android platforms

If disable_ffmpeg_video_decoders is set to true on platfoms other
than Android ffmped_video_decoder.* is always included.
This patch removes those sources and fixes dependencies in code.

BUG=
==========

to

==========
Fix disabling FFMpeg video decoders on non-Android platforms

If disable_ffmpeg_video_decoders is set to true on platfoms other
than Android ffmped_video_decoder.* is always included.
This patch removes those sources and fixes dependencies in code.
==========

[rduszynski, 2017-04-12 14:40:04]

rduszynski@opera.com changed reviewers:
+ jam@chromium.org

[rduszynski, 2017-04-12 14:59:32]

@reviewer Some changes were made with introduction of
disable_ffmpeg_video_decoders, however not all non-Android specific paths were
included

[jam, 2017-04-12 17:25:49]

On 2017/04/12 14:59:32, rduszynski wrote:
> @reviewer Some changes were made with introduction of
> disable_ffmpeg_video_decoders, however not all non-Android specific paths were
> included

lgtm for content, please get a media/owner review as well

[rduszynski, 2017-04-12 17:49:23]

rduszynski@opera.com changed reviewers:
+ hubbe@chromium.org

[rduszynski, 2017-04-12 17:55:46]

On 2017/04/12 17:25:49, jam wrote:
> On 2017/04/12 14:59:32, rduszynski wrote:
> > @reviewer Some changes were made with introduction of
> > disable_ffmpeg_video_decoders, however not all non-Android specific paths
were
> > included
> 
> lgtm for content, please get a media/owner review as well

Added.

FYI 'git cl owners' lists you as an owner of media too (probably due to /OWNERS)

[chromium-reviews, 2017-04-12 18:34:36]

On vacation this week. Try someone else (or wait until next week.)

    /Hubbe

On Wed, Apr 12, 2017 at 10:55 AM, <rduszynski@opera.com> wrote:

> On 2017/04/12 17:25:49, jam wrote:
> > On 2017/04/12 14:59:32, rduszynski wrote:
> > > @reviewer Some changes were made with introduction of
> > > disable_ffmpeg_video_decoders, however not all non-Android specific
> paths
> were
> > > included
> >
> > lgtm for content, please get a media/owner review as well
>
> Added.
>
> FYI 'git cl owners' lists you as an owner of media too (probably due to
> /OWNERS)
>
> https://codereview.chromium.org/2808093008/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[DaleCurtis, 2017-04-12 18:37:38]

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

[DaleCurtis, 2017-04-12 18:37:39]

lgtm, though I have the vague feeling this was done for some way that I can't
remember.

[rduszynski, 2017-04-13 11:33:24]

The CQ bit was checked by rduszynski@opera.com

[commit-bot: I haz the power, 2017-04-13 11:33:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-13 11:33:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-13 11:33:38]

The author rduszynski@opera.com has not signed Google Contributor License
Agreement. Please visit https://cla.developers.google.com to sign and manage
CLA.

[Mostyn Bramley-Moore, 2017-04-13 12:23:31]

On 2017/04/13 11:33:38, commit-bot: I haz the power wrote:
> The author mailto:rduszynski@opera.com has not signed Google Contributor
License
> Agreement. Please visit https://cla.developers.google.com to sign and manage
> CLA.

I believe there's a corporate CLA that covers *@opera.com - I also added
rduszynski@opera.com to the authorized-chromium-contributors-list@opera.com
google group...

[Mostyn Bramley-Moore, 2017-04-13 12:23:48]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2017-04-13 12:24:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-13 14:00:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492086228288590, "parent_rev":
"50ddc321d12aa3b47c5e39331e5a2d4a4466226c", "commit_rev":
"ab15c62867108abf92582f3f6c41a766b10324c0"}

[commit-bot: I haz the power, 2017-04-13 14:01:31]

Description was changed from

==========
Fix disabling FFMpeg video decoders on non-Android platforms

If disable_ffmpeg_video_decoders is set to true on platfoms other
than Android ffmped_video_decoder.* is always included.
This patch removes those sources and fixes dependencies in code.
==========

to

==========
Fix disabling FFMpeg video decoders on non-Android platforms

If disable_ffmpeg_video_decoders is set to true on platfoms other
than Android ffmped_video_decoder.* is always included.
This patch removes those sources and fixes dependencies in code.

Review-Url: https://codereview.chromium.org/2808093008
Cr-Commit-Position: refs/heads/master@{#464394}
Committed:
https://chromium.googlesource.com/chromium/src/+/ab15c62867108abf92582f3f6c41...
==========

[commit-bot: I haz the power, 2017-04-13 14:01:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ab15c62867108abf92582f3f6c41...