Issue metadata
Sign in to add a comment
|
Regression : Preview picture is still seen even on clicking 'Discard photo' option in 'Set your profile picture' screen |
||||||||||||||||||||||
Issue descriptionChrome Version: 64.0.3256.0/10093.0.0 dev channel Peppy,Kip and Paine OS: Chrome What steps will reproduce the problem? (1)Recover build -> Sign into User which already has default 'Google profile picture' and click on next button (2)Now in 'Set your profile picture' screen -> Click on 'Take photo' option -> So that Camera screen is seen -> Press 'Enter' button and picture is captured (3)Now try to discard photo and observe photo gets discarded from 'Photo from internal camera' option but still preview picture is seen (Please refer video) Expected: On clicking 'Discard photo' picture should get discarded and focus should traverse to default 'Google profile picture' screen Actual: Instead even on clicking 'Discard photo' option photo gets discarded from 'Photo from internal camera' option but still preview picture is seen with 'Discard photo' option and on clicking it nothing happens This is Regression issue as same is working fine in 63.0.3238.0/10028.0.0 dev-channel Peppy. @stevenjb : Please confirm the Issue
,
Nov 3 2017
I can't reproduce this after applying https://chromium-review.googlesource.com/c/chromium/src/+/753404 so it's possible that fixes both issues.
,
Dec 4 2017
Hi, this is tagged as a beta blocker for M64. We're not that far out and this bug hasn't been updated in a while. Ping? Thanks...
,
Dec 5 2017
I can't reproduce this so I think it was fixed with that other bug. mmanchala@, please re-open if you can still repo.
,
Dec 5 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-64; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-64 label, otherwise remove Merge-TBD label. Thanks.
,
Dec 11 2017
Able to reproduce the issue on latest M-65 i.e. on 65.0.3287.0/10203.0.0 Kip. Re-opening this issue as per comment #4 Thanks..!!
,
Dec 11 2017
Please re-evaluate today (Monday, 11-Dec) re: Beta Blocker Status. We're targeting Beta for tomorrow, so please update if a) this isn't blocking, b) the impacted boards can be identified, c) this is resolved / not reproducible, d) a merge / fix can be submitted today. Thanks!
,
Dec 12 2017
Marking this as RBS as it should not gate beta.
,
Dec 12 2017
Re. #8: functionality is generally fine
,
Jan 4 2018
Ping as no traction in some time. Please eval as a stable blocker.
,
Jan 8 2018
I still can't repro this. I think it's a duplicate of 786887 that was fixed on Dec 2: https://chromium.googlesource.com/chromium/src/+/e5a074a4c902e83f0d50cb06e9b6f400702333ba
,
Jan 23 2018
Seems like this was only repro'd on M65 (c#6) back in Dec Still repro?
,
Feb 20 2018
If no one has seen this please close it, this is marked as blocking stable 65 and we are only a few weeks away.
,
Feb 20 2018
Is this still reproducible? Is it specific to Kip? Please re-open with some more details if still an issue.
,
Feb 20 2018
[Auto-generated comment by a script] We noticed that this issue is targeted for M-65; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-65 label, otherwise remove Merge-TBD label. Thanks.
,
Apr 4 2018
,
Aug 3
C#14>> Issue is working fine on M-69 dev 69.0.3497.24/10895.11.0.It is not specific to Kip device. Thanks..!!
,
Aug 3
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by reve...@chromium.org
, Nov 3 2017