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

Issue 2304233002: [Merge-M53] arc: Open launcher after installing app from Android (Closed)

Created:
4 years, 3 months ago by khmel
Modified:
4 years, 3 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2785
Project:
chromium
Visibility:
Public.

Description

arc: Open launcher after installing app from Android This Implements showing app list when user installs new package. App list is not shown for app from system packages, or auxulary packages. App list is shown on the page that contains newly installed app. BUG=b/30953079 BUG=640226 TEST=unit_tests extended browser_tests TEST=manually on device TBR=reviewer@chromium.org NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2276553002 Cr-Commit-Position: refs/heads/master@{#414100} (cherry picked from commit 29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33)

Patch Set 1 #

Patch Set 2 : resolve browser_tests #

Unified diffs Side-by-side diffs Delta from patch set Stats (+106 lines, -17 lines) Patch
M chrome/browser/ui/app_list/arc/arc_app_list_prefs.h View 3 chunks +9 lines, -2 lines 0 comments Download
M chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc View 1 5 chunks +44 lines, -4 lines 0 comments Download
M chrome/browser/ui/app_list/arc/arc_package_syncable_service.h View 2 chunks +4 lines, -1 line 0 comments Download
M chrome/browser/ui/app_list/arc/arc_package_syncable_service.cc View 2 chunks +7 lines, -2 lines 0 comments Download
M chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc View 1 6 chunks +33 lines, -4 lines 0 comments Download
M ui/app_list/views/app_list_main_view.h View 1 chunk +2 lines, -2 lines 0 comments Download
M ui/app_list/views/app_list_main_view.cc View 2 chunks +7 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (3 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2304233002/20001
4 years, 3 months ago (2016-09-02 20:03:12 UTC) #3
commit-bot: I haz the power
4 years, 3 months ago (2016-09-02 20:06:53 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001)

Powered by Google App Engine
This is Rietveld 408576698