Storage manager: Hide "Offline files" when Google Drive volume is not mounted.

Storage manager: Hide "Offline files" when Google Drive volume is not mounted.

The "Google Drive" volume in Files app does not always exist.
(e.g. Guest session, Supervised user, or disabled by setting.)
Showing "Offline files" in storage manager doesn't make sense in such cases and
causes a crash.
We should hide "Offline files" item when Google Drive volume doesn't exist.

BUG=629347
TEST=manually tested on a guest user and a supervised user.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[fukino, 2016-07-27 11:08:11]

Description was changed from

==========
Storage manager: Hide "Offline files" when Google Drive volume is not mounted.

The "Google Drive" volume in Files app does not always exist.
(e.g. Guest session, Supervised user, or disabled by setting.)
Showing "Offline files" in storage manager doesn't make sense in such cases and
causes a crash.
We should hide "Offline files" item when Google Drive volume doesn't exist.

BUG=629347
TEST=manually tested on a guest user and a supervised user.
==========

to

==========
Storage manager: Hide "Offline files" when Google Drive volume is not mounted.

The "Google Drive" volume in Files app does not always exist.
(e.g. Guest session, Supervised user, or disabled by setting.)
Showing "Offline files" in storage manager doesn't make sense in such cases and
causes a crash.
We should hide "Offline files" item when Google Drive volume doesn't exist.

BUG=629347
TEST=manually tested on a guest user and a supervised user.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[fukino, 2016-07-27 11:09:32]

fukino@chromium.org changed reviewers:
+ dbeam@chromium.org

[fukino, 2016-07-27 11:09:33]

PTAL.

[Dan Beam, 2016-07-27 13:03:11]

lgtm although maybe you should start thinking about tests if this is causing
crashes

[fukino, 2016-07-28 00:52:52]

On 2016/07/27 13:03:11, Dan Beam wrote:
> lgtm although maybe you should start thinking about tests if this is causing
> crashes

Thanks! I'll add a test in a follow-up CL.

[fukino, 2016-07-28 00:53:05]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2016-07-28 00:53:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 02:30:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-28 02:31:46]

Description was changed from

==========
Storage manager: Hide "Offline files" when Google Drive volume is not mounted.

The "Google Drive" volume in Files app does not always exist.
(e.g. Guest session, Supervised user, or disabled by setting.)
Showing "Offline files" in storage manager doesn't make sense in such cases and
causes a crash.
We should hide "Offline files" item when Google Drive volume doesn't exist.

BUG=629347
TEST=manually tested on a guest user and a supervised user.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Storage manager: Hide "Offline files" when Google Drive volume is not mounted.

The "Google Drive" volume in Files app does not always exist.
(e.g. Guest session, Supervised user, or disabled by setting.)
Showing "Offline files" in storage manager doesn't make sense in such cases and
causes a crash.
We should hide "Offline files" item when Google Drive volume doesn't exist.

BUG=629347
TEST=manually tested on a guest user and a supervised user.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2016-07-28 02:31:47]

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