Support "display": "fullscreen" for sites added to the home screen.

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java

 // Copyright 2015 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.webapps;
 
 import android.content.Intent;
 import android.graphics.Bitmap;
 import android.graphics.Color;
 import android.graphics.drawable.Drawable;
@@ skipping 74 matching lines @@
         super.onNewIntent(intent);
 
         WebappInfo newWebappInfo = createWebappInfo(intent);
         if (newWebappInfo == null) {
             Log.e(TAG, "Failed to parse new Intent: " + intent);
             finish();
         } else if (!TextUtils.equals(mWebappInfo.id(), newWebappInfo.id())) {
             mWebappInfo = newWebappInfo;
             resetSavedInstanceState();
             if (mIsInitialized) initializeUI(null);
             // TODO(dominickn): send the web app into fullscreen if mDisplayMode is

[dominickn, 2017/01/18 05:56:17]

Please delete this comment - it's not needed any more. :)

[Leo, 2017/01/18 23:03:12]

Acknowledged.

             // WebDisplayMode.Fullscreen. See crbug.com/581522
         }
     }
 
     protected boolean isInitialized() {
         return mIsInitialized;
     }
 
     protected WebappInfo createWebappInfo(Intent intent) {
         return (intent == null) ? WebappInfo.createEmpty() : WebappInfo.create(intent);
     }
 
     private void initializeUI(Bundle savedInstanceState) {
         // We do not load URL when restoring from saved instance states.
         if (savedInstanceState == null && mWebappInfo.isInitialized()) {
             if (TextUtils.isEmpty(getActivityTab().getUrl())) {
                 getActivityTab().loadUrl(new LoadUrlParams(
                         mWebappInfo.uri().toString(), PageTransition.AUTO_TOPLEVEL));
             }
         } else {
             if (NetworkChangeNotifier.isOnline()) getActivityTab().reloadIgnoringCache();
         }
 
         getActivityTab().addObserver(createTabObserver());
-        getActivityTab().getTabWebContentsDelegateAndroid().setDisplayMode(
-                WebDisplayMode.Standalone);
-        // TODO(dominickn): send the web app into fullscreen if mDisplayMode is
-        // WebDisplayMode.Fullscreen. See crbug.com/581522
+
+        // Add Fullscreen support and the other modes will fallback to Standalone.

[dominickn, 2017/01/18 05:56:17]

Nit: the comment here probably isn't necessary (the code is very descriptive).
Also, in the future there will also be display:minimal-ui. That currently falls
back to a browser tab.

[Leo, 2017/01/18 23:03:12]

Acknowledged.

+        if (mWebappInfo.displayMode() == WebDisplayMode.Fullscreen) {
+            getActivityTab().getTabWebContentsDelegateAndroid().setDisplayMode(
+                    WebDisplayMode.Fullscreen);
+            enterImmersiveMode(getWindow().getDecorView());
+        } else {
+            getActivityTab().getTabWebContentsDelegateAndroid().setDisplayMode(
+                    WebDisplayMode.Standalone);
+        }
     }
 
     @Override
     public void preInflationStartup() {
         WebappInfo info = createWebappInfo(getIntent());
 
         String id = "";
         if (info != null) {
             mWebappInfo = info;
             id = info.id();
@@ skipping 449 matching lines @@
         return new WebappDelegateFactory(this);
     }
 
     // We're temporarily disable CS on webapp since there are some issues. (http://crbug.com/471950)
     // TODO(changwan): re-enable it once the issues are resolved.
     @Override
     protected boolean isContextualSearchAllowed() {
         return false;
     }
 }