cros: Enable Drive apps integration in app launcher.

cros: Enable Drive apps integration in app launcher.

BUG=412954
Committed: https://crrev.com/fde6f25b66d632e49751365559841c1e1c436856
Cr-Commit-Position: refs/heads/master@{#294610}

[xiyuan, 2014-09-10 20:59:55]

xiyuan@chromium.org changed reviewers:
+ abodenha@chromium.org

[Albert Bodenhamer, 2014-09-10 21:33:32]

I'd kinda prefer to keep the flag but reverse its default behavior.

We still have the accessor. It'd still be easy to turn this off, but I'd feel a
bit more comfortable leaving the flag around.

[xiyuan, 2014-09-10 21:58:07]

CL updated to use a disable flag instead of removing enable flag.

[Albert Bodenhamer, 2014-09-10 22:04:27]

lgtm

[xiyuan, 2014-09-11 21:57:26]

The CQ bit was checked by xiyuan@chromium.org

[commit-bot: I haz the power, 2014-09-11 22:24:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/556403002/40001

[commit-bot: I haz the power, 2014-09-12 00:58:14]

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

[commit-bot: I haz the power, 2014-09-12 00:58:15]

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

[xiyuan, 2014-09-12 01:03:43]

The CQ bit was checked by xiyuan@chromium.org

[commit-bot: I haz the power, 2014-09-12 01:04:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/556403002/40001

[commit-bot: I haz the power, 2014-09-12 02:15:08]

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

[commit-bot: I haz the power, 2014-09-12 02:15:10]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[xiyuan, 2014-09-12 03:26:16]

xiyuan@chromium.org changed reviewers:
+ isherman@chromium.org

[xiyuan, 2014-09-12 03:26:17]

Ilya, could I get an owner stamp for histograms.xml? Thanks.

[Ilya Sherman, 2014-09-12 07:29:40]

Histograms LGTM % a nit:

https://codereview.chromium.org/556403002/diff/60001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/556403002/diff/60001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:44626: -  <int value="2137347307"
label="enable-drive-apps-in-app-list"/>
Please keep this mapping as well.

[xiyuan, 2014-09-12 16:22:40]

https://codereview.chromium.org/556403002/diff/60001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/556403002/diff/60001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:44626: -  <int value="2137347307"
label="enable-drive-apps-in-app-list"/>
On 2014/09/12 07:29:40, Ilya Sherman wrote:
> Please keep this mapping as well.

Done.

[xiyuan, 2014-09-12 16:22:54]

The CQ bit was checked by xiyuan@chromium.org

[commit-bot: I haz the power, 2014-09-12 16:23:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/556403002/80001

[commit-bot: I haz the power, 2014-09-12 17:26:20]

Committed patchset #5 (id:80001) as 02d8753c6773164396d1ab1e646ffecce8cd9406

[commit-bot: I haz the power, 2014-09-12 17:30:46]

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