Add an UMA for file types selected when Quick View is launched.

Add an UMA for file types selected when Quick View is launched.

BUG=627697
TEST=opened a file and moved to the next with <Right> and checked chrome://histograms. Confirmed 1 FileBrowser.QuickView.FileTypeOnLaunch is exported and 2 FileBrowser.QuickView.FileType are exported.

Committed: https://crrev.com/621feebe12cd4625776cd1a9099acd1c22a176de
Cr-Commit-Position: refs/heads/master@{#412761}

[oka, 2016-08-17 06:11:22]

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

[oka, 2016-08-17 06:11:23]

PTAL.
asvitkine@ for tools/metrics/histograms/histograms.xml.
Thanks.

[oka, 2016-08-17 06:12:07]

Description was changed from

==========
Add an UMA for selected file types when Quick View is launched.

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

to

==========
Add an UMA for file types selected when Quick View is launched.

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

[oka, 2016-08-17 06:15:08]

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

[oka, 2016-08-17 06:15:09]

+holte@ for tools/metrics/histograms/histograms.xml.
(because reviewer name of Alexei contains 'very slow'.)

Thanks.

[Steven Holte, 2016-08-17 21:58:31]

lgtm

[oka, 2016-08-18 01:57:08]

oka@chromium.org changed reviewers:
- asvitkine@chromium.org

[oka, 2016-08-18 01:57:25]

On 2016/08/17 21:58:31, Steven Holte wrote:
> lgtm

Thank you Steven.

[fukino, 2016-08-18 04:39:22]

lgtm with a request.

Please be more descriptive on TEST= line (you checked chrome://histograms or
chrome://user-actions, right?), since the issue doesn't have a specific repro
steps.

[oka, 2016-08-18 05:48:40]

Description was changed from

==========
Add an UMA for file types selected when Quick View is launched.

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

to

==========
Add an UMA for file types selected when Quick View is launched.

BUG=627697
TEST=opened a file and moved to the next with <Right> and checked
chrome://histograms. Confirmed 1 FileBrowser.QuickView.FileTypeOnLaunch is
exported and 2 FileBrowser.QuickView.FileType are exported.
==========

[oka, 2016-08-18 05:49:05]

On 2016/08/18 04:39:22, fukino wrote:
> lgtm with a request.
> 
> Please be more descriptive on TEST= line (you checked chrome://histograms or
> chrome://user-actions, right?), since the issue doesn't have a specific repro
> steps.

Done. Thanks!

[oka, 2016-08-18 05:49:06]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-08-18 05:49:20]

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-18 08:09:44]

Description was changed from

==========
Add an UMA for file types selected when Quick View is launched.

BUG=627697
TEST=opened a file and moved to the next with <Right> and checked
chrome://histograms. Confirmed 1 FileBrowser.QuickView.FileTypeOnLaunch is
exported and 2 FileBrowser.QuickView.FileType are exported.
==========

to

==========
Add an UMA for file types selected when Quick View is launched.

BUG=627697
TEST=opened a file and moved to the next with <Right> and checked
chrome://histograms. Confirmed 1 FileBrowser.QuickView.FileTypeOnLaunch is
exported and 2 FileBrowser.QuickView.FileType are exported.
==========

[commit-bot: I haz the power, 2016-08-18 08:09:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-18 08:13:28]

Description was changed from

==========
Add an UMA for file types selected when Quick View is launched.

BUG=627697
TEST=opened a file and moved to the next with <Right> and checked
chrome://histograms. Confirmed 1 FileBrowser.QuickView.FileTypeOnLaunch is
exported and 2 FileBrowser.QuickView.FileType are exported.
==========

to

==========
Add an UMA for file types selected when Quick View is launched.

BUG=627697
TEST=opened a file and moved to the next with <Right> and checked
chrome://histograms. Confirmed 1 FileBrowser.QuickView.FileTypeOnLaunch is
exported and 2 FileBrowser.QuickView.FileType are exported.

Committed: https://crrev.com/621feebe12cd4625776cd1a9099acd1c22a176de
Cr-Commit-Position: refs/heads/master@{#412761}
==========

[commit-bot: I haz the power, 2016-08-18 08:13:29]

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