Remove packaged app Windows shortcuts when app is uninstalled.

base/file_util_win.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 "base/file_util.h"
 
 #include <windows.h>
 #include <propvarutil.h>
 #include <psapi.h>
 #include <shellapi.h>
@@ skipping 329 matching lines @@
   bool is_resolved = false;
 
   // Get pointer to the IShellLink interface
   result = i_shell_link.CreateInstance(CLSID_ShellLink, NULL,
                                        CLSCTX_INPROC_SERVER);
   if (SUCCEEDED(result)) {
     base::win::ScopedComPtr<IPersistFile> persist;
     // Query IShellLink for the IPersistFile interface
     result = persist.QueryFrom(i_shell_link);
     if (SUCCEEDED(result)) {
-      WCHAR temp_path[MAX_PATH];
       // Load the shell link
       result = persist->Load(path->value().c_str(), STGM_READ);
       if (SUCCEEDED(result)) {
         // Try to find the target of a shortcut
         result = i_shell_link->Resolve(0, SLR_NO_UI);
         if (SUCCEEDED(result)) {
+          WCHAR temp_path[MAX_PATH];
           result = i_shell_link->GetPath(temp_path, MAX_PATH,
                                   NULL, SLGP_UNCPRIORITY);
           *path = FilePath(temp_path);
           is_resolved = true;
         }
       }
     }
   }
 
   return is_resolved;
 }
 
+bool GetShortcutArguments(const FilePath& shortcut_path, string16* args) {

[brettw, 2012/08/05 05:24:25]

There's too much copy-and-pasted code here from the previous function (including
a comment that doesn't even apply here). This should either be one function or
the two should be refactored somehow.

[brettw, 2012/08/05 17:55:24]

Maybe we should add a function to return a ScopedCOMPtr<IShellLink> with a given
shortcut path loaded into it. Then you can implement the GetArguments logic
yourself in your own code (since this is only called once, I'm not sure we need
a base function for it) and you can also implement ResolveShortcut in terms of
this function.

[benwells, 2012/08/07 12:24:05]

OK, that all makes sense. For now I've merged these two into one function, and
restructured a bit (also errors in GetPath and GetArguments, while unlikely, are
now handled).

I'm not sure if I like this or your suggestion for a function to return a loaded
shell link. I thought this would feel better but now I'm not so sure - if
someone else ever wants some other shortcut field (e.g. description)  having it
the way you suggest would be better.

+  base::ThreadRestrictions::AssertIOAllowed();
+  DCHECK(args);
+
+  HRESULT result;
+  base::win::ScopedComPtr<IShellLink> i_shell_link;
+
+  // Get pointer to the IShellLink interface
+  result = i_shell_link.CreateInstance(CLSID_ShellLink, NULL,
+                                       CLSCTX_INPROC_SERVER);
+  if (SUCCEEDED(result)) {
+    base::win::ScopedComPtr<IPersistFile> persist;
+    // Query IShellLink for the IPersistFile interface
+    result = persist.QueryFrom(i_shell_link);
+    if (SUCCEEDED(result)) {
+      // Load the shell link
+      result = persist->Load(shortcut_path.value().c_str(), STGM_READ);
+      if (SUCCEEDED(result)) {
+        // Try to find the target of a shortcut
+        WCHAR temp_args[MAX_PATH];
+        result = i_shell_link->GetArguments(temp_args, MAX_PATH);
+        *args = string16(temp_args);
+        return true;
+      }
+    }
+  }
+
+  return false;
+}
+
 bool CreateOrUpdateShortcutLink(const wchar_t *source,
                                 const wchar_t *destination,
                                 const wchar_t *working_dir,
                                 const wchar_t *arguments,
                                 const wchar_t *description,
                                 const wchar_t *icon,
                                 int icon_index,
                                 const wchar_t* app_id,
                                 uint32 options) {
   base::ThreadRestrictions::AssertIOAllowed();
@@ skipping 728 matching lines @@
   HANDLE cp = GetCurrentProcess();
   if (::GetMappedFileNameW(cp, file_view, mapped_file_path, kMaxPathLength)) {
     *nt_path = FilePath(mapped_file_path);
     success = true;
   }
   ::UnmapViewOfFile(file_view);
   return success;
 }
 
 }  // namespace file_util