- From: ianbjacobs <notifications@github.com>
- Date: Tue, 29 Jan 2019 15:36:03 -0800
- To: w3c/payment-handler <payment-handler@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/payment-handler/pull/331/review/197839892@github.com>
ianbjacobs commented on this pull request.
Thanks, Danyao! I have made some minor editorial suggestions. I used the new GitHub tool (shown to me by Marcos) so if you are happy with the changes you can easily incorporate them.
> @@ -2235,7 +2235,10 @@ <h2>
<li>The <a>CanMakePaymentEvent</a> event should not be fired in
private browsing mode. The user agent should behave as if
<a data-lt="CanMakePaymentEvent.respondWith()">respondWith()</a>
- was called with <code>true</code>.
+ was called with <code>false</code>. We acknowledge a consequent
+ risk: if a merchant controls both the origin of the Payment Request
```suggestion
risk: if an entity controls both the origin of the Payment Request
```
> @@ -2235,7 +2235,10 @@ <h2>
<li>The <a>CanMakePaymentEvent</a> event should not be fired in
private browsing mode. The user agent should behave as if
<a data-lt="CanMakePaymentEvent.respondWith()">respondWith()</a>
- was called with <code>true</code>.
+ was called with <code>false</code>. We acknowledge a consequent
+ risk: if a merchant controls both the origin of the Payment Request
+ API call and the payment handler, it is possible to deduce from the
```suggestion
API call and the origin of the payment handler, that entity may be able to deduce
```
> @@ -2235,7 +2235,10 @@ <h2>
<li>The <a>CanMakePaymentEvent</a> event should not be fired in
private browsing mode. The user agent should behave as if
<a data-lt="CanMakePaymentEvent.respondWith()">respondWith()</a>
- was called with <code>true</code>.
+ was called with <code>false</code>. We acknowledge a consequent
+ risk: if a merchant controls both the origin of the Payment Request
+ API call and the payment handler, it is possible to deduce from the
+ result that the user may be in private browsing mode.
```suggestion
that the user may be in private browsing mode.
```
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/payment-handler/pull/331#pullrequestreview-197839892
Received on Tuesday, 29 January 2019 23:36:26 UTC