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

Issue 23935004: Preserve the app in the AppLauncherHandler if it's disabled. (Closed)

Created:
7 years, 3 months ago by vasilii
Modified:
7 years, 3 months ago
Reviewers:
Evan Stade
CC:
chromium-reviews, dbeam+watch-ntp_chromium.org, estade+watch_chromium.org, pedrosimonetti+watch_chromium.org, battre, Yoyo Zhou
Visibility:
Public.

Description

Preserve the app in the AppLauncherHandler if it's disabled. BUG=284400 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221673

Patch Set 1 #

Patch Set 2 : tiny change for consistency #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M chrome/browser/ui/webui/ntp/app_launcher_handler.cc View 1 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
vasilii
Hi Evan, please review this Stable Blocker.
7 years, 3 months ago (2013-09-05 09:26:34 UTC) #1
Evan Stade
lgtm
7 years, 3 months ago (2013-09-05 17:05:51 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vasilii@chromium.org/23935004/9001
7 years, 3 months ago (2013-09-06 07:33:40 UTC) #3
commit-bot: I haz the power
7 years, 3 months ago (2013-09-06 10:28:24 UTC) #4
Message was sent while issue was closed.
Change committed as 221673

Powered by Google App Engine
This is Rietveld 408576698