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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java

Issue 652673002: NOT FOR REVIEW: Modify ToggleFullscreenMode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@fixPowerBlockerNonMedia
Patch Set: Rebase Created 6 years, 2 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: content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java b/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
index d2387a312a98ad38b5e355fb8a4553445acfeb5d..50d00ece387cd5154b158fbca308c47e06db0615 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
@@ -419,8 +419,10 @@ public class ContentVideoView extends FrameLayout
Log.e(TAG, "Wrong type of context, can't create fullscreen video");
return null;
}
- return new ContentVideoView(context, nativeContentVideoView,
- contentViewCore.getContentVideoViewClient());
+ ContentVideoViewClient client = contentViewCore.getContentVideoViewClient();
+ ContentVideoView videoView = new ContentVideoView(context, nativeContentVideoView, client);
+ client.enterFullscreenVideo(videoView);
+ return videoView;
}
private static boolean isActivityContext(Context context) {
@@ -472,6 +474,9 @@ public class ContentVideoView extends FrameLayout
if (mVideoSurfaceView != null) {
removeSurfaceView();
setVisibility(View.GONE);
+
+ // To prevent re-entrance, call this after removeSurfaceView.
+ mClient.exitFullscreenVideo();
}
if (nativeViewDestroyed) {
mNativeContentVideoView = 0;
« no previous file with comments | « content/common/view_messages.h ('k') | content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698