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

Unified Diff: media/video/capture/mac/avfoundation_glue.mm

Issue 443783003: Turn off AVFoundation field trial (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
Index: media/video/capture/mac/avfoundation_glue.mm
diff --git a/media/video/capture/mac/avfoundation_glue.mm b/media/video/capture/mac/avfoundation_glue.mm
index 551f759df59679b5998e2a4583edd7c2a0ad85bc..cbcfc6f196e374e2a50a4b172c589ec9ee610fe8 100644
--- a/media/video/capture/mac/avfoundation_glue.mm
+++ b/media/video/capture/mac/avfoundation_glue.mm
@@ -9,7 +9,6 @@
#include "base/command_line.h"
#include "base/lazy_instance.h"
#include "base/mac/mac_util.h"
-#include "base/metrics/field_trial.h"
#include "media/base/media_switches.h"
namespace {
@@ -125,9 +124,7 @@ bool AVFoundationGlue::IsAVFoundationSupported() {
// crbug.com/396764
// TODO(vrk): Does this really need to be static?
static bool should_enable_avfoundation =
- command_line->HasSwitch(switches::kEnableAVFoundation) ||
- base::FieldTrialList::FindFullName("AVFoundationMacVideoCapture")
- == "Enabled";
+ command_line->HasSwitch(switches::kEnableAVFoundation);
andresp-chromium 2014/08/06 06:50:33 Why remove the finch experiment here?
vrk (LEFT CHROMIUM) 2014/08/07 01:25:00 My thinking was that if we never wanted to turn on
// Try to load AVFoundation. Save result in static bool to avoid loading
// AVFoundationBundle every call.
static bool loaded_successfully = [AVFoundationBundle() load];

Powered by Google App Engine
This is Rietveld 408576698