[Mac] Rebuild app shims when they fail to dyload Chrome Framework.

chrome/browser/web_applications/web_app_mac.mm

 // 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.
 
 #import "chrome/browser/web_applications/web_app_mac.h"
 
 #import <Carbon/Carbon.h>
 #import <Cocoa/Cocoa.h>
 
 #include "apps/app_shim/app_shim_mac.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/mac/foundation_util.h"
 #include "base/mac/launch_services_util.h"
 #include "base/mac/mac_util.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/path_service.h"
 #include "base/process/process_handle.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
+#include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/browser_process.h"
 #import "chrome/browser/mac/dock.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/shell_integration.h"
+#include "chrome/browser/ui/app_list/app_list_service.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_version_info.h"
 #import "chrome/common/mac/app_mode_common.h"
 #include "content/public/browser/browser_thread.h"
+#include "extensions/browser/extension_registry.h"
 #include "extensions/common/extension.h"
 #include "grit/chrome_unscaled_resources.h"
 #include "grit/chromium_strings.h"
 #import "skia/ext/skia_utils_mac.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/base/l10n/l10n_util.h"
 #import "ui/base/l10n/l10n_util_mac.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image_family.h"
@@ skipping 182 matching lines @@
   // Launch without activating (kLSLaunchDontSwitch).
   base::mac::OpenApplicationWithPath(
       shim_path, command_line, kLSLaunchDefaults | kLSLaunchDontSwitch, NULL);
 }
 
 base::FilePath GetAppLoaderPath() {
   return base::mac::PathForFrameworkBundleResource(
       base::mac::NSToCFCast(@"app_mode_loader.app"));
 }
 
+void UpdateAndLaunchShimOnFileThread(
+    const web_app::ShortcutInfo& shortcut_info,
+    const extensions::FileHandlersInfo& file_handlers_info) {
+  base::FilePath shortcut_data_dir = web_app::GetWebAppDataDirectory(
+      shortcut_info.profile_path, shortcut_info.extension_id, GURL());
+  web_app::internals::UpdatePlatformShortcuts(
+      shortcut_data_dir, base::string16(), shortcut_info, file_handlers_info);
+  LaunchShimOnFileThread(shortcut_info);
+}
+
+void UpdateAndLaunchShim(
+    const web_app::ShortcutInfo& shortcut_info,
+    const extensions::FileHandlersInfo& file_handlers_info) {
+  content::BrowserThread::PostTask(
+      content::BrowserThread::FILE, FROM_HERE,
+      base::Bind(
+          &UpdateAndLaunchShimOnFileThread, shortcut_info, file_handlers_info));
+}
+
 base::FilePath GetLocalizableAppShortcutsSubdirName() {
   static const char kChromiumAppDirName[] = "Chromium Apps.localized";
   static const char kChromeAppDirName[] = "Chrome Apps.localized";
   static const char kChromeCanaryAppDirName[] = "Chrome Canary Apps.localized";
 
   switch (chrome::VersionInfo::GetChannel()) {
     case chrome::VersionInfo::CHANNEL_UNKNOWN:
       return base::FilePath(kChromiumAppDirName);
 
     case chrome::VersionInfo::CHANNEL_CANARY:
@@ skipping 607 matching lines @@
 
 void MaybeLaunchShortcut(const web_app::ShortcutInfo& shortcut_info) {
   if (!apps::IsAppShimsEnabled())
     return;
 
   content::BrowserThread::PostTask(
       content::BrowserThread::FILE, FROM_HERE,
       base::Bind(&LaunchShimOnFileThread, shortcut_info));
 }
 
+bool MaybeRebuildShortcut(const CommandLine& command_line) {
+  if (!command_line.HasSwitch(app_mode::kAppShimError))
+    return false;
+
+  std::string bundle_base_name =
+      command_line.GetSwitchValuePath(app_mode::kAppShimError)
+          .BaseName().RemoveExtension().value();
+  if (bundle_base_name.length() < 40) {

[tapted, 2014/05/05 07:02:01]

40 -> constant?

But all the returns below here `return true;` Could you just post a task to read
the plist?

Something like

  base::FilePath shim_path =
      command_line.GetSwitchValuePath(app_mode::kAppShimError);
  base::PostTaskAndReplyWithResult(
      BrowserThread::GetBlockingPool(),
      FROM_HERE,
      base::Bind(BuildShortcutInfoFromBundle, shim_path),
      base::Bind(RebuildAppAndLaunch, shim_path));
  return true;
}

  void RebuildAppAndLaunch(const base::FilePath& app_data_path,
      const ShortcutInfo& info) {
    if (info.extension_id ==  app_mode::kAppListModeId) {
      app_list_service->CreateShortcut();
      app_list_service->Show();
      return;
    }
    PostTaskAndReply(
        ..
        Bind(UpdatePlatformShortcuts(app_data_path, "", info,
FileHandlersInfo()),
        Bind(LanuchShim, ..));
  }


The string stuff just feels a little weird. E.g. we go through some extra hoops
to handle someone moving a shim (beyond just finder drag & drop creating an
alias). And in that case, there's a higher chance the shim has been renamed from
the `Profile extension_id` pattern as well

[jackhou1, 2014/05/06 04:47:58]

Done.

Good idea. It still needs to get the Profile* and Extension*, and load the icon
though.

+    if (bundle_base_name == app_mode::kAppListModeId) {
+      AppListService* app_list_service =
+          AppListService::Get(chrome::HOST_DESKTOP_TYPE_NATIVE);
+      app_list_service->CreateShortcut();
+      app_list_service->Show();
+    }
+    return true;
+  }
+
+  std::vector<std::string> parts;
+  base::SplitString(bundle_base_name, ' ', &parts);
+  std::string app_id = parts.back();

[tapted, 2014/05/05 07:02:01]

hypothetically.... will this crash if bundle_base_name was an empty string?

[jackhou1, 2014/05/06 04:47:58]

Removed.

+  parts.pop_back();
+  base::FilePath profile_path(JoinString(parts, ' '));
+
+  ProfileManager* profile_manager = g_browser_process->profile_manager();
+  Profile* profile = profile_manager->GetProfileByPath(
+      profile_manager->user_data_dir().Append(profile_path));
+  if (!profile || !profile_manager->IsValidProfile(profile))
+    return true;
+
+  extensions::ExtensionRegistry* registry =
+      extensions::ExtensionRegistry::Get(profile);
+  const extensions::Extension* extension =
+      registry->GetExtensionById(app_id,
+                                 extensions::ExtensionRegistry::ENABLED);
+  if (!extension || !extension->is_platform_app())
+    return true;
+
+  web_app::internals::GetInfoForApp(
+      extension, profile, base::Bind(&UpdateAndLaunchShim));
+  return true;
+}
+
 namespace internals {
 
 bool CreatePlatformShortcuts(
     const base::FilePath& app_data_path,
     const web_app::ShortcutInfo& shortcut_info,
     const extensions::FileHandlersInfo& file_handlers_info,
     const web_app::ShortcutLocations& creation_locations,
     ShortcutCreationReason creation_reason) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
   WebAppShortcutCreator shortcut_creator(
@@ skipping 32 matching lines @@
         BuildShortcutInfoFromBundle(*it);
     WebAppShortcutCreator shortcut_creator(
         it->DirName(), shortcut_info, extensions::FileHandlersInfo());
     shortcut_creator.DeleteShortcuts();
   }
 }
 
 }  // namespace internals
 
 }  // namespace web_app