Add support for reporting history load with counts + timing + a 'multi-machine' dimension.

Add support for reporting history load with counts + timing +  a 'multi-machine' dimension.

TEST=*ImportHistory*

TBR=hirono  // for background.js

Committed: https://crrev.com/875b896a3256c5b86c8725e81489e99ea6c2b4c9
Cr-Commit-Position: refs/heads/master@{#321232}

[Steve McKay, 2015-03-18 18:19:52]

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

[Steve McKay, 2015-03-18 18:23:59]

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

[Steve McKay, 2015-03-18 18:23:59]

+hirono for background.js

[Ben Kwa, 2015-03-18 21:37:15]

lgtm with a couple of questions.

https://codereview.chromium.org/1008723004/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/background/js/import_history.js (right):

https://codereview.chromium.org/1008723004/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/background/js/import_history.js:783: * @param {?}
ignored
Is this here to stop the compiler from complaining about something?

https://codereview.chromium.org/1008723004/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/background/js/import_history.js:1081:
this.createRealHistoryLoader_ = function() {
Why'd you do this instead of making this a proper member function?

[Steve McKay, 2015-03-18 22:47:22]

smckay@chromium.org changed reviewers:
- hirono@chromium.org

[Steve McKay, 2015-03-18 22:47:30]

The CQ bit was checked by smckay@chromium.org

[Steve McKay, 2015-03-18 22:47:51]

Thanks!

https://codereview.chromium.org/1008723004/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/background/js/import_history.js (right):

https://codereview.chromium.org/1008723004/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/background/js/import_history.js:783: * @param {?}
ignored
On 2015/03/18 21:37:15, Ben Kwa wrote:
> Is this here to stop the compiler from complaining about something?

Honestly, I don't know. I just typed it according to the field's type.

https://codereview.chromium.org/1008723004/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/background/js/import_history.js:1081:
this.createRealHistoryLoader_ = function() {
On 2015/03/18 21:37:15, Ben Kwa wrote:
> Why'd you do this instead of making this a proper member function?

No, I did this to avoid saving off tracker in a field.

[commit-bot: I haz the power, 2015-03-18 22:48:13]

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

[commit-bot: I haz the power, 2015-03-18 23:28:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-18 23:31:43]

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