Start app shim when app launched.

chrome/browser/extensions/platform_app_launcher.cc

 // Copyright (c) 2012 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 "chrome/browser/extensions/platform_app_launcher.h"
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/api/app_runtime/app_runtime_api.h"
 #include "chrome/browser/extensions/api/file_handlers/app_file_handler_util.h"
 #include "chrome/browser/extensions/api/file_system/file_system_api.h"
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_prefs.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/extensions/lazy_background_task_queue.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/extensions/app_metro_infobar_delegate_win.h"
+#include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/extensions/extension.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/web_contents.h"
 #include "net/base/mime_util.h"
 #include "net/base/net_util.h"
 #include "webkit/fileapi/file_system_types.h"
 
 #if defined(OS_CHROMEOS)
@@ skipping 48 matching lines @@
   base::FilePath relative_path(command_line->GetArgs()[0]);
   base::FilePath absolute_path(relative_path);
   if (!MakePathAbsolute(current_directory, &absolute_path)) {
     LOG(WARNING) << "Cannot make absolute path from " << relative_path.value();
     return false;
   }
   *path = absolute_path;
   return true;
 }
 
+void DoNothing(bool) {}
+
 // Helper method to launch the platform app |extension| with no data. This
 // should be called in the fallback case, where it has been impossible to
 // load or obtain file launch data.
 void LaunchPlatformAppWithNoData(Profile* profile, const Extension* extension) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   extensions::AppEventRouter::DispatchOnLaunchedEvent(profile, extension);
+  web_app::internals::LaunchShim(profile, extension, base::Bind(&DoNothing));

[tapted, 2013/05/16 04:51:00]

Would a base::Closure() with no arguments have the effect of DoNothing?

But we shouldn't be diving into web_app::internals here anyway.

And.. it might be preferable to not Dispatch the OnLaunched event until the shim
itself has launched. i.e. leave it up to the shim to do the launch -- so long as
we can guarantee a clear success/fail result, and fallback to dispatching the
event from the browser process directly.

But of course that might mean some delay. Let's discuss with benwells.

[jackhou1, 2013/05/17 05:45:36]

Nup, it needs to take a bool.

 }
 
 // Class to handle launching of platform apps to open a specific path.
 // An instance of this class is created for each launch. The lifetime of these
 // instances is managed by reference counted pointers. As long as an instance
 // has outstanding tasks on a message queue it will be retained; once all
 // outstanding tasks are completed it will be deleted.
 class PlatformAppPathLauncher
     : public base::RefCountedThreadSafe<PlatformAppPathLauncher> {
  public:
@@ skipping 292 matching lines @@
 void RestartPlatformAppWithFileEntries(
     Profile* profile,
     const Extension* extension,
     const std::vector<SavedFileEntry>& file_entries) {
   scoped_refptr<SavedFileEntryLauncher> launcher = new SavedFileEntryLauncher(
       profile, extension, file_entries);
   launcher->Launch();
 }
 
 }  // namespace extensions