Add quick launcher for launching mojo apps from within mash

Add quick launcher for launching mojo apps from within mash

R=sky@chromium.org
BUG=

Committed: https://crrev.com/ae9770b4de6eedaf41b5e2ab1f7b9c6b7c0d2421
Cr-Commit-Position: refs/heads/master@{#367783}

[sky, 2016-01-05 23:33:56]

https://codereview.chromium.org/1560223002/diff/20001/mash/quick_launch/quick...
File mash/quick_launch/quick_launch.cc (right):

https://codereview.chromium.org/1560223002/diff/20001/mash/quick_launch/quick...
mash/quick_launch/quick_launch.cc:32: QuickLaunchUI(mojo::ApplicationImpl* app)
nit: explicit

https://codereview.chromium.org/1560223002/diff/20001/mash/quick_launch/quick...
mash/quick_launch/quick_launch.cc:69: base::TrimWhitespaceASCII(working,
base::TRIM_ALL, &working);
Isn't working utf8, and not ascii? Seems like you should strip it using the
string16 variant, then convert to utf8 and append mojo.

https://codereview.chromium.org/1560223002/diff/20001/mash/quick_launch/quick...
mash/quick_launch/quick_launch.cc:97: new QuickLaunchUI(app), gfx::Rect(10, 625,
500, 75));
Wouldn't it be better to let the preferred height come from the textfield?
Otherwise we'll have to futz with this value as we change assets/fonts.

[sky, 2016-01-06 00:08:35]

LGTM

[Ben Goodger (Google), 2016-01-06 00:40:54]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-06 00:40:54]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1560223002/#ps80001
(title: ".")

[commit-bot: I haz the power, 2016-01-06 00:42:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1560223002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1560223002/80001

[commit-bot: I haz the power, 2016-01-06 00:57:31]

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

[commit-bot: I haz the power, 2016-01-06 00:57:31]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ben Goodger (Google), 2016-01-06 04:47:11]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-06 04:47:12]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1560223002/#ps100001
(title: ".")

[commit-bot: I haz the power, 2016-01-06 04:47:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1560223002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1560223002/100001

[commit-bot: I haz the power, 2016-01-06 05:33:48]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-01-06 05:35:19]

Description was changed from

==========
Add quick launcher for launching mojo apps from within mash

R=sky@chromium.org
BUG=
==========

to

==========
Add quick launcher for launching mojo apps from within mash

R=sky@chromium.org
BUG=

Committed: https://crrev.com/ae9770b4de6eedaf41b5e2ab1f7b9c6b7c0d2421
Cr-Commit-Position: refs/heads/master@{#367783}
==========

[commit-bot: I haz the power, 2016-01-06 05:35:20]

Patchset 6 (id:??) landed as
https://crrev.com/ae9770b4de6eedaf41b5e2ab1f7b9c6b7c0d2421
Cr-Commit-Position: refs/heads/master@{#367783}