Fullscreen webapps are fullscreen so they should be able to control the cutout
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8633462d2d7702777c674e3d040a2001bd763b37 commit 8633462d2d7702777c674e3d040a2001bd763b37 Author: Becca Hughes <beccahughes@chromium.org> Date: Mon Jul 09 21:29:38 2018 [Display Cutout] Do not create DisplayCutoutHostImpl Do not create DisplayCutoutHostImpl automatically in WebContentsImpl BUG= 859241 Change-Id: I1dff20d653d1712f529f595c3fb9d0ce08eaceeb Reviewed-on: https://chromium-review.googlesource.com/1129557 Reviewed-by: Bo <boliu@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#573451} [modify] https://crrev.com/8633462d2d7702777c674e3d040a2001bd763b37/content/browser/web_contents/web_contents_impl.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bda2c4df785721b35cdf789f89d6a3291a97a889 commit bda2c4df785721b35cdf789f89d6a3291a97a889 Author: Becca Hughes <beccahughes@chromium.org> Date: Tue Jul 10 00:54:12 2018 [Display Cutout] Support for fullscreen webapps When a webapp is has the fullscreen display mode set the fullscreen API is disabled and the activity is in an "immersive" mode. We should tell WebContents when this happens so we can support display cutouts in this mode. BUG= 859241 Change-Id: I4ce0a647180aa72f2eb5361ad8335956ecb383a2 Reviewed-on: https://chromium-review.googlesource.com/1121537 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by: Bo <boliu@chromium.org> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#573568} [modify] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/chrome/android/java_sources.gni [modify] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/chrome/android/javatests/src/org/chromium/chrome/browser/display_cutout/DisplayCutoutTest.java [modify] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/chrome/android/javatests/src/org/chromium/chrome/browser/display_cutout/DisplayCutoutTestRule.java [add] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/chrome/android/javatests/src/org/chromium/chrome/browser/display_cutout/WebappDisplayCutoutTest.java [add] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/chrome/android/javatests/src/org/chromium/chrome/browser/display_cutout/WebappDisplayCutoutTestRule.java [modify] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/content/browser/display_cutout_browsertest.cc [modify] https://crrev.com/bda2c4df785721b35cdf789f89d6a3291a97a889/content/browser/web_contents/web_contents_impl.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/05a2ce154e311843ac6301b365a5161695a68dc5 commit 05a2ce154e311843ac6301b365a5161695a68dc5 Author: Becca Hughes <beccahughes@chromium.org> Date: Wed Jul 18 17:49:27 2018 [Display Cutout] Remove support for standalone It looks like standalone apps will still show the system UI (and should) so they will be blocked by the Android OS from extending into the display cutout. Therefore, we should not give them default control of the cutout BUG= 859241 Change-Id: Ie26d41fe983c1f45e4177676a3f936a03ca3a07c Reviewed-on: https://chromium-review.googlesource.com/1134080 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#576126} [modify] https://crrev.com/05a2ce154e311843ac6301b365a5161695a68dc5/chrome/android/javatests/src/org/chromium/chrome/browser/display_cutout/WebappDisplayCutoutTest.java [modify] https://crrev.com/05a2ce154e311843ac6301b365a5161695a68dc5/content/browser/display_cutout/display_cutout_browsertest.cc [modify] https://crrev.com/05a2ce154e311843ac6301b365a5161695a68dc5/content/browser/display_cutout/display_cutout_host_impl.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d11d650ce098aa271d168b5de822042cb7abea87 commit d11d650ce098aa271d168b5de822042cb7abea87 Author: Becca Hughes <beccahughes@chromium.org> Date: Tue Jul 31 17:01:28 2018 [Display Cutout] Pull out WebContentsObserver Remove WebContentsObserver from DisplayCutoutHostImpl BUG= 859241 Change-Id: Icd79a7ae476afe5c5b1e4a778a93b7f411d512cf Reviewed-on: https://chromium-review.googlesource.com/1130224 Reviewed-by: Bo <boliu@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#579439} [modify] https://crrev.com/d11d650ce098aa271d168b5de822042cb7abea87/content/browser/display_cutout/display_cutout_host_impl.cc [modify] https://crrev.com/d11d650ce098aa271d168b5de822042cb7abea87/content/browser/display_cutout/display_cutout_host_impl.h [modify] https://crrev.com/d11d650ce098aa271d168b5de822042cb7abea87/content/browser/web_contents/web_contents_impl.cc [modify] https://crrev.com/d11d650ce098aa271d168b5de822042cb7abea87/content/browser/web_contents/web_contents_impl.h
Comment 1 by e...@chromium.org
, Jul 1Labels: -Type-Bug Type-Feature