MSE: Remove Chromium-side support for LegacyFrameProcessor

MSE: Remove Chromium-side support for LegacyFrameProcessor

This is the second of a three-sided change to remove
LegacyFrameProcessor from Blink and Chromium now that the new
FrameProcessor is stabilized.

R=acolwell@chromium.org
BUG=249422
TEST=No media_unittest or MSE layout test regression locally on Linux

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

[wolenetz, 2014-06-03 02:57:42]

In anticipation of hopefully removing LegacyFrameProcessor soon:
PTAL @ PS1
acolwell@: OWNERS review for everything
dalecurtis@: just FYI this removal is on its way hopefully soon

See part 1/3 (https://codereview.chromium.org/310733002/) for a description of
this 3-sided change sequence.

I think merging FrameProcessorBase into FrameProcessor is distinct enough to be
done in a later CL. wdyt?

[wolenetz, 2014-06-03 20:00:20]

On 2014/06/03 02:57:42, wolenetz wrote:
> In anticipation of hopefully removing LegacyFrameProcessor soon:
> PTAL @ PS1
> acolwell@: OWNERS review for everything
> dalecurtis@: just FYI this removal is on its way hopefully soon
> 
> See part 1/3 (https://codereview.chromium.org/310733002/) for a description of
> this 3-sided change sequence.
> 
> I think merging FrameProcessorBase into FrameProcessor is distinct enough to
be
> done in a later CL. wdyt?

I'll have a rebase uploaded soon as PS2 due to
https://codereview.chromium.org/278673003, which added a new pipeline
integration test that needs to be de-parameterized like the rest of them in this
CL.

[wolenetz, 2014-06-03 20:13:36]

PTAL @ PS2. I'm fine with review not beginning until next Monday (6/9/2014), to
give the non-legacy frame processor some bake time on trunk.

[wolenetz, 2014-06-09 20:34:03]

PTAL @ PS3 (just a rebase). Intent is to land this by EOW or early next week if
still no regressions seen with new FrameProcessor.

[acolwell GONE FROM CHROMIUM, 2014-06-13 18:26:42]

lgtm

https://codereview.chromium.org/315483002/diff/40001/media/filters/chunk_demu...
File media/filters/chunk_demuxer.cc (right):

https://codereview.chromium.org/315483002/diff/40001/media/filters/chunk_demu...
media/filters/chunk_demuxer.cc:1169: scoped_ptr<FrameProcessorBase>
frame_processor(
s/Base//?

[wolenetz, 2014-06-16 20:01:38]

Thanks for review. Landing this pends first landing Blink CL
https://codereview.chromium.org/310733002/ and rolling Blink DEPS (and any
further rebasing that's required meanwhile...)

https://codereview.chromium.org/315483002/diff/40001/media/filters/chunk_demu...
File media/filters/chunk_demuxer.cc (right):

https://codereview.chromium.org/315483002/diff/40001/media/filters/chunk_demu...
media/filters/chunk_demuxer.cc:1169: scoped_ptr<FrameProcessorBase>
frame_processor(
On 2014/06/13 18:26:42, acolwell wrote:
> s/Base//?

Done. I'm leaving the merger of FrameProcessorBase and FrameProcessor to a later
CL.

[wolenetz, 2014-06-18 21:07:55]

The CQ bit was checked by wolenetz@chromium.org

[commit-bot: I haz the power, 2014-06-18 21:11:41]

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

[commit-bot: I haz the power, 2014-06-19 03:20:52]

Change committed as 278255