tree 61b22eada74ffcb2cf7150dc04d8f325a2972eaf
parent 50536661c7bba31321eb0bec514a587a849a1018
author Tim Van Patten <timvp@google.com> 1692719410 +0000
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> 1693588308 +0000

Reland "Don't destroy splash screen in daemonize mode"

This reverts commit bcbe1eb04186b0f2c6a0b9a8aa8e5232989f720c.

Reason for revert: CL is fixed with term_deactivate() call added,
so frecon doesn't consume key presses when the terminal is
backgrounded.

TEST=Manual verification the brightness buttons behave as expected in
VT1 and VT2 on Nocturne device. Note that frecon's keymap doesn't match
the devices, so the FN keys being offset is expected.

Original change's description:
> Revert "Don't destroy splash screen in daemonize mode"
>
> This reverts commit b75a2f832b1cd24d52c433dcfe6338ab2468196d.
>
> Reason for revert: b/293308798: Button presses are still consumed when
> the splash screen is not destroyed, breaking things like brightness
> changes.
>
> Original change's description:
> > Don't destroy splash screen in daemonize mode
> >
> > Keep the splash screen in daemonize mode until the login prompt is
> > visible, so it can be drawn to by the chargesplash utility.
> >
> > BUG=b:228370390
> > TEST=Manual verification
> >
> > Change-Id: I9402830f35f92e9f412baeabbfed96c653e1561a
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/frecon/+/4523598
> > Reviewed-by: Dominik Behr <dbehr@chromium.org>
> > Tested-by: Tim Van Patten <timvp@google.com>
> > Commit-Queue: Tim Van Patten <timvp@google.com>
>
> BUG=b:228370390
> BUG=b:293308798
>
> Change-Id: Ica33d0b0caaa18c0eb6a91b9858cd0a61565c2b9
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/frecon/+/4763855
> Reviewed-by: Ross Zwisler <zwisler@google.com>
> Reviewed-by: Stéphane Marchesin <marcheu@chromium.org>
> Commit-Queue: Tim Van Patten <timvp@google.com>
> Tested-by: Tim Van Patten <timvp@google.com>

BUG=b:228370390
BUG=b:293308798

Change-Id: I5c14f60d516cfdd571f4fc292c0ad79bc92ff484
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/frecon/+/4802507
Reviewed-by: Dominik Behr <dbehr@chromium.org>
Reviewed-by: Ross Zwisler <zwisler@google.com>
Tested-by: Tim Van Patten <timvp@google.com>
Commit-Queue: Tim Van Patten <timvp@google.com>