Add inline completion to quicklaunch

Add inline completion to quicklaunch

R=sky@chromium.org
BUG=

Committed: https://crrev.com/5aaf128d3a64093322e48cee4b934d78a1dcdb4c
Cr-Commit-Position: refs/heads/master@{#384404}

[sky, 2016-03-31 17:29:32]

LGTM

https://codereview.chromium.org/1850543002/diff/40001/mash/quick_launch/quick...
File mash/quick_launch/quick_launch_application.cc (right):

https://codereview.chromium.org/1850543002/diff/40001/mash/quick_launch/quick...
mash/quick_launch/quick_launch_application.cc:73: case ui::VKEY_RETURN: {
nit: run git cl format as your indentation is off.

https://codereview.chromium.org/1850543002/diff/40001/mash/quick_launch/quick...
mash/quick_launch/quick_launch_application.cc:137: std::set<base::string16>
names_;
names_ is rather vague, app_names_?

https://codereview.chromium.org/1850543002/diff/40001/mojo/services/catalog/p...
File mojo/services/catalog/public/interfaces/catalog.mojom (right):

https://codereview.chromium.org/1850543002/diff/40001/mojo/services/catalog/p...
mojo/services/catalog/public/interfaces/catalog.mojom:12: // If |names| is null,
all available packages are returned.
packages->entries?

[Ben Goodger (Google), 2016-03-31 17:37:35]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-03-31 17:37:35]

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/1850543002/#ps60001
(title: ".")

[commit-bot: I haz the power, 2016-03-31 17:38:00]

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

[commit-bot: I haz the power, 2016-03-31 18:06:59]

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

[commit-bot: I haz the power, 2016-03-31 18:07:00]

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

[Ben Goodger (Google), 2016-03-31 19:11:38]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-03-31 19:11:39]

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/1850543002/#ps80001
(title: ".")

[commit-bot: I haz the power, 2016-03-31 19:11:59]

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

[commit-bot: I haz the power, 2016-03-31 21:49:37]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-31 21:51:04]

Description was changed from

==========
Add inline completion to quicklaunch

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

to

==========
Add inline completion to quicklaunch

R=sky@chromium.org
BUG=

Committed: https://crrev.com/5aaf128d3a64093322e48cee4b934d78a1dcdb4c
Cr-Commit-Position: refs/heads/master@{#384404}
==========

[commit-bot: I haz the power, 2016-03-31 21:51:05]

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