Delete browser MSE implementation.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 829f6226658c7eee94113739f08051d4ff799fe9..f97e71fce02a0649c45f4ac4cdb68d30434c4c3e 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -793,6 +793,11 @@ bool IsHttpPost(const blink::WebURLRequest& request) {
 // Note that HLS and MP4 detection are pre-redirect and path-based. It is
 // possible to load such a URL and find different content.
 bool UseWebMediaPlayerImpl(const GURL& url) {
+  // Always use WMPI for playing blob URLs since WMPA could never play them very

[ncarter (slow), 2016/09/29 21:17:08]

Is this hunk supposed to be part of this CL?

I don't immediately see how this is connected to the code deletion. There's no
SchemeIsBlob() in the deleted lines of the diff, so this check doesn't appear to
be relocated code.

[DaleCurtis, 2016/09/29 21:55:54]

Yes. All MSE playbacks are blob:// but we still have a chrome://flag for
disabling the unified media pipeline. Without this users who disable via the
flag for testing purposes will lose all MSE playbacks; this whole function is
something that will go away once we can delete the flag and
WebMediaPlayerAndroid.

+  // well and no longer has support for MSE based playbacks.
+  if (url.SchemeIsBlob())
+    return true;
+
   // WMPI does not support HLS.
   if (media::MediaCodecUtil::IsHLSURL(url))
     return false;