New issue
Advanced search Search tips

Issue 888154 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 848920
Owner:
Closed: Oct 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Crostini: Terminal is showing for a moment in non-primary user

Project Member Reported by avkodipelli@chromium.org, Sep 21

Issue description

Chrome Version: 70.0.3538.28
Chrome OS Version: 11021.23.0
Chrome OS Platform: All
Network info: WiFi

Please specify Cr-* of the system to which this bug/feature applies (add
the label below).

Steps To Reproduce:
(1) Login first user. Install crosstini.
(2) Open terminal. Sign out user.
(3) login second user.
(4) Add first user from second user using multiuser option .
(5) Observe after login first user.

Expected Result:
- Only primary user has option to crostini.

Actual Result:
- Observing terminal for a moment(flashing) in non-primary user account as shown in below observation video.

How frequently does this problem reproduce? (Always, sometimes, hard to
reproduce?)
Always.

Not sure whether it is really has any impact but filing low priority bug for confirmation.

 
Labels: -Pri-3 -M-70 M-71 Pri-1
Owner: nverne@chromium.org
Status: Assigned (was: Untriaged)
Let's let this sit on 70, fix in 71.
This looks like intended behavior for apps on ChromeOS. We try to restore the apps that were running prior to the last logout. 

We can "fix" by either 
- attempting to launch the container with the app
- killing the terminal's crosh app when a user signs out (which makes it different from every other ChromOS app?)
Mergedinto: 848920
Status: Duplicate (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 8

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

commit e2f0a90ec6e47284f1cb14290b79b137369de3cd
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Thu Nov 08 21:18:36 2018

Introduce Promise::MarkAsHandled

We are migrating streams implementation from JavaScript (v8Extra) to
C++. One of missing features is an ability to set
promise.[[PromiseIsHandled]] to true, used for example in
https://streams.spec.whatwg.org/#rs-pipe-through. This CL introduces
the feature.

Bug:  chromium:894357 ,  chromium:888154 , chromium:902633
Change-Id: If6487b29a74a212761e6d2ef04ef3ca0e6957dce
Reviewed-on: https://chromium-review.googlesource.com/c/1322296
Reviewed-by: Adam Klein <adamk@chromium.org>
Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57370}
[modify] https://crrev.com/e2f0a90ec6e47284f1cb14290b79b137369de3cd/include/v8.h
[modify] https://crrev.com/e2f0a90ec6e47284f1cb14290b79b137369de3cd/src/api.cc
[modify] https://crrev.com/e2f0a90ec6e47284f1cb14290b79b137369de3cd/test/cctest/test-api.cc

Sign in to add a comment