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

Unified Diff: content/browser/renderer_host/render_view_host_impl.cc

Issue 2510353004: Deprecating AutoplayExperimentHelper (Closed)
Patch Set: cleaned up the layout code no longer used Created 4 years, 1 month 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: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index e19736a7ecbf312614899a577d0d931039449b59..a2674f28e11710d67889273fd3e6eecfebe4656d 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -433,22 +433,14 @@ WebPreferences RenderViewHostImpl::ComputeWebkitPrefs() {
// On Android, user gestures are normally required, unless that requirement
// is disabled with a command-line switch or the equivalent field trial is
// is set to "Enabled".
- const std::string autoplay_group_name = base::FieldTrialList::FindFullName(
- "MediaElementAutoplay");
prefs.user_gesture_required_for_media_playback = !command_line.HasSwitch(
- switches::kDisableGestureRequirementForMediaPlayback) &&
- (autoplay_group_name.empty() || autoplay_group_name != "Enabled");
+ switches::kDisableGestureRequirementForMediaPlayback);
prefs.progress_bar_completion = GetProgressBarCompletionPolicy();
prefs.use_solid_color_scrollbars = true;
#endif
- // Handle autoplay gesture override experiment.
- // Note that anything but a well-formed string turns the experiment off.
- prefs.autoplay_experiment_mode = base::FieldTrialList::FindFullName(
- "MediaElementGestureOverrideExperiment");
-
prefs.touch_enabled = ui::AreTouchEventsEnabled();
prefs.device_supports_touch = prefs.touch_enabled &&
ui::GetTouchScreensAvailability() ==
« no previous file with comments | « no previous file | content/public/common/common_param_traits_macros.h » ('j') | tools/metrics/histograms/histograms.xml » ('J')

Powered by Google App Engine
This is Rietveld 408576698