Add entries for common FSP providers so we get real names in Analytics places.

Add entries for common FSP providers so we get real names in Analytics places.

TEST=manual

Committed: https://crrev.com/118856988dba956fda8dc30e4fdeb3f18baa5261
Cr-Commit-Position: refs/heads/master@{#330016}

[Steve McKay, 2015-05-13 20:55:20]

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

[Steve McKay, 2015-05-13 20:56:57]

NOTE:
(YT) == Yoichiro Tanaka
(FB) == François Beaufort

I thought it would be good to qualify these in case other similarly named
providers come along.

[Steve McKay, 2015-05-13 20:59:34]

Oh. And lastly, I based this on the current reporting from analytics:
http://go/filesapp-unknown-providers

Once we add an entry to metrics.FileSystemProviders, an unknown provider hit
will not be sent for the corresponding mount.

[fukino, 2015-05-13 23:12:00]

lgtm with optional nits.

https://codereview.chromium.org/1144433003/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/common/js/metrics_events.js (right):

https://codereview.chromium.org/1144433003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/common/js/metrics_events.js:36: 
optional nit: We usually use one blank line between top-level blocks.

https://codereview.chromium.org/1144433003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/common/js/metrics_events.js:61: 
ditto

[Steve McKay, 2015-05-15 01:03:20]

Respond to review comments.

[Steve McKay, 2015-05-15 01:03:35]

The CQ bit was checked by smckay@chromium.org

[Steve McKay, 2015-05-15 01:03:35]

The patchset sent to the CQ was uploaded after l-g-t-m from fukino@chromium.org
Link to the patchset: https://codereview.chromium.org/1144433003/#ps20001
(title: "Respond to review comments.")

[Steve McKay, 2015-05-15 01:04:20]

Dones. Thanks!

https://codereview.chromium.org/1144433003/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/common/js/metrics_events.js (right):

https://codereview.chromium.org/1144433003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/common/js/metrics_events.js:36: 
On 2015/05/13 23:12:00, fukino wrote:
> optional nit: We usually use one blank line between top-level blocks.

Done.

https://codereview.chromium.org/1144433003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/common/js/metrics_events.js:61: 
On 2015/05/13 23:12:00, fukino wrote:
> ditto

Done.

[commit-bot: I haz the power, 2015-05-15 01:05:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1144433003/20001

[commit-bot: I haz the power, 2015-05-15 01:40:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-15 01:42:02]

Patchset 2 (id:??) landed as
https://crrev.com/118856988dba956fda8dc30e4fdeb3f18baa5261
Cr-Commit-Position: refs/heads/master@{#330016}