Add file types to FileBrowser.ViewingFileType UMA stat.

Add file types to FileBrowser.ViewingFileType UMA stat.

*.crdownload
*.crx
*.dmg
*.exe
*.html
*.jar
*.ps
*.torrent
*.txt
*.zip

BUG=397181
TEST=manually
1.Go to chrome://histogram.
2.Open the each file.
3.Confirtm the count is increased.

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

[iseki, 2014-09-04 05:58:59]

iseki@chromium.org changed reviewers:
+ hirono@chromium.org

[iseki, 2014-09-04 05:58:59]

Please take a look.

[hirono, 2014-09-04 06:23:39]

On 2014/09/04 05:58:59, iseki wrote:
> Please take a look.

Please also update.
https://code.google.com/p/chromium/codesearch#chromium/src/tools/metrics/hist...

[hirono, 2014-09-04 06:25:56]

https://codereview.chromium.org/539853002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/file_tasks.js (right):

https://codereview.chromium.org/539853002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/file_tasks.js:134: '.crdownload',
'.crx', '.dmg', '.doc', '.docx', '.exe', '.flac', '.gif',
We should not change the index of existing item to prevent from breaking the
existing data.
Please add new extensions at the end of list.

[iseki, 2014-09-04 06:50:47]

Thank you for your review.

https://codereview.chromium.org/539853002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/file_tasks.js (right):

https://codereview.chromium.org/539853002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/file_tasks.js:134: '.crdownload',
'.crx', '.dmg', '.doc', '.docx', '.exe', '.flac', '.gif',
On 2014/09/04 06:25:56, hirono wrote:
> We should not change the index of existing item to prevent from breaking the
> existing data.
> Please add new extensions at the end of list.

Done.

[hirono, 2014-09-04 06:57:25]

lgtm with a nit.

https://codereview.chromium.org/539853002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/file_tasks.js (right):

https://codereview.chromium.org/539853002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_tasks.js:139: '.html', '.jar',
'.ps', '.torrent', '.txt', '.zip',
Maybe we'd like to track .htm also as well as the .jpg/.jpeg pair at #134.

[iseki, 2014-09-04 07:01:07]

Thank you for your reveiw :)

https://codereview.chromium.org/539853002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/file_tasks.js (right):

https://codereview.chromium.org/539853002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_tasks.js:139: '.html', '.jar',
'.ps', '.torrent', '.txt', '.zip',
On 2014/09/04 06:57:25, hirono wrote:
> Maybe we'd like to track .htm also as well as the .jpg/.jpeg pair at #134.

Done.

[iseki, 2014-09-04 07:03:55]

iseki@chromium.org changed reviewers:
+ mpearson@chromium.org

[iseki, 2014-09-04 07:03:55]

Please take a look.

[iseki, 2014-09-04 07:05:58]

@Mark P
Please take a look following file:
 tools/metrics/histograms/histograms.xml

[Mark P, 2014-09-04 18:23:50]

lgtm

[iseki, 2014-09-04 23:20:35]

The CQ bit was checked by iseki@chromium.org

[commit-bot: I haz the power, 2014-09-04 23:24:03]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/iseki@chromium.org/539853002/40001

[commit-bot: I haz the power, 2014-09-05 04:28:50]

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

[commit-bot: I haz the power, 2014-09-05 04:28:50]

Try jobs failed on following builders:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[iseki, 2014-09-05 04:31:43]

The CQ bit was checked by iseki@chromium.org

[commit-bot: I haz the power, 2014-09-05 04:33:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/iseki@chromium.org/539853002/40001

[commit-bot: I haz the power, 2014-09-05 05:46:13]

Committed patchset #3 (id:40001) as d0caf34b008819781f8687c25d09b3157d82ec5b

[commit-bot: I haz the power, 2014-09-10 03:37:20]

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