From 914636612ba4bf032b809e1dcf09a11f87384515 Mon Sep 17 00:00:00 2001 From: "benwells@chromium.org" <benwells@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Date: Wed, 11 Jul 2012 06:10:07 +0000 Subject: Remove app shortcuts when app is uninstalled on Linux. To support this, shortcut creation on Linux for extensions has been modified so that the filename encodes the extension ID and the profile. Also, when creating shortcuts any existing shortcuts are removed first. Web page shortcuts are not affected. BUG=130456 TEST=Test uninstalling apps removes their shortcuts; test uninstalling apps is not broken in any way; test shortcuts for web apps are not broken in any way. Review URL: https://chromiumcodereview.appspot.com/10698114 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146065 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/shell_integration_unittest.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'chrome/browser/shell_integration_unittest.cc') diff --git a/chrome/browser/shell_integration_unittest.cc b/chrome/browser/shell_integration_unittest.cc index 63921e7..31ea4a3 100644 --- a/chrome/browser/shell_integration_unittest.cc +++ b/chrome/browser/shell_integration_unittest.cc @@ -139,7 +139,7 @@ TEST(ShellIntegrationTest, GetDesktopShortcutTemplate) { } } -TEST(ShellIntegrationTest, GetDesktopShortcutFilename) { +TEST(ShellIntegrationTest, GetWebShortcutFilename) { const struct { const FilePath::CharType* path; const char* url; @@ -156,7 +156,7 @@ TEST(ShellIntegrationTest, GetDesktopShortcutFilename) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(test_cases); i++) { EXPECT_EQ(std::string(chrome::kBrowserProcessExecutableName) + "-" + test_cases[i].path, - ShellIntegrationLinux::GetDesktopShortcutFilename( + ShellIntegrationLinux::GetWebShortcutFilename( GURL(test_cases[i].url)).value()) << " while testing " << test_cases[i].url; } -- cgit v1.1