Add tests for redirecting to hosted app URLs in subframes |
|||
Issue descriptionIssue 715756 was a regression that reached stable channel because we had no tests for loading subframes that redirect to hosted app URLs. We should add a test for this case. Alex, would you be able to write one, to close the gap from issue 697513 (where we noticed it affected data URLs with plugins but not hosted apps)? (We're also hoping to remove the hosted apps process model in issue 721947, but that will probably take a while.)
,
May 17 2017
,
May 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c1a9cc5af57748545858620aab0e9cd8e24dcc93 commit c1a9cc5af57748545858620aab0e9cd8e24dcc93 Author: alexmos <alexmos@chromium.org> Date: Fri May 19 00:12:53 2017 Add a test for subframes redirecting to hosted app URLs. We recently had a regression that reached stable channel because we had no tests for loading subframes that redirect to hosted app URLs. This CL adds test coverage for that case. BUG= 721949 , 715756 Review-Url: https://codereview.chromium.org/2891633002 Cr-Commit-Position: refs/heads/master@{#472986} [modify] https://crrev.com/c1a9cc5af57748545858620aab0e9cd8e24dcc93/chrome/browser/ui/extensions/hosted_app_browsertest.cc
,
May 19 2017
Thanks! |
|||
►
Sign in to add a comment |
|||
Comment 1 by creis@chromium.org
, May 16 2017