New issue
Advanced search Search tips

Issue 811953 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Chrome closes merchant website after payment, if it embeds an iframe at the scope of the payment handler.

Project Member Reported by rouslan@chromium.org, Feb 13 2018

Issue description

^^^
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 13 2018

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

commit 0ee7e9ac67f87c4148fdebb4712dfa49f6d593b0
Author: Rouslan Solomakhin <rouslan@chromium.org>
Date: Tue Feb 13 23:40:21 2018

[Payments] Close only top-level payment handler windows after payment.

Before this patch, all websites with an iframe from the scope of a
payment handler would be closed after a payment via this payment
handler. This could close the merchant page after payment, for example.

This patch skips render frame hosts that have a parent when identifying
the windows that should be closed.

After this patch, only websites where the top level context is in the
scope of the payment handler will be closed after payment via this
payment handler.

Bug:  811953 
Change-Id: Ib75cc09e22d23e5816b9b436ddb4bb962bfd376b
Reviewed-on: https://chromium-review.googlesource.com/917044
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536551}
[modify] https://crrev.com/0ee7e9ac67f87c4148fdebb4712dfa49f6d593b0/content/browser/payments/payment_app_provider_impl.cc

Status: Fixed (was: Started)

Sign in to add a comment