| Index: components/external_video_surface/android/java/src/org/chromium/components/external_video_surface/ExternalVideoSurfaceContainer.java
|
| diff --git a/components/external_video_surface/android/java/src/org/chromium/components/external_video_surface/ExternalVideoSurfaceContainer.java b/components/external_video_surface/android/java/src/org/chromium/components/external_video_surface/ExternalVideoSurfaceContainer.java
|
| index 739252faa88fccb47361d4c4acd9539f4090631a..68b7e72324e1662353f2e451ac5dc60ed14923fd 100644
|
| --- a/components/external_video_surface/android/java/src/org/chromium/components/external_video_surface/ExternalVideoSurfaceContainer.java
|
| +++ b/components/external_video_surface/android/java/src/org/chromium/components/external_video_surface/ExternalVideoSurfaceContainer.java
|
| @@ -14,7 +14,6 @@ import android.view.ViewGroup;
|
| import org.chromium.base.VisibleForTesting;
|
| import org.chromium.base.annotations.CalledByNative;
|
| import org.chromium.base.annotations.JNINamespace;
|
| -import org.chromium.content.browser.ContainerViewObserver;
|
| import org.chromium.content.browser.ContentViewCore;
|
| import org.chromium.content.browser.RenderCoordinates;
|
|
|
| @@ -69,7 +68,6 @@ public class ExternalVideoSurfaceContainer implements SurfaceHolder.Callback {
|
| private final long mNativeExternalVideoSurfaceContainer;
|
| private final ContentViewCore mContentViewCore;
|
| private ViewGroup mContainerView;
|
| - private ContainerViewObserver mContainerViewObserver;
|
| private int mPlayerId = INVALID_PLAYER_ID;
|
| private SurfaceView mSurfaceView;
|
|
|
| @@ -185,36 +183,23 @@ public class ExternalVideoSurfaceContainer implements SurfaceHolder.Callback {
|
| private void createSurfaceView() {
|
| assert mSurfaceView == null;
|
| assert mContainerView == null;
|
| - assert mContainerViewObserver == null;
|
|
|
| mSurfaceView = new NoPunchingSurfaceView(mContentViewCore.getContext());
|
| mSurfaceView.getHolder().addCallback(this);
|
| - // SurfaceHoder.surfaceCreated() will be called after the SurfaceView is attached to
|
| + // SurfaceHolder.surfaceCreated() will be called after the SurfaceView is attached to
|
| // the Window and becomes visible.
|
| mContainerView = mContentViewCore.getContainerView();
|
| mContainerView.addView(mSurfaceView);
|
| - mContainerViewObserver = new ContainerViewObserver() {
|
| - @Override
|
| - public void onContainerViewChanged(ViewGroup newContainerView) {
|
| - mContainerView.removeView(mSurfaceView);
|
| - mContainerView = newContainerView;
|
| - mContainerView.addView(mSurfaceView);
|
| - }
|
| - };
|
| - mContentViewCore.addContainerViewObserver(mContainerViewObserver);
|
| }
|
|
|
| private void removeSurfaceView() {
|
| assert mSurfaceView != null;
|
| assert mContainerView != null;
|
| - assert mContainerViewObserver != null;
|
|
|
| - // SurfaceHoder.surfaceDestroyed() will be called in ViewGroup.removeView()
|
| + // SurfaceHolder.surfaceDestroyed() will be called in ViewGroup.removeView()
|
| // as soon as the SurfaceView is detached from the Window.
|
| - mContentViewCore.removeContainerViewObserver(mContainerViewObserver);
|
| mContainerView.removeView(mSurfaceView);
|
|
|
| - mContainerViewObserver = null;
|
| mSurfaceView = null;
|
| mContainerView = null;
|
| }
|
| @@ -309,4 +294,3 @@ public class ExternalVideoSurfaceContainer implements SurfaceHolder.Callback {
|
| private native void nativeSurfaceDestroyed(
|
| long nativeExternalVideoSurfaceContainerImpl, int playerId);
|
| }
|
| -
|
|
|