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

Issue 300623004: Create start menu shortcut when an ephemeral app is promoted (Closed)

Created:
6 years, 7 months ago by tmdiep
Modified:
6 years, 6 months ago
Reviewers:
benwells
CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@install_base
Visibility:
Public.

Description

Create start menu shortcut when an ephemeral app is promoted Start menu shortcuts are created during the installation process for regular apps, but not ephemeral apps. This patch ensures that shortcuts are created when ephemeral apps are promoted to regular installed apps. This patch also adds a check to exclude extensions before creating or updating shortcuts. BUG=374018 TEST=Launch an app ephemerally and verify that start menu shortcuts have not been added. Install the app and verify that shortcuts are added. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273274

Patch Set 1 #

Patch Set 2 : Rebase and optimize for extensions #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M chrome/browser/apps/shortcut_manager.cc View 1 1 chunk +4 lines, -1 line 1 comment Download

Messages

Total messages: 6 (0 generated)
tmdiep
A small patch for a change :) Follows https://codereview.chromium.org/297263003. I couldn't find an easy way ...
6 years, 6 months ago (2014-05-28 05:13:34 UTC) #1
benwells
lgtm
6 years, 6 months ago (2014-05-28 06:26:17 UTC) #2
tmdiep
The CQ bit was checked by tmdiep@chromium.org
6 years, 6 months ago (2014-05-28 14:49:12 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tmdiep@chromium.org/300623004/40001
6 years, 6 months ago (2014-05-28 14:51:36 UTC) #4
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: android_chromium_gn_compile_rel on tryserver.chromium ...
6 years, 6 months ago (2014-05-28 16:48:49 UTC) #5
commit-bot: I haz the power
6 years, 6 months ago (2014-05-28 16:58:14 UTC) #6
Message was sent while issue was closed.
Change committed as 273274

Powered by Google App Engine
This is Rietveld 408576698