Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(63)

Unified Diff: third_party/WebKit/LayoutTests/payments/payment-request-interface.html

Issue 2406713002: PaymentRequest: Ignore shipping options if there are duplicated IDs. (Closed)
Patch Set: test Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/payments/payment-request-interface.html
diff --git a/third_party/WebKit/LayoutTests/payments/payment-request-interface.html b/third_party/WebKit/LayoutTests/payments/payment-request-interface.html
index cda7c6a1ad3dc77d1a5f8c659f99f889d4f0e479..2b23b60fd9ed37a112d4501567bed39040b4e81c 100644
--- a/third_party/WebKit/LayoutTests/payments/payment-request-interface.html
+++ b/third_party/WebKit/LayoutTests/payments/payment-request-interface.html
@@ -265,10 +265,6 @@ generate_tests(assert_throws, [
['Duplicate supported payment method identifiers should throw TypeError.', null, function() {
new PaymentRequest([{'supportedMethods': ['foo']}, {'supportedMethods': ['foo']}], buildDetails(), {})
}],
- ['Duplicate shipping option identifiers should throw TypeError.', null, function() {
- var shippingOptions = [buildItem({'id': 'express', 'selected': false}), buildItem({'id': 'express', 'selected': true})];
- new PaymentRequest([{'supportedMethods': ['foo']}], {'total': buildItem(), 'displayItems': [buildItem()], 'shippingOptions': shippingOptions}, {'requestShipping': true})
please use gerrit instead 2016/10/12 01:28:02 Please add a test 'No shipping option selected for
zino 2016/10/12 18:05:24 Done.
- }],
['Absence of total should throw TypeError.', null, function() {
new PaymentRequest([{'supportedMethods': ['foo']}], {'displayItems': [buildItem()]})
}],

Powered by Google App Engine
This is Rietveld 408576698