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

Issue 11741035: Exclude hosted apps from post-sideload UI. (Closed)

Created:
7 years, 11 months ago by Matt Perry
Modified:
7 years, 11 months ago
Reviewers:
Yoyo Zhou
CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Exclude hosted apps from post-sideload UI. BUG=168243 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175200

Patch Set 1 #

Patch Set 2 : test fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -5 lines) Patch
chrome/browser/extensions/extension_service.cc View 1 chunk +4 lines, -2 lines 0 comments Download
chrome/browser/extensions/extension_service_unittest.cc View 1 1 chunk +1 line, -3 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Matt Perry
7 years, 11 months ago (2013-01-04 18:12:51 UTC) #1
Yoyo Zhou
LGTM Does the same logic apply to platform apps?
7 years, 11 months ago (2013-01-04 18:21:50 UTC) #2
Matt Perry
On 2013/01/04 18:21:50, Yoyo Zhou wrote: > LGTM > Does the same logic apply to ...
7 years, 11 months ago (2013-01-04 18:40:55 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/11741035/1
7 years, 11 months ago (2013-01-04 18:42:03 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/11741035/12001
7 years, 11 months ago (2013-01-04 19:42:27 UTC) #5
commit-bot: I haz the power
7 years, 11 months ago (2013-01-04 21:49:57 UTC) #6
Message was sent while issue was closed.
Change committed as 175200

Powered by Google App Engine
This is Rietveld 408576698