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

Unified Diff: android_webview/native/aw_settings.cc

Issue 2299883007: Remove video overlay support from WebView (Closed)
Patch Set: Removed get/setVideoOverlayForEmbeddedEncryptedVideoEnabled() from AwSettings 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 5f0745d0249719300dffa3875782841a9fc85894..5680f02ce8ab1f37a9376f171e22fbcab90a2a92 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