Revert "Add support for closing Android Shell instances from Java."

content/shell/android/java/src/org/chromium/content_shell/ShellManager.java

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.content_shell;
 
 import android.content.Context;
 import android.util.AttributeSet;
 import android.view.LayoutInflater;
 import android.widget.FrameLayout;
 
 import org.chromium.base.CalledByNative;
 import org.chromium.base.JNINamespace;
-import org.chromium.base.ThreadUtils;
 import org.chromium.content.browser.ContentView;
 import org.chromium.content.browser.ContentViewRenderView;
 import org.chromium.ui.base.WindowAndroid;
 
 /**
  * Container and generator of ShellViews.
  */
 @JNINamespace("content")
 public class ShellManager extends FrameLayout {
 
@@ skipping 51 matching lines @@
      */
     public Shell getActiveShell() {
         return mActiveShell;
     }
 
     /**
      * Creates a new shell pointing to the specified URL.
      * @param url The URL the shell should load upon creation.
      */
     public void launchShell(String url) {
-        ThreadUtils.assertOnUiThread();
         nativeLaunchShell(url);
     }
 
     /**
      * Enter or leave overlay video mode.
      * @param enabled Whether overlay mode is enabled.
      */
     public void setOverlayVideoMode(boolean enabled) {
         if (mContentViewRenderView == null) return;
         mContentViewRenderView.setOverlayVideoMode(enabled);
     }
 
     @SuppressWarnings("unused")
     @CalledByNative
-    private Object createShell(long nativeShellPtr) {
+    private Object createShell() {
         assert mContentViewRenderView != null;
         LayoutInflater inflater =
                 (LayoutInflater) getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
         Shell shellView = (Shell) inflater.inflate(R.layout.shell_view, null);
-        shellView.initialize(nativeShellPtr, mWindow);
+        shellView.setWindow(mWindow);
 
-        if (mActiveShell != null) mActiveShell.close();
+        if (mActiveShell != null) closeShell(mActiveShell);
 
         shellView.setContentViewRenderView(mContentViewRenderView);
         addView(shellView, new FrameLayout.LayoutParams(
                 FrameLayout.LayoutParams.MATCH_PARENT, FrameLayout.LayoutParams.MATCH_PARENT));
         mActiveShell = shellView;
         ContentView contentView = mActiveShell.getContentView();
         if (contentView != null) {
             mContentViewRenderView.setCurrentContentView(contentView);
             contentView.onShow();
         }
 
         return shellView;
     }
 
     @SuppressWarnings("unused")
     @CalledByNative
-    private void removeShell(Shell shellView) {
+    private void closeShell(Shell shellView) {
         if (shellView == mActiveShell) mActiveShell = null;
         ContentView contentView = shellView.getContentView();
         if (contentView != null) contentView.onHide();
         shellView.setContentViewRenderView(null);
+        shellView.setWindow(null);
         removeView(shellView);
     }
 
     private static native void nativeInit(Object shellManagerInstance);
     private static native void nativeLaunchShell(String url);
 }