Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(268)

Issue 11275333: Merge 166285 - Re-commit: Introduce RemoveChromeTaskbarShortcuts() to delete all pinned-to-taskbar … (Closed)

Created:
8 years, 1 month ago by gab
Modified:
8 years, 1 month ago
Reviewers:
gab
CC:
chromium-reviews, grt+watch_chromium.org, erikwright+watch_chromium.org
Visibility:
Public.

Description

Merge 166285 - Re-commit: Introduce RemoveChromeTaskbarShortcuts() to delete all pinned-to-taskbar shortcuts owned by the uninstalled Chrome. Original commit http://crrev.com/165505 Reverted in http://crrev.com/165524 BUG=158632 TEST=No user-level shortcut Chrome left behind in taskbar post user-level self-destruct. Review URL: https://chromiumcodereview.appspot.com/11368040 TBR=gab@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=167788

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+77 lines, -15 lines) Patch
M base/base_paths_win.h View 1 chunk +2 lines, -0 lines 0 comments Download
M base/base_paths_win.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M base/path_service_unittest.cc View 1 chunk +22 lines, -13 lines 0 comments Download
M chrome/browser/shell_integration_win.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/installer/setup/uninstall.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/installer/util/shell_util.h View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/installer/util/shell_util.cc View 1 chunk +31 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
gab
8 years, 1 month ago (2012-11-15 00:29:49 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698