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

Issue 766834 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug-Regression



Sign in to add a comment

User profile batch keeps showing after teleporting back to the owner's desktop

Project Member Reported by hirono@chromium.org, Sep 19 2017

Issue description

Chrome Version: 60.0.3112.114
OS: Chrome

What steps will reproduce the problem?
(1) Log-in two accounts A and B
(2) Teleport B's browser window to A's desktop
(3) Teleport B's browser window back to B's desktop

What is the expected result?

User profile batch should be disappear when the window gets back to B's desktop 

What happens instead?

User profile batch keeps showing in B's desktop
 

Comment 1 by hirono@chromium.org, Sep 19 2017

Labels: OS-Chrome
Owner: warx@chromium.org
Status: Assigned (was: Untriaged)
Reproducible on TOT 10150.0.0, 64.0.3274.0

Comment 4 by warx@chromium.org, Dec 21 2017

Cc: x...@chromium.org osh...@chromium.org skuhne@chromium.org
Assuming "user profile batch" is the avatar shown on the top left of browser window...

I don't know whether this is a regression or not. cc people to confirm.

Comment 5 by x...@chromium.org, Dec 21 2017

It seems a bug regression to me.

Comment 6 by warx@chromium.org, Dec 21 2017

Owner: sdantul...@chromium.org
Hi Sahiti, could you test an old version like m56 to see if this is a regression?

If this is a regression, you could bounce this back to me. Thanks
Issue reproducible on M56 (9000.91.0, 56.0.2924.110) as well.

Comment 8 by osh...@chromium.org, Dec 21 2017

I think it's a regression but not caught until recently.

Comment 9 by warx@chromium.org, Dec 22 2017

Owner: warx@chromium.org
OK, thanks. Claim back to take a look.

Comment 10 by warx@chromium.org, Dec 22 2017

Labels: -Type-Bug Type-Bug-Regression
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 23 2018

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

commit 18d470b2b461bc014ae98c411ca9f11102676962
Author: Qiang Xu <warx@google.com>
Date: Fri Mar 23 23:22:27 2018

cros: fix avatar still shown on owner desktop

changes:
- Fix regression from crrev.com/1972033002.
- Rename multi_user_window_manager_test.h to
  multi_user_window_manager_test.h.

Bug:  766834 
Test: manual test and added test coverage
Change-Id: I08d9beca388bf71eecb19f9c7b6412c6f8157365
Reviewed-on: https://chromium-review.googlesource.com/977040
Commit-Queue: Qiang Xu <warx@google.com>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545619}
[rename] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/browser/ui/ash/multi_user/test_multi_user_window_manager.cc
[rename] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/browser/ui/ash/multi_user/test_multi_user_window_manager.h
[modify] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/browser/ui/browser_navigator_browsertest_chromeos.cc
[modify] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc
[modify] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
[modify] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash_browsertest.cc
[modify] https://crrev.com/18d470b2b461bc014ae98c411ca9f11102676962/chrome/test/BUILD.gn

Comment 12 by warx@chromium.org, Mar 23 2018

Status: Fixed (was: Assigned)
Fixed in m67

Sign in to add a comment