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

Unified Diff: media/base/media.cc

Issue 2349763003: Remove field trial and enable JellyBean for Spitzer! (Closed)
Patch Set: Created 4 years, 3 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/media.cc
diff --git a/media/base/media.cc b/media/base/media.cc
index 9fd89ccc7f02773115a30e7f5f9d93da555c1919..38daf05b8edd18960083a6eec0b38b5f9c298c79 100644
--- a/media/base/media.cc
+++ b/media/base/media.cc
@@ -88,25 +88,8 @@ bool PlatformHasOpusSupport() {
}
bool IsUnifiedMediaPipelineEnabled() {
- // 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 disabled_via_cli =
- base::CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kDisableUnifiedMediaPipeline);
- // TODO(watk, dalecurtis): AVDA has bugs on API level 16 and 17 so it's
- // disabled for now. http://crbug.com/597467
- const bool api_level_supported =
- base::android::BuildInfo::GetInstance()->sdk_int() >= 18;
-
- return !disabled_via_cli && api_level_supported &&
- !base::StartsWith(group_name, "Disabled",
- base::CompareCase::SENSITIVE);
+ return !base::CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kDisableUnifiedMediaPipeline);
}
bool ArePlatformDecodersAvailable() {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698