Log JumpListUpdater functions' execution time to UMA

Log JumpListUpdater functions' execution time to UMA

Recent UMA data shows that the UpdateJumpList method alone in jumplist.cc 
takes 1+ second for more than 1% Canary users. Therefore, it's worthwhile
to investigate all the pieces of this method, which is JumpListUpdater
class related.

BUG=40407, 179576
Review-Url: https://codereview.chromium.org/2818193002
Cr-Commit-Position: refs/heads/master@{#464997}
Committed: https://chromium.googlesource.com/chromium/src/+/ce5d2adea8adbc9e950371ef08513de8e83d790d

[chengx, 2017-04-14 20:13:08]

Description was changed from

==========
Log function run time for JumpListUpdater.

BUG=40407, 179576
==========

to

==========
Log JumpListUpdater functions' execution time to UMA

Recent UMA data shows that UpdateJumpList method along in jumplist.cc 
takes 1+ second for more than 1% Canary users. Therefore, it's worthwhile
to investigate all the pieces of this method, which is JumpListUpdater
class related.

BUG=40407, 179576
==========

[chengx, 2017-04-14 20:18:02]

chengx@chromium.org changed reviewers:
+ gab@chromium.org, isherman@chromium.org

[chengx, 2017-04-14 20:18:03]

PTAL, thanks!

gab@ for jumplist_updater.cc
isherman@ for histograms.xml

[chengx, 2017-04-14 20:18:39]

Description was changed from

==========
Log JumpListUpdater functions' execution time to UMA

Recent UMA data shows that UpdateJumpList method along in jumplist.cc 
takes 1+ second for more than 1% Canary users. Therefore, it's worthwhile
to investigate all the pieces of this method, which is JumpListUpdater
class related.

BUG=40407, 179576
==========

to

==========
Log JumpListUpdater functions' execution time to UMA

Recent UMA data shows that the UpdateJumpList method alone in jumplist.cc 
takes 1+ second for more than 1% Canary users. Therefore, it's worthwhile
to investigate all the pieces of this method, which is JumpListUpdater
class related.

BUG=40407, 179576
==========

[Ilya Sherman, 2017-04-14 20:22:10]

lgtm

[chengx, 2017-04-17 18:33:29]

chengx@chromium.org changed reviewers:
+ pmonette@chromium.org

[chengx, 2017-04-17 18:33:29]

Gentle ping~

Adding another file owner pmonette@ in case gab@ is busy.

[Patrick Monette, 2017-04-17 19:09:30]

lgtm

[chengx, 2017-04-17 19:11:14]

The CQ bit was checked by chengx@chromium.org

[commit-bot: I haz the power, 2017-04-17 19:11:40]

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, 2017-04-17 20:19:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492456274811110, "parent_rev":
"13b8f41be9f248fd67aa1188fc453a06e3825b6d", "commit_rev":
"ce5d2adea8adbc9e950371ef08513de8e83d790d"}

[commit-bot: I haz the power, 2017-04-17 20:20:35]

Description was changed from

==========
Log JumpListUpdater functions' execution time to UMA

Recent UMA data shows that the UpdateJumpList method alone in jumplist.cc 
takes 1+ second for more than 1% Canary users. Therefore, it's worthwhile
to investigate all the pieces of this method, which is JumpListUpdater
class related.

BUG=40407, 179576
==========

to

==========
Log JumpListUpdater functions' execution time to UMA

Recent UMA data shows that the UpdateJumpList method alone in jumplist.cc 
takes 1+ second for more than 1% Canary users. Therefore, it's worthwhile
to investigate all the pieces of this method, which is JumpListUpdater
class related.

BUG=40407, 179576

Review-Url: https://codereview.chromium.org/2818193002
Cr-Commit-Position: refs/heads/master@{#464997}
Committed:
https://chromium.googlesource.com/chromium/src/+/ce5d2adea8adbc9e950371ef0851...
==========

[commit-bot: I haz the power, 2017-04-17 20:20:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ce5d2adea8adbc9e950371ef0851...

[gab, 2017-04-18 19:14:43]

lgtm, sorry for delay