Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1074)

Unified Diff: content/renderer/render_frame_impl.cc

Issue 1690063002: Fix mime type mappings when the unified media pipeline is enabled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix browser tests. Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | media/base/media.h » ('j') | media/base/media.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index bb2156a285c74eaaef452b2e81c7f5b534d43756..23733ffecdb378190120ac798a9d87aee20b944b 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -133,6 +133,7 @@
#include "gin/modules/module_registry.h"
#include "media/audio/audio_output_device.h"
#include "media/base/audio_renderer_mixer_input.h"
+#include "media/base/media.h"
#include "media/base/media_log.h"
#include "media/base/media_switches.h"
#include "media/blink/url_index.h"
@@ -2497,22 +2498,7 @@ blink::WebMediaPlayer* RenderFrameImpl::createMediaPlayer(
if (!CanUseWebMediaPlayerImpl(load_type, url)) {
return CreateAndroidWebMediaPlayer(client, encrypted_client, params);
} else if (!MustUseWebMediaPlayerImpl(load_type, url)) {
- // TODO(dalecurtis): This experiment is temporary and should be removed once
- // we have enough data to support the primacy of the unified media pipeline;
- // see http://crbug.com/533190 for details.
- //
- // Note: It's important to query the field trial state first, to ensure that
- // UMA reports the correct group.
- const std::string group_name =
- base::FieldTrialList::FindFullName("UnifiedMediaPipelineTrial");
- const bool enabled_via_cli =
- base::CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kEnableUnifiedMediaPipeline);
- const bool enable_unified_media_pipeline =
- enabled_via_cli ||
- base::StartsWith(group_name, "Enabled", base::CompareCase::SENSITIVE);
-
- if (!enable_unified_media_pipeline)
+ if (!media::IsUnifiedMediaPipelineEnabled())
return CreateAndroidWebMediaPlayer(client, encrypted_client, params);
}
#endif // defined(OS_ANDROID)
« no previous file with comments | « no previous file | media/base/media.h » ('j') | media/base/media.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698