| commit | 2075c7ed7438b0b0796f29ec3a88e14ffeba51ff | [log] [tgz] |
|---|---|---|
| author | EhsanK <[email protected]> | Mon Aug 21 02:42:39 2017 |
| committer | Commit Bot <[email protected]> | Mon Aug 21 02:42:39 2017 |
| tree | 8ad32edf69b43806a579ac1b2d3c4183a13febcc | |
| parent | 3e0f5df1e114fd76debfbcfcfd0973c9cc8d1dca [diff] |
Show/Hide RenderWidgetHostViews under an OOPIF when its Frame Owner becomes Visible/Hidden Right now changing the visibility state of a cross-origin iframe will lead to calls to RenderWidgetHostView::Show/Hide on its corresponding view. Therefore, when hiding a frame the RenderWidget will stop generating compositor frames. However, when the iframe itself has nested cross-origin frames, the current calls to Hide() and Show() are not replicated for their views. This CL adds the missing code to inform the subviews about changes in the visibility state. The CL also modifies the existing test for visibility change a test to further verify that hiding a cross origin iframe will also hide any child view nested within it. Bug: 628704 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation Change-Id: I3589d1e90ae7ee8bfdb654625c451cc0e6704785 Reviewed-on: https://chromium-review.googlesource.com/599006 Commit-Queue: Ehsan Karamad <[email protected]> Reviewed-by: Alex Moshchuk <[email protected]> Reviewed-by: Ken Buchanan <[email protected]> Reviewed-by: Lucas Gadani <[email protected]> Cr-Commit-Position: refs/heads/master@{#495861}
ChromiumChromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .