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

Issue 786090 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

[Autofill] When unmasking an expired card, the input fields are still visible.

Project Member Reported by ma...@chromium.org, Nov 16 2017

Issue description

Visual bug when using an expired card from Google Payments. 

The month/year fields are still visible when the "Confirming" progress overlay comes on top. 

Everything needs to paint to a layer.
 

Comment 1 Deleted

Comment 2 by ma...@chromium.org, Nov 16 2017

cvc from 2017-11-16 15:24:57.png
15.6 KB View Download
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 12 2017

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

commit 747b7f343f668905d6020777d91d1e3fad9d7cb1
Author: Mathieu Perreault <mathp@chromium.org>
Date: Tue Dec 12 14:17:54 2017

[Payments] Make contents not visible while progress spinner is shown

Hide the main contents of the dialog while the progress spinner is being
shown.

Also, remove FadeOutView.

Bug:  786090 
Test: visual
Change-Id: I0c848547b74cd3af21e19f275b012aa43688170b
Reviewed-on: https://chromium-review.googlesource.com/818704
Reviewed-by: Trent Apted <tapted@chromium.org>
Commit-Queue: Mathieu Perreault <mathp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523427}
[modify] https://crrev.com/747b7f343f668905d6020777d91d1e3fad9d7cb1/chrome/browser/ui/views/autofill/card_unmask_prompt_views.cc
[modify] https://crrev.com/747b7f343f668905d6020777d91d1e3fad9d7cb1/chrome/browser/ui/views/autofill/card_unmask_prompt_views.h

Comment 4 by ma...@chromium.org, Dec 12 2017

Status: Fixed (was: Assigned)

Comment 5 by hdodda@chromium.org, Dec 13 2017

Cc: hdodda@chromium.org
Labels: Needs-Feedback
@mathp-- Could you please help us with the steps to reproduce the issue and verify the fix , if it has to be verified from TE End.

Thanks!

Sign in to add a comment