New issue
Advanced search Search tips

Issue 697517 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

Do not notify onInstrumentDetailsError for getInstruments

Project Member Reported by gogerald@chromium.org, Mar 1 2017

Issue description

Do not notify onInstrumentDetailsError for getInstruments, instead respondToGetInstrumentsQuery(null)
 
Cc: rouslan@chromium.org
Components: UI>Browser>Autofill>Payments
Labels: -Pri-3 OS-Android Pri-1
Owner: gogerald@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 1 2017

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

commit a3ca447f6268a42db6ebc87acca0a601f28a757d
Author: gogerald <gogerald@chromium.org>
Date: Wed Mar 01 19:25:15 2017

Should not notify onInstrumentDetailsError for getInstruments

BUG= 697517 

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

[modify] https://crrev.com/a3ca447f6268a42db6ebc87acca0a601f28a757d/chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java

Labels: Merge-Request-58 M-58
Project Member

Comment 4 by sheriffbot@chromium.org, Mar 1 2017

Labels: -Merge-Request-58 Merge-Review-58 Hotlist-Merge-Review
This bug requires manual review: We don't branch M58 until 2017-03-02.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Did you mean M-57?
Labels: -Hotlist-Merge-Review -M-58 -Merge-Review-58
Status: Fixed (was: Untriaged)
Yes, but looks not worth to do that, so remove the labels and close this bug,
Components: -UI>Browser>Autofill>Payments UI>Browser>Payments

Sign in to add a comment