"Redirecting URLs to Packaged Apps" implementation: revised

apps/app_window_contents.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "apps/app_window_contents.h"
 
 #include "apps/native_app_window.h"
+#include "apps/url_redirector.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/api/app_window.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 112 matching lines @@
     }
     default:
       NOTREACHED() << "Received unexpected notification";
   }
 }
 
 bool AppWindowContents::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(AppWindowContents, message)
     IPC_MESSAGE_HANDLER(ExtensionHostMsg_Request, OnRequest)
+    IPC_MESSAGE_HANDLER(ExtensionHostMsg_RedirectUrl,
+                        LaunchAppWithUrl)
     IPC_MESSAGE_HANDLER(ExtensionHostMsg_UpdateDraggableRegions,
                         UpdateDraggableRegions)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 extensions::WindowController*
 AppWindowContents::GetExtensionWindowController() const {
   return NULL;
 }
 
 content::WebContents* AppWindowContents::GetAssociatedWebContents() const {
   return web_contents_.get();
 }
 
 void AppWindowContents::OnRequest(
     const ExtensionHostMsg_Request_Params& params) {
   extension_function_dispatcher_->Dispatch(
       params, web_contents_->GetRenderViewHost());
 }
 
 void AppWindowContents::UpdateDraggableRegions(
     const std::vector<extensions::DraggableRegion>& regions) {
   host_->UpdateDraggableRegions(regions);
 }
 
+void AppWindowContents::LaunchAppWithUrl(const GURL& url,
+                                         const GURL& referrer_url) const {
+  UrlRedirector::MaybeLaunchAppWithUrl(

[benwells, 2013/09/03 01:10:56]

Can you just create a UrlRedirector tab helper thing like in
browser_tab_contents? Have a look at c/b/ui/apps/chrome_shell_window_delegate
InitWebContents.

[sergeygs, 2013/09/03 06:15:24]

Great advice, thanks, I just didn't find that on my own. Moved the IPC
dispatching around as suggested, and it works!

+      Profile::FromBrowserContext(web_contents_->GetBrowserContext()),
+      url, referrer_url);
+}
+
 void AppWindowContents::SuspendRenderViewHost(
     content::RenderViewHost* rvh) {
   DCHECK(rvh);
   content::BrowserThread::PostTask(
       content::BrowserThread::IO, FROM_HERE,
       base::Bind(&content::ResourceDispatcherHost::BlockRequestsForRoute,
                  base::Unretained(content::ResourceDispatcherHost::Get()),
                  rvh->GetProcess()->GetID(), rvh->GetRoutingID()));
 }
 
 }  // namespace apps