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

Issue 602659 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Bug

Blocking:
issue 587995



Sign in to add a comment

Use JSON.stringify instead of JSONValue.

Project Member Reported by rouslan@chromium.org, Apr 12 2016

Issue description

Work started in https://codereview.chromium.org/1847823004/. haavardm@ will be helping out.
 
Labels: M-54
Cc: sanjoy....@samsung.com
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 2 2016

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

commit f68abde2798e2713e39b6911d68db0f219ae50f2
Author: rob.buis <rob.buis@samsung.com>
Date: Wed Nov 02 01:18:02 2016

Use JSON::Stringify instead of JSONValue

Use JSON::Stringify instead of JSONValue. Keep the old
behavior to not accept Arrays.

BUG= 602659 

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

[modify] https://crrev.com/f68abde2798e2713e39b6911d68db0f219ae50f2/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp

Rouslan do you think this can be closed now? I have https://codereview.chromium.org/2473533002/ open but that is mostly cleanup.
Status: Fixed (was: Assigned)
OK
Cc: haava...@opera.com
Owner: rob.b...@samsung.com

Sign in to add a comment