PaymentHandler: Implement PaymentRequestEvent constructor.

PaymentHandler: Implement PaymentRequestEvent constructor.

Need the constructor and init dictionary for compatibility with DOM events.
This CL contains IDL change but blink-intent is not required because it is still
behind runtime flag.

Related spec link:
  https://w3c.github.io/payment-handler/#dom-paymentrequestevent
  https://w3c.github.io/payment-handler/#dom-paymentrequesteventinit

BUG=661608, 720027
TEST=payment-request-event.html

Review-Url: https://codereview.chromium.org/2903643002
Cr-Commit-Position: refs/heads/master@{#477382}
Committed: https://chromium.googlesource.com/chromium/src/+/bb3f76109a2e6b8be54745b7235c650b7b6f77c1

[zino, 2017-06-02 18:04:09]

Description was changed from

==========
PaymentHandler: Implement PaymentRequestEvent constructor.

BUG=
==========

to

==========
PaymentHandler: Implement PaymentRequestEvent constructor.

Need the constructor and init dictionary for compatibility with DOM events.
This CL contains IDL change but blink-intent is not required because it is still
behind runtime flag.

Related spec link:
  https://w3c.github.io/payment-handler/#dom-paymentrequestevent
  https://w3c.github.io/payment-handler/#dom-paymentrequesteventinit

BUG=661608,720027
TEST=payment-request-event.html
==========

[zino, 2017-06-02 18:04:22]

jinho.bang@samsung.com changed reviewers:
+ gogerald@chromium.org, rouslan@chromium.org

[zino, 2017-06-02 18:06:41]

PTAL

[please use gerrit instead, 2017-06-05 15:38:55]

LGTM % comment

https://codereview.chromium.org/2903643002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/payments/PaymentRequestEvent.h (right):

https://codereview.chromium.org/2903643002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/payments/PaymentRequestEvent.h:50:
PaymentRequestEvent(const AtomicString& type, const PaymentRequestEventInit&);
Would you mind having a single constructor that both Create() methods invoke?

[zino, 2017-06-06 15:03:12]

Thank you for review!

https://codereview.chromium.org/2903643002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/payments/PaymentRequestEvent.h (right):

https://codereview.chromium.org/2903643002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/payments/PaymentRequestEvent.h:50:
PaymentRequestEvent(const AtomicString& type, const PaymentRequestEventInit&);
On 2017/06/05 15:38:55, rouslan wrote:
> Would you mind having a single constructor that both Create() methods invoke?

Done.

[zino, 2017-06-06 17:40:12]

The CQ bit was checked by jinho.bang@samsung.com

[zino, 2017-06-06 17:40:12]

The patchset sent to the CQ was uploaded after l-g-t-m from rouslan@chromium.org
Link to the patchset: https://codereview.chromium.org/2903643002/#ps40001
(title: "PaymentHandler: Implement PaymentRequestEvent constructor.")

[commit-bot: I haz the power, 2017-06-06 17:40:42]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-06 19:59:09]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496770812330700,
"parent_rev": "bb1cff19414a4a999dc667baae35dd37ed9d545c", "commit_rev":
"bb3f76109a2e6b8be54745b7235c650b7b6f77c1"}

[commit-bot: I haz the power, 2017-06-06 19:59:20]

Description was changed from

==========
PaymentHandler: Implement PaymentRequestEvent constructor.

Need the constructor and init dictionary for compatibility with DOM events.
This CL contains IDL change but blink-intent is not required because it is still
behind runtime flag.

Related spec link:
  https://w3c.github.io/payment-handler/#dom-paymentrequestevent
  https://w3c.github.io/payment-handler/#dom-paymentrequesteventinit

BUG=661608,720027
TEST=payment-request-event.html
==========

to

==========
PaymentHandler: Implement PaymentRequestEvent constructor.

Need the constructor and init dictionary for compatibility with DOM events.
This CL contains IDL change but blink-intent is not required because it is still
behind runtime flag.

Related spec link:
  https://w3c.github.io/payment-handler/#dom-paymentrequestevent
  https://w3c.github.io/payment-handler/#dom-paymentrequesteventinit

BUG=661608,720027
TEST=payment-request-event.html

Review-Url: https://codereview.chromium.org/2903643002
Cr-Commit-Position: refs/heads/master@{#477382}
Committed:
https://chromium.googlesource.com/chromium/src/+/bb3f76109a2e6b8be54745b7235c...
==========

[commit-bot: I haz the power, 2017-06-06 19:59:21]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/bb3f76109a2e6b8be54745b7235c...