Add UMA for Quick View (file type and volume type).

Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually (using Linux, checked chrome://histograms.)

Committed: https://crrev.com/3a0160731876cbef11d9d7a7365c33656599fd45
Cr-Commit-Position: refs/heads/master@{#409755}

[oka, 2016-07-27 08:09:42]

The CQ bit was checked by oka@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-27 08:10:01]

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

[oka, 2016-07-27 08:10:27]

The CQ bit was unchecked by oka@chromium.org

[oka, 2016-07-27 08:13:32]

nit

[oka, 2016-07-27 08:14:24]

The CQ bit was checked by oka@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-27 08:14:37]

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

[oka, 2016-07-27 08:21:28]

Description was changed from

==========
WIP: QuickView: Add UMA
==========

to

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually
==========

[oka, 2016-07-27 08:21:29]

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

[oka, 2016-07-27 08:21:42]

PTAL.

[oka, 2016-07-27 08:22:15]

Description was changed from

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually
==========

to

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually (using Linux, checked chrome://histograms.)
==========

[fukino, 2016-07-27 09:21:37]

lgtm

[oka, 2016-08-01 08:05:37]

oka@chromium.org changed reviewers:
+ holte@chromium.org

[oka, 2016-08-01 08:05:38]

+holte@ for histograms.xml.

[oka, 2016-08-02 00:11:20]

On 2016/08/01 08:05:38, oka wrote:
> +holte@ for histograms.xml.

holte, friendly ping.

[Steven Holte, 2016-08-02 19:24:38]

lgtm

[oka, 2016-08-02 23:05:01]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-08-02 23:05:21]

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-08-02 23:07:42]

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

[commit-bot: I haz the power, 2016-08-02 23:07:43]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[oka, 2016-08-03 09:17:30]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-08-03 09:17:51]

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-08-03 09:21:54]

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

[commit-bot: I haz the power, 2016-08-03 09:21:55]

Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[oka, 2016-08-04 07:56:29]

Rebase.

[oka, 2016-08-04 07:59:32]

Rebase.

[oka, 2016-08-04 08:59:45]

The CQ bit was checked by oka@chromium.org

[oka, 2016-08-04 08:59:46]

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

[commit-bot: I haz the power, 2016-08-04 08:59:57]

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-08-04 10:16:35]

Description was changed from

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually (using Linux, checked chrome://histograms.)
==========

to

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually (using Linux, checked chrome://histograms.)
==========

[commit-bot: I haz the power, 2016-08-04 10:16:36]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-08-04 10:18:20]

Description was changed from

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually (using Linux, checked chrome://histograms.)
==========

to

==========
Add UMA for Quick View (file type and volume type).

FileBrowser.QuickView.FileType   - File type shown in Quick View.
FileBrowser.QuickView.VolumeType - Volume type where Quick View is opened.

BUG=627697
TEST=manually (using Linux, checked chrome://histograms.)

Committed: https://crrev.com/3a0160731876cbef11d9d7a7365c33656599fd45
Cr-Commit-Position: refs/heads/master@{#409755}
==========

[commit-bot: I haz the power, 2016-08-04 10:18:21]

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