Respect --profile-directory flag when used with --show-app-list flag.

Respect --profile-directory flag when used with --show-app-list flag.

This CL allows specification of the profile to show the app list for when
launching the app list via the command line.

Currently --show-app-list always uses the last used app list profile, but we
want to allow testers to more easily load the app list for a particular
profile.

On Windows, this can be used for making shorcuts to a particular profile's
app list but this will always bind to the main shortcut due to the app id.

BUG=400874
Committed: https://crrev.com/5814471fa14e46aacaf7bc61009aac89452ac400
Cr-Commit-Position: refs/heads/master@{#292291}

[calamity, 2014-08-26 05:11:30]

calamity@chromium.org changed reviewers:
+ tapted@chromium.org

[tapted, 2014-08-26 05:38:03]

lgtm - I would just add a bit more context to the CL description. Something like

Currently --show-app-list always uses the last used app list profile, but we
want to allow power-users to manually configure app list shortcuts for specific
profiles.


Also.. just check what happens when you have two shortcuts, and click one while
an app list for the other is already showing. I think it will just switch the
visible app list profile rather than reshowing, which is probably OK. But we
wouldn't want the keep alive to die or do other bad stuff.

Aand, I think the app list window won't necessarily be "attached" to the correct
app list shortcut, since the win32 app id won't be different. It would be good
to note these limitations in the CL description as well.

[calamity, 2014-08-27 06:19:46]

On 2014/08/26 05:38:03, tapted wrote:
> lgtm - I would just add a bit more context to the CL description. Something
like
> 
> Currently --show-app-list always uses the last used app list profile, but we
> want to allow power-users to manually configure app list shortcuts for
specific
> profiles.
> 
Done.
> 
> Also.. just check what happens when you have two shortcuts, and click one
while
> an app list for the other is already showing. I think it will just switch the
> visible app list profile rather than reshowing, which is probably OK. But we
> wouldn't want the keep alive to die or do other bad stuff.
> 
Seems to be fine. It does indeed just switch.
> Aand, I think the app list window won't necessarily be "attached" to the
correct
> app list shortcut, since the win32 app id won't be different. It would be good
> to note these limitations in the CL description as well.
Done.

[calamity, 2014-08-27 06:20:03]

calamity@chromium.org changed reviewers:
+ sky@chromium.org

[calamity, 2014-08-27 06:20:03]

+sky for OWNERS.

[sky, 2014-08-27 15:15:08]

LGTM

[calamity, 2014-08-28 00:06:40]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-08-28 00:07:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/calamity@chromium.org/500333002/1

[commit-bot: I haz the power, 2014-08-28 01:20:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-28 01:20:07]

Failed to commit the patch.

[calamity, 2014-08-28 01:52:54]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-08-28 01:53:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/calamity@chromium.org/500333002/1

[commit-bot: I haz the power, 2014-08-28 01:55:01]

Committed patchset #1 (id:1) as a97712ec2f7db8be779c09d3d0d037d6a6cba068

[commit-bot: I haz the power, 2014-09-10 02:56:24]

Patchset 1 (id:??) landed as
https://crrev.com/5814471fa14e46aacaf7bc61009aac89452ac400
Cr-Commit-Position: refs/heads/master@{#292291}