New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 729701 link

Starred by 3 users

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: On profile error, UI shows nothing but the wallpaper after login

Project Member Reported by afakhry@chromium.org, Jun 5 2017

Issue description

- Trigger a manual profile error by running:
    echo "sjkhdgasjdhdfgadjhgfdajshgd" > /home/chronos/Default/History
- Reboot
- Login and observe that the UI is empty.

Expected:
  - After login, you get the profile error dialog as in screenshot_1.

Actual:
  - UI is empty [screenshot_2]

Bisecting ... Culprtit CL is https://codereview.chromium.org/2891223002. Assigning to xiyuan.
 
screenshot_1.png
521 KB View Download
screenshot_2.png
915 KB View Download
Labels: M-60
The profile error dialog runs a nested message loop. It seems to me that mojo messages are not delivered when there is a nested message loop. As a result, ACTIVE session state is not delivered to ash. Hence the wallpaper is not moved.

Comment 4 by xiy...@chromium.org, Jun 14 2017

Labels: Merge-Request-60
Project Member

Comment 5 by sheriffbot@chromium.org, Jun 15 2017

Labels: -Merge-Request-60 Hotlist-Merge-Review Merge-Review-60
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-60 Merge-Approved-60
Project Member

Comment 7 by sheriffbot@chromium.org, Jun 23 2017

Cc: josa...@chromium.org
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

Comment 8 by xiy...@chromium.org, Jun 23 2017

Labels: -Hotlist-Merge-Review -Merge-Approved-60 merge-merged-3112
Status: Fixed (was: Assigned)
Merged to branch 3112 in https://codereview.chromium.org/2951503002/

Comment 9 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment