The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c0643ee1bacea9a40d8b6554e3a39dad6a7e8919

commit c0643ee1bacea9a40d8b6554e3a39dad6a7e8919
Author: Ivan Sandrk <isandrk@google.com>
Date: Thu Sep 28 12:20:11 2017

[BUGFIX] Show a toast when an app automatically goes fullscreen in public session

Currently, a toast shows only when using the html5 fullscreen API when an app
goes fullscreen in ChromeOS public session. It should also be shown when an app
uses the extension windowing API to go fullscreen. Fix by including
FULLSCREEN_TYPE_WINDOW_API in the types that trigger the toast and update tests.

Bug: chromium:769138
Test: See bug
Change-Id: If1dacf1edbe577ed50fd2ab2c994cd094b71f2df
Reviewed-on: https://chromium-review.googlesource.com/686894
Commit-Queue: Ivan Šandrk <isandrk@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504974}
[modify] https://crrev.com/c0643ee1bacea9a40d8b6554e3a39dad6a7e8919/chrome/browser/ui/views/apps/chrome_native_app_window_views_aura_ash.cc
[modify] https://crrev.com/c0643ee1bacea9a40d8b6554e3a39dad6a7e8919/chrome/browser/ui/views/apps/chrome_native_app_window_views_aura_ash.h
[modify] https://crrev.com/c0643ee1bacea9a40d8b6554e3a39dad6a7e8919/chrome/browser/ui/views/apps/chrome_native_app_window_views_aura_ash_interactive_uitest.cc

Thanks for reporting this Jing, should be fixed now!

isandrk@

When tested in M62.0.3202.43 9901.35.0 dev celes, I did not see any notification when launching the Enterprise DeviceAttributes Test Extension app as it automatically went full screen in public session.  I tried both manual and auto launch the app in public session with the same result.

I wonder if this notification comes in a popup balloon or something.  Thanks!

As verified in M63.0.3230.0 10001.0.0 dev daisy, a pop-up notification is seen upon launching app in full screen mode.

This bug requires manual review: We are only 7 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Approved for 62. 

I'd merge this, but this fix CL is dependant on another CL that is not in M62.

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

@Raj, Would #c10 block the launch of the Full Screen mode in Public Sessions feature for M62. 

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

This issue hasn't been updated in the last 6 weeks, so removing its merge approval label. Please re-request a merge if needed.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot