Add PaymentResponse.prototype.onpayerdetailchange (#724)

marcoscaceres · web-flow · commit d27cc1667655 · 2018-07-03T13:02:46.000-04:00
diff --git a/index.html b/index.html
@@ -3152,7 +3152,7 @@ <h2>
       </h2>
       <pre class="idl">
         [SecureContext, Exposed=Window]
-        interface PaymentResponse {
+        interface PaymentResponse : EventTarget  {
           [Default] object toJSON();
 
           readonly attribute DOMString requestId;
@@ -3166,6 +3166,8 @@ <h2>
 
           Promise&lt;void&gt; complete(optional PaymentComplete result = "unknown");
           Promise&lt;void&gt; retry(PaymentValidationErrors errorFields);
+
+          attribute EventHandler onpayerdetailchange;
         };
       </pre>
       <p class="note">
@@ -3523,6 +3525,14 @@ <h2>
           </li>
         </ol>
       </section>
+      <section data-dfn-for="PaymentResponse" data-link-for="PaymentResponse">
+        <h2>
+          <dfn>onpayerdetailchange</dfn> attribute
+        </h2>
+        <p>
+          Allows a developer to handle "<a>payerdetailchange</a>" events.
+        </p>
+      </section>
       <section>
         <h2>
           Internal Slots
@@ -3602,6 +3612,9 @@ <h2>
             <th>
               Dispatched when…
             </th>
+            <th>
+              Target
+            </th>
           </tr>
           <tr>
             <td>
@@ -3613,6 +3626,9 @@ <h2>
             <td>
               The user provides a new shipping address.
             </td>
+            <td>
+              <a>PaymentRequest</a>
+            </td>
           </tr>
           <tr>
             <td>
@@ -3624,6 +3640,24 @@ <h2>
             <td>
               The user chooses a new shipping option.
             </td>
+            <td>
+              <a>PaymentRequest</a>
+            </td>
+          </tr>
+          <tr>
+            <td>
+              <code><dfn>payerdetailchange</dfn></code>
+            </td>
+            <td>
+              <a>PaymentRequestUpdateEvent</a>
+            </td>
+            <td>
+              The user changes the payer name, the payer email, or the payer
+              phone (see <a>payer detail changed algorithm</a>).
+            </td>
+            <td>
+              <a>PaymentResponse</a>
+            </td>
           </tr>
           <tr>
             <td>
@@ -3812,8 +3846,14 @@ <h2>
             <li>If <var>event</var>.<a>[[\waitForUpdate]]</a> is true, then <a>
               throw</a> an "<a>InvalidStateError</a>" <a>DOMException</a>.
             </li>
-            <li>Let <var>request</var> be the value of <var>event</var>'s
-            <a data-cite="DOM#event-target">target</a>.
+            <li>If <var>event</var>'s <a data-cite=
+            "DOM#event-target">target</a> is an instance of
+            <a>PaymentResponse</a>, let <var>request</var> be
+            <var>event</var>'s <a data-cite=
+            "DOM#event-target">target</a>.<a>[[\request]]</a>.
+            </li>
+            <li>Otherwise, let <var>request</var> be the value of
+            <var>event</var>'s <a data-cite="DOM#event-target">target</a>.
             </li>
             <li>Assert: <var>request</var> is an instance of
             <a>PaymentRequest</a>.
@@ -3989,15 +4029,11 @@ <h2>
             <a>Queue a task</a> on the <a>user interaction task source</a> to
             run the following steps:
             <ol>
-              <li>If the <var>request</var>.<a>[[\updating]]</a> is true, then
-              terminate this algorithm and take no further action. Only one
-              update may take place at a time. The <a>user agent</a> SHOULD
-              ensure that this never occurs.
+              <li>Assert: <var>request</var>.<a>[[\updating]]</a> is false.
+              Only one update can take place at a time.
               </li>
-              <li>If the <var>request</var>.<a>[[\state]]</a> is not set to
-              "<a>interactive</a>", then terminate this algorithm and take no
-              further action. The <a>user agent</a> user interface SHOULD
-              ensure that this never occurs.
+              <li>Assert: <var>request</var>.<a>[[\state]]</a> is
+              "<a>interactive</a>".
               </li>
               <li data-link-for="PaymentMethodChangeEvent">
                 <a>Fire an event</a> named "<a>paymentmethodchange</a>" at
@@ -4021,19 +4057,15 @@ <h2>
           with an event name of <var>name</var>:
         </p>
         <ol class="algorithm">
-          <li>If the <var>request</var>.<a>[[\updating]]</a> is true, then
-          terminate this algorithm and take no further action. Only one update
-          may take place at a time. The <a>user agent</a> SHOULD ensure that
-          this never occurs.
+          <li>Assert: <var>request</var>.<a>[[\updating]]</a> is false. Only
+          one update can take place at a time.
           </li>
-          <li>If the <var>request</var>.<a>[[\state]]</a> is not set to
-          "<a>interactive</a>", then terminate this algorithm and take no
-          further action. The <a>user agent</a> user interface SHOULD ensure
-          that this never occurs.
+          <li>Assert: <var>request</var>.<a>[[\state]]</a> is
+          "<a>interactive</a>".
           </li>
           <li>Let <var>event</var> be the result of <a data-cite=
-          "!DOM#concept-event-create">creating an event</a> using
-          <a>PaymentRequestUpdateEvent</a>.
+          "!DOM#concept-event-create">creating an event</a> using the
+          <a>PaymentRequestUpdateEvent</a> interface.
           </li>
           <li>Initialize <var>event</var>'s <code><a data-cite=
           "!DOM#dom-event-type">type</a></code> attribute to <var>name</var>.
@@ -4049,6 +4081,84 @@ <h2>
           </li>
         </ol>
       </section>
+      <section>
+        <h2>
+          Payer detail changed algorithm
+        </h2>
+        <p>
+          The user agent MUST run the <dfn>payer detail changed algorithm</dfn>
+          when the user changes the <var>payer name</var>, or the <var>payer
+          email</var>, or the <var>payer phone</var> in the user interface:
+        </p>
+        <ol class="algorithm">
+          <li>Let <var>request</var> be the <a>PaymentRequest</a> object that
+          the user is interacting with.
+          </li>
+          <li>If <var>request</var>.<a>[[\response]]</a> is null, return.
+          </li>
+          <li>Let <var>response</var> be
+          <var>request</var>.<a>[[\response]]</a>.
+          </li>
+          <li>
+            <a>Queue a task</a> on the <a>user interaction task source</a> to
+            run the following steps:
+            <ol data-link-for="PaymentResponse">
+              <li>Assert: <var>request</var>.<a>[[\updating]]</a> is false.
+              </li>
+              <li>Assert: <var>request</var>.<a>[[\state]]</a> is
+              "<a>interactive</a>".
+              </li>
+              <li>Let <var>options</var> be
+              <var>request</var>.<a>[[\options]]</a>.
+              </li>
+              <li>If <var>payer name</var> changed and
+              <var>options</var>.<a data-link-for=
+              "PaymentOptions">requestPayerName</a> is true:
+                <ol>
+                  <li>Set <var>response</var>'s <a>payerName</a> attribute to
+                  <var>payer name</var>.
+                  </li>
+                </ol>
+              </li>
+              <li>If <var>payer email</var> changed and
+              <var>options</var>.<a data-link-for=
+              "PaymentOptions">requestPayerEmail</a> is true:
+                <ol>
+                  <li>Set <var>response</var>'s <a>payerEmail</a> attribute to
+                  <var>payer email</var>.
+                  </li>
+                </ol>
+              </li>
+              <li>If <var>payer phone</var> changed and
+              <var>options</var>.<a data-link-for=
+              "PaymentOptions">requestPayerPhone</a> is true:
+                <ol>
+                  <li>Set <var>response</var>'s <a>payerPhone</a> attribute to
+                  <var>payer phone</var>.
+                  </li>
+                </ol>
+              </li>
+              <li>Let <var>event</var> be the result of <a data-cite=
+              "!DOM#concept-event-create">creating an event</a> using
+              <a>PaymentRequestUpdateEvent</a>.
+              </li>
+              <li>Initialize <var>event</var>'s <code><a data-cite=
+              "!DOM#dom-event-type">type</a></code> attribute to
+              "<a>payerdetailchange</a>".
+              </li>
+              <li>
+                <a data-cite="!DOM#concept-event-dispatch">Dispatch</a>
+                <var>event</var> at <var>response</var>.
+              </li>
+              <li data-link-for="PaymentRequestUpdateEvent">If
+              <var>event</var>.<a>[[\waitForUpdate]]</a> is true, disable any
+              part of the user interface that could cause another change to the
+              payer details to be fired.
+              </li>
+            </ol>
+          </li>
+        </ol>
+      </section>
       <section>
         <h2>
           User accepts the payment request algorithm
