Remove the extraneous Shipping section header when there's no default selected shipping address |
||||
Issue descriptionThis makes the behavior consistent with the Payment and Contact Info sections.
,
Oct 19 2017
,
Oct 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9222331a65722f8b9a127c0816b8b5bf9e1b645a commit 9222331a65722f8b9a127c0816b8b5bf9e1b645a Author: Mohamad Ahmadi <mahmadi@chromium.org> Date: Fri Oct 20 20:47:51 2017 [PR] No shipping section header if there's no selected shipping address Bug: 776439 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I2de6460213dfeda39a2040d2039c05025067cd21 Reviewed-on: https://chromium-review.googlesource.com/728715 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by: Marc-Antoine Courteau <macourteau@chromium.org> Cr-Commit-Position: refs/heads/master@{#510553} [modify] https://crrev.com/9222331a65722f8b9a127c0816b8b5bf9e1b645a/ios/chrome/browser/ui/payments/payment_request_mediator.mm [modify] https://crrev.com/9222331a65722f8b9a127c0816b8b5bf9e1b645a/ios/chrome/browser/ui/payments/payment_request_view_controller.mm [modify] https://crrev.com/9222331a65722f8b9a127c0816b8b5bf9e1b645a/ios/chrome/browser/ui/payments/payment_request_view_controller_data_source.h [modify] https://crrev.com/9222331a65722f8b9a127c0816b8b5bf9e1b645a/ios/chrome/browser/ui/payments/payment_request_view_controller_unittest.mm
,
Oct 21 2017
,
Jan 5 2018
Verified in 64.0.3282.75 beta, iPhone X iOS 11.2, iPhone 7 iOS11 Looks good |
||||
►
Sign in to add a comment |
||||
Comment 1 by mahmadi@chromium.org
, Oct 19 2017Labels: -Pri-3 M-64 OS-iOS Pri-2
Owner: mahmadi@chromium.org
Status: Assigned (was: assing)