Chromium Code Reviews
Help | Chromium Project | Sign in
(151)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 8 months ago by gab
Modified:
2 years, 8 months 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
Trybot results:
Commit: CQ not working?

Messages

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

          
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 5fa3ca5