"Redirecting URLs to Packaged Apps" implementation: revised

apps/app_window_contents.cc

Index: apps/app_window_contents.cc
diff --git a/apps/app_window_contents.cc b/apps/app_window_contents.cc
index d163c2ce84ace7bce1cdb270d9412ce4e87b2d98..46b30cf1d8feb388dbb37f2a19702f4a9ca65763 100644
--- a/apps/app_window_contents.cc
+++ b/apps/app_window_contents.cc
@@ -5,6 +5,7 @@
 #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"
@@ -137,6 +138,8 @@ 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)
@@ -164,6 +167,13 @@ void AppWindowContents::UpdateDraggableRegions(
   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);