📺 Move fullscreen web content to a new Activity.

chrome/android/java/src/org/chromium/chrome/browser/FullscreenWebContentsActivity.java

Index: chrome/android/java/src/org/chromium/chrome/browser/FullscreenWebContentsActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/FullscreenWebContentsActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/FullscreenWebContentsActivity.java
new file mode 100644
index 0000000000000000000000000000000000000000..3691d4d7290712476e1a53c5b5314f5256fe5f87
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/FullscreenWebContentsActivity.java
@@ -0,0 +1,104 @@
+// Copyright 2017 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.chrome.browser;
+
+import android.app.Activity;
+import android.content.ComponentName;
+import android.content.Intent;
+import android.provider.Browser;
+
+import org.chromium.base.Log;
+import org.chromium.chrome.R;
+import org.chromium.chrome.browser.fullscreen.ChromeFullscreenManager;
+import org.chromium.chrome.browser.tab.Tab;
+import org.chromium.chrome.browser.tabmodel.AsyncTabParamsManager;
+import org.chromium.chrome.browser.tabmodel.TabReparentingParams;
+import org.chromium.chrome.browser.util.IntentUtils;
+import org.chromium.chrome.browser.webapps.FullScreenActivity;
+
+/**
+ * An Activity used to display fullscreen WebContents.
+ */
+public class FullscreenWebContentsActivity extends FullScreenActivity {
+    private static final String TAG = "FullWebConActivity";
+
+    @Override
+    protected Tab createTab() {
+        assert getIntent().hasExtra(IntentHandler.EXTRA_TAB_ID);
+
+        int tabId = IntentUtils.safeGetIntExtra(
+                getIntent(), IntentHandler.EXTRA_TAB_ID, Tab.INVALID_TAB_ID);
+        TabReparentingParams params = (TabReparentingParams) AsyncTabParamsManager.remove(tabId);
+
+        Tab tab = params.getTabToReparent();
+        tab.attachAndFinishReparenting(this, createTabDelegateFactory(), params);
+        return tab;
+    }
+
+    @Override
+    protected int getControlContainerLayoutId() {
+        // TODO(peconn): Determine if there's something more suitable to use here.
+        return R.layout.webapp_control_container;
+    }
+
+    @Override
+    protected ChromeFullscreenManager createFullscreenManager() {
+        // Create a Fullscreen manager that won't change the Tab's fullscreen state when the
+        // Activity ends - we handle leaving fullscreen ourselves.
+        return new ChromeFullscreenManager(this, false, false);
+    }
+
+    @Override
+    public boolean supportsFullscreenActivity() {
+        return true;
+    }
+
+    public static void toggleFullscreenMode(final boolean enableFullscreen, final Tab tab) {
+        if (tab.getFullscreenManager() == null) {
+            Log.w(TAG, "Cannot toggle fullscreen, manager is null.");
+            return;
+        }
+
+        if (tab.getFullscreenManager().getTab() == tab) {
+            tab.getFullscreenManager().setTab(null);
+        }
+
+        Runnable setFullscreen = new Runnable() {
+            @Override
+            public void run() {
+                // The Tab's FullscreenManager changes when it is moved.
+                tab.getFullscreenManager().setTab(tab);
+                tab.toggleFullscreenMode(enableFullscreen);
+            }
+        };
+
+        Intent intent = new Intent();
+        Activity activity = tab.getActivity();
+
+        if (enableFullscreen) {
+            // Send to the FullscreenWebContentsActivity.
+            intent.setClass(tab.getActivity(), FullscreenWebContentsActivity.class);
+
+            intent.putExtra(IntentHandler.EXTRA_PARENT_COMPONENT, activity.getComponentName());
+            intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
+            intent.addFlags(Intent.FLAG_ACTIVITY_MULTIPLE_TASK);

[Theresa, 2017/04/26 18:32:11]

Why do we need multiple task? Can there be multiple fullscreen activities at
once?

[PEConn, 2017/04/27 08:53:11]

I was thinking of the multiwindow case - we want to allow both
ChromeTabbedActivity's to show fullscreen content.

[Theresa, 2017/04/27 15:32:25]

I think that this flag will allow an unlimited number of tasks to be created,
when we want at most two.  I think it's fine to leave for now, but we should do
some testing around multi-window and fullscreen activities.

+        } else {
+            // Send back to the Activity it came from.
+            ComponentName parent = IntentUtils.safeGetParcelableExtra(
+                    activity.getIntent(), IntentHandler.EXTRA_PARENT_COMPONENT);
+            if (parent != null) {
+                intent.setComponent(parent);
+            } else {
+                Log.d(TAG, "Cannot return fullscreen tab to parent Activity.");
+                intent.setClass(activity, ChromeTabbedActivity.class);

[Theresa, 2017/04/26 18:32:11]

We should probably send this back through ChromeLauncherActivity if we can't get
the parent component.

[PEConn, 2017/04/27 08:53:11]

Done.

+            }
+
+            // TODO(peconn): Deal with tricky multiwindow scenarios.
+        }
+        intent.putExtra(Browser.EXTRA_APPLICATION_ID, activity.getPackageName());
+
+        tab.detachAndStartReparenting(intent, null, setFullscreen);
+    }
+}