Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[FedCM] Check top frame's visibility instead of iframe's
Currently we only proceed with FedCM API if the render frame is visible. This makes sense when FedCM is used in the top frame. However, it's common for top frame to embed a cross-origin iframe, possibly invisible, to invoke FedCM API. This patch checks the top frame's visibility instead of the iframe's to make sure FedCM API can be used as long as the top frame is visible. (cherry picked from commit 2deafce) Change-Id: I299f1d3fb6dfb50b45cb5323efc6a97c95d45132 Bug: 1481308 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4860279 Reviewed-by: Zachary Tan <tanzachary@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Original-Commit-Position: refs/heads/main@{#1195567} Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4892208 Auto-Submit: Yi Gu <yigu@chromium.org> Commit-Queue: Zachary Tan <tanzachary@chromium.org> Cr-Commit-Position: refs/branch-heads/5993@{#785} Cr-Branched-From: 5113507-refs/heads/main@{#1192594}
- Loading branch information
Showing
2 changed files
with
19 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters