desktop-pwas: Pass opener to new app window

When a website calls window.open() with a URL for which there is a PWA
installed, we cancel the navigation in the newly opened tab and start a
new navigation in a new app window.

This patch plumbs the opener that would have existed in the new tab to
the new app window, so that the new app window can access it.

This only takes care of the child window. The parent window is not yet
able to access the newly opened app window.

Bug: 804881
Change-Id: Ie08c3425b13b4940435e6cf3614a015d61ed10a3
Reviewed-on: https://chromium-review.googlesource.com/906042Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536013}

chrome/browser/extensions/bookmark_app_navigation_throttle.cc

@@ -424,6 +424,7 @@ void BookmarkAppNavigationThrottle::OpenBookmarkApp(
       profile, bookmark_app.get(), extensions::LAUNCH_CONTAINER_WINDOW,
       WindowOpenDisposition::CURRENT_TAB, extensions::SOURCE_URL_HANDLER);
   launch_params.override_url = navigation_handle()->GetURL();
+  launch_params.opener = source->GetOpener();
 
   DVLOG(1) << "Opening app.";
   OpenApplication(launch_params);

chrome/browser/ui/extensions/app_launch_params.cc

@@ -34,7 +34,8 @@ AppLaunchParams::AppLaunchParams(Profile* profile,
       command_line(base::CommandLine::NO_PROGRAM),
       source(source),
       play_store_status(PlayStoreStatus::PLAY_STORE_STATUS_UNKNOWN),
-      display_id(display_id) {
+      display_id(display_id),
+      opener(nullptr) {
 #if defined(OS_CHROMEOS)
   // TODO(b/34478891): Remove this from app launch.
   if (set_playstore_status) {

chrome/browser/ui/extensions/app_launch_params.h

@@ -19,6 +19,10 @@
 
 class Profile;
 
+namespace content {
+class RenderFrameHost;
+}
+
 namespace extensions {
 class Extension;
 }
@@ -78,6 +82,10 @@ struct AppLaunchParams {
   // display::kInvalidDisplayId means that the display does not exist or is not
   // set.
   int64_t display_id;
+
+  // The frame that initiated the open. May be null. If set, the new app will
+  // have |opener| as its window.opener.
+  content::RenderFrameHost* opener;
 };
 
 // Helper to create AppLaunchParams using extensions::GetLaunchContainer with

chrome/browser/ui/extensions/application_launch.cc

@@ -213,8 +213,12 @@ WebContents* OpenApplicationWindow(const AppLaunchParams& params,
       (extension ? ui::PAGE_TRANSITION_AUTO_BOOKMARK
                  : ui::PAGE_TRANSITION_AUTO_TOPLEVEL);
 
-  WebContents* web_contents =
-      chrome::AddSelectedTabWithURL(browser, url, transition);
+  NavigateParams nav_params(browser, url, transition);
+  nav_params.disposition = WindowOpenDisposition::NEW_FOREGROUND_TAB;
+  nav_params.opener = params.opener;
+  Navigate(&nav_params);
+
+  WebContents* web_contents = nav_params.target_contents;
   web_contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   web_contents->GetRenderViewHost()->SyncRendererPrefs();