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

Issue 2361573002: [Merge-M54] arc: Prevent showing Arc app window for secondary user profile. (Closed)

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

Description

[Merge-M54] arc: Prevent showing Arc app window for secondary user profile. Arc has a limitation to be visible for primary profile only. However, there are situations when Arc task can be started while the secondary profile is shown. Before this led to case when Arc window was shown, which is unexpected. This CL fixes this issue and prevents showing Arc window for non-primary profiles. Test=Manually on the device. Start Arc app in deferred mode and switch to the secondary profile fast. Arc app is started but window is not shown. Switch back to primary profile and window is automatically restored there. BUG=b/31500778 BUG=647379 BUG=644751 TBR=reviewer@chromium.org NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2345043002 Cr-Commit-Position: refs/heads/master@{#419497} (cherry picked from commit a4d9c53e0ca156489f3a9b970925594786bab1c0)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -14 lines) Patch
M chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_controller.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.cc View 4 chunks +27 lines, -13 lines 0 comments Download

Messages

Total messages: 3 (1 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/2361573002/1
4 years, 3 months ago (2016-09-21 20:30:15 UTC) #2
commit-bot: I haz the power
4 years, 3 months ago (2016-09-21 20:33:51 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698