Log metrics for loading SQLiteChannelIDStore

Log metrics for loading SQLiteChannelIDStore

BUG=548423
Review-Url: https://codereview.chromium.org/2807853002
Cr-Commit-Position: refs/heads/master@{#463437}
Committed: https://chromium.googlesource.com/chromium/src/+/9d64fcad3c27082ef8c94a54e41ebe30337e37b1

[nharper, 2017-04-07 23:46:49]

nharper@chromium.org changed reviewers:
+ mattm@chromium.org, vanupam@chromium.org

[mattm, 2017-04-08 04:17:38]

https://codereview.chromium.org/2807853002/diff/1/net/extras/sqlite/sqlite_ch...
File net/extras/sqlite/sqlite_channel_id_store.cc (right):

https://codereview.chromium.org/2807853002/diff/1/net/extras/sqlite/sqlite_ch...
net/extras/sqlite/sqlite_channel_id_store.cc:261: if (!key)
Should this be recorded somehow?

https://codereview.chromium.org/2807853002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2807853002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:12746: +    The status ofloading the
Channel ID database from disk. This histogram
of loading

[nharper, 2017-04-10 19:29:48]

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

[nharper, 2017-04-10 19:29:49]

https://codereview.chromium.org/2807853002/diff/1/net/extras/sqlite/sqlite_ch...
File net/extras/sqlite/sqlite_channel_id_store.cc (right):

https://codereview.chromium.org/2807853002/diff/1/net/extras/sqlite/sqlite_ch...
net/extras/sqlite/sqlite_channel_id_store.cc:261: if (!key)
On 2017/04/08 04:17:38, mattm wrote:
> Should this be recorded somehow?

Added a new enum value for this.

https://codereview.chromium.org/2807853002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2807853002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:12746: +    The status ofloading the
Channel ID database from disk. This histogram
On 2017/04/08 04:17:38, mattm wrote:
> of loading

Done.

[Steven Holte, 2017-04-10 21:15:30]

lgtm

[mattm, 2017-04-10 21:42:51]

lgtm

[nharper, 2017-04-10 22:02:00]

The CQ bit was checked by nharper@chromium.org

[commit-bot: I haz the power, 2017-04-10 22:03:18]

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-10 23:12:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491861720010280,
"parent_rev": "2a2f87ba99ef135283c6cf81a173189e78357965", "commit_rev":
"9d64fcad3c27082ef8c94a54e41ebe30337e37b1"}

[commit-bot: I haz the power, 2017-04-10 23:13:06]

Description was changed from

==========
Log metrics for loading SQLiteChannelIDStore

BUG=548423
==========

to

==========
Log metrics for loading SQLiteChannelIDStore

BUG=548423

Review-Url: https://codereview.chromium.org/2807853002
Cr-Commit-Position: refs/heads/master@{#463437}
Committed:
https://chromium.googlesource.com/chromium/src/+/9d64fcad3c27082ef8c94a54e41e...
==========

[commit-bot: I haz the power, 2017-04-10 23:13:07]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/9d64fcad3c27082ef8c94a54e41e...