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

Issue 659080 link

Starred by 2 users
Status: Fixed
Owner:
█ warx@chromium.org
Last visit > 30 days ago
Closed: Oct 2016
Cc:
ajha@chromium.org
█ kavvaru@chromium.org
█ brajkumar@chromium.org
█ durga.behera@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: User manager is seen in back end on clicking manage people

Project Member Reported by sc00335...@techmahindra.com, Oct 25 2016 
Version: 56.0.2900.0
OS: Ubuntu 14.04

What steps will reproduce the problem?
(1)Launch chrome >> Click on profile icon present on top right corner and select Manage People >> Observe for User manager window

Expected: User manager window should be seen in front end.
Actual:Instead it is seen back of current window.

This is a regression issue broken in M56. Will provide other info soon.
Actual_user manager.ogv
457 KB View Download
Expected_user manager.ogv
542 KB View Download

Comment 1 by sc00335...@techmahindra.com, Oct 25 2016 

Issue is not seen in windows.

Manual Bisect Info:

Good build: 56.0.2897.0  
Bad build: 56.0.2899.0

Comment 2 by brajkumar@chromium.org, Oct 25 2016

Labels: -Needs-Bisect hasbisect-per-revision ReleaseBlock-Stable
Owner: warx@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Ubuntu 14.04 using chrome latest Dev #56.0.2900.0. This issue is not seen on Mac OS as well.

Bisect Information:
------------------------
You are probably looking for a change made after 426904 (known good), but no later than 426905 (first known bad).

CHANGELOG URL:
----------------
https://chromium.googlesource.com/chromium/src/+log/b07f20559996aa8154a3a67e742676a9799e37c4..5c16e0e849526c1c627e08a05351ab38a9cfbcf1

From the CL above, assigning the issue to the concern owner 

@warx - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Note: Adding RB-stable, please feel free to edit if this is not the case.

Thanks!

Comment 3 by warx@chromium.org, Oct 25 2016 

Hi, I didn't see the difference between the two videos. What is the specific problem?

Comment 4 by warx@chromium.org, Oct 25 2016

Status: Started (was: Assigned)
Never mind. I just reproduced it. It is probably related to my CL. I will have a check.
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 27 2016 

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

commit c5e551b4da8b4bf7cc8fcfbdff3a052f19619bf3
Author: warx <warx@chromium.org>
Date: Thu Oct 27 22:12:11 2016

Remove RestoreFocusedView in OnWindowActivated for DesktopNativeWidgetAura

RestoreFocusedView upon desktop native widget activation should be done in DesktopNativeWidgetAura::HandleActivationChanged, instead of OnWindowActivated, as in OnWindowActivated, GetWidget()->IsActive() is asynced for Linux. So we should remove the related code.

BUG= 659080 
TEST=manual build test sees bug fixed

Review-Url: https://codereview.chromium.org/2451323002
Cr-Commit-Position: refs/heads/master@{#428157}

[modify] https://crrev.com/c5e551b4da8b4bf7cc8fcfbdff3a052f19619bf3/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
[modify] https://crrev.com/c5e551b4da8b4bf7cc8fcfbdff3a052f19619bf3/ui/views/widget/desktop_aura/desktop_native_widget_aura.h

Comment 6 by warx@chromium.org, Oct 27 2016

Status: Fixed (was: Started)

Comment 7 by durga.behera@chromium.org, Nov 1 2016

Labels: TE-Verified-M56 TE-Verified-56.0.2906.0
Verified the issue on Ubuntu 14.04 using 56.0.2906.0 and its working fine nw.
Please find the screen cast for the same.
659080_Nov_1.ogv
919 KB View Download

Comment 8 by shimazu@chromium.org, Nov 4 2016 

 Issue 661038  has been merged into this issue.
►

Sign in to add a comment