Chromium Code Reviews| 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 a29a3a3ddbff64826fa767220a2da37c890e60e3..7bb8602bb2b4a6dcd28be56822d031758589ea20 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 |
| @@ -418,14 +418,17 @@ public class ContentVideoView extends FrameLayout |
| @CalledByNative |
| private static ContentVideoView createContentVideoView( |
| - Context context, long nativeContentVideoView, ContentVideoViewClient client) { |
| + ContentViewCore contentViewCore, long nativeContentVideoView) { |
| ThreadUtils.assertOnUiThread(); |
| // The context needs be Activity to create the ContentVideoView correctly. |
| + Context context = contentViewCore.getContext(); |
| if (!isActivityContext(context)) { |
| Log.e(TAG, "Wrong type of context, can't create fullscreen video"); |
| return null; |
| } |
| - ContentVideoView videoView = new ContentVideoView(context, nativeContentVideoView, client); |
| + ContentVideoViewClient client = contentViewCore.getContentVideoViewClient(); |
| + ContentVideoView videoView = new ContentVideoView(context, nativeContentVideoView, |
| + contentViewCore.getContentVideoViewClient()); |
|
no sievers
2014/10/15 18:16:38
nit: contentViewCore.getContentVideoViewClient() -
Ignacio Solla
2014/10/15 18:38:42
Done.
|
| client.enterFullscreenVideo(videoView); |
| return videoView; |
| } |