Blacklist once.unicornmedia.com from Spitzer playback on Android.

content/renderer/render_frame_impl.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 812 matching lines @@
   if (base::EndsWith(url.path(), ".mp4",
                      base::CompareCase::INSENSITIVE_ASCII) &&
       !media::HasPlatformDecoderSupport()) {
     return false;
   }
 
   // Indicates if the Android MediaPlayer should be used instead of WMPI.
   if (GetContentClient()->renderer()->ShouldUseMediaPlayerForURL(url))
     return false;
 
+  // Blacklist the following domains since they are sending content that we
+  // can't identify as HLS until redirects are followed.
+  // TODO(tguibert): Remove once http://crbug.com/619729 is fixed.
+  if (url.DomainIs("once.unicornmedia.com"))

[ncarter (slow), 2016/06/13 23:19:04]

My gut reaction is that this is too hacky to approve. A web browser shouldn't
have any hardcoded references to particular sites like this.

[DaleCurtis, 2016/06/13 23:25:47]

Does your opinion change if you think of this more as an experimental opt-out
for the new path?

+    return false;
+
   // Otherwise enable WMPI if indicated via experiment or command line.
   return media::IsUnifiedMediaPipelineEnabled();
 }
 #endif  // defined(OS_ANDROID)
 
 #if defined(ENABLE_MOJO_CDM)
 // Returns whether mojo CDM should be used at runtime. Note that even when mojo
 // CDM is enabled at compile time (ENABLE_MOJO_CDM is defined), there are cases
 // where we want to choose other CDM types. For example, on Android when we use
 // WebMediaPlayerAndroid, we still want to use ProxyMediaKeys. In the future,
@@ skipping 5392 matching lines @@
   // event target. Potentially a Pepper plugin will receive the event.
   // In order to tell whether a plugin gets the last mouse event and which it
   // is, we set |pepper_last_mouse_event_target_| to null here. If a plugin gets
   // the event, it will notify us via DidReceiveMouseEvent() and set itself as
   // |pepper_last_mouse_event_target_|.
   pepper_last_mouse_event_target_ = nullptr;
 #endif
 }
 
 }  // namespace content