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

Unified Diff: android_webview/native/aw_settings.cc

Issue 2299883007: Remove video overlay support from WebView (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
Index: android_webview/native/aw_settings.cc
diff --git a/android_webview/native/aw_settings.cc b/android_webview/native/aw_settings.cc
index 77892fd59a0e8101b5edeb55f59101a100af638f..f369a22f5b607209140686e4fa6e3b916635fcda 100644
--- a/android_webview/native/aw_settings.cc
+++ b/android_webview/native/aw_settings.cc
@@ -212,18 +212,6 @@ void AwSettings::UpdateRendererPreferencesLocked(
update_prefs = true;
}
- bool video_overlay =
- Java_AwSettings_getVideoOverlayForEmbeddedVideoEnabledLocked(env, obj);
- bool force_video_overlay =
- Java_AwSettings_getForceVideoOverlayForTests(env, obj);
- if (video_overlay !=
- prefs->use_video_overlay_for_embedded_encrypted_video ||
- force_video_overlay != prefs->use_view_overlay_for_all_video) {
- prefs->use_video_overlay_for_embedded_encrypted_video = video_overlay;
- prefs->use_view_overlay_for_all_video = force_video_overlay;
- update_prefs = true;
- }
-
if (prefs->accept_languages.compare(
AwContentBrowserClient::GetAcceptLangsImpl())) {
prefs->accept_languages = AwContentBrowserClient::GetAcceptLangsImpl();

Powered by Google App Engine
This is Rietveld 408576698