MSE: Always use new FrameProcessor's coded frame processing

This change ignores Blink's signalling of whether or not to use
LegacyFrameProcessor, and instead always uses the new, compliant,
FrameProcessor to handle SourceBuffer coded frame processing.

R=acolwell@chromium.org
TEST=Linux Debug: no media_unittest or MSE layout test regression; Linux Release with proprietary codecs: no regression of yt-dash-mse-test ToT
BUG=249422

Review URL: https://codereview.chromium.org/307263003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274468 0039d316-1c4b-4281-b951-d872f2087c98

content/renderer/media/webmediasource_impl.cc

@@ -43,19 +43,12 @@ WebMediaSource::AddStatus WebMediaSourceImpl::addSourceBuffer(
   for (size_t i = 0; i < codecs.size(); ++i)
     new_codecs[i] = codecs[i].utf8().data();
 
-  bool use_legacy_frame_processor = false;
-  switch (frame_processor_choice) {
-    case UseLegacyFrameProcessor:
-      use_legacy_frame_processor = true;
-      break;
-    case UseNewFrameProcessor:
-      break;
-  }
-
+  // Ignore |frame_processor_choice|. Instead, request the new FrameProcessor.
+  // TODO(wolenetz): Remove |frame_processor_choice| and LegacyFrameProcessor
+  // once the new FrameProcessor has stabilized. See http://crbug.com/249422.
   WebMediaSource::AddStatus result =
       static_cast<WebMediaSource::AddStatus>(
-          demuxer_->AddId(id, type.utf8().data(), new_codecs,
-                          use_legacy_frame_processor));
+          demuxer_->AddId(id, type.utf8().data(), new_codecs, false));
 
   if (result == WebMediaSource::AddStatusOk)
     *source_buffer = new WebSourceBufferImpl(id, demuxer_);