Add data use measurement to DoodleFetcher

Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=690467
TBR=asvitkine

Review-Url: https://codereview.chromium.org/2698643002
Cr-Commit-Position: refs/heads/master@{#451937}
Committed: https://chromium.googlesource.com/chromium/src/+/c5b606c3188dac60dd3f8e29bafbb90d95089c3e

[fhorschig, 2017-02-21 10:11:59]

Description was changed from

==========
Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=
==========

to

==========
Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=690467
==========

[fhorschig, 2017-02-21 11:59:39]

fhorschig@chromium.org changed reviewers:
+ treib@chromium.org

[fhorschig, 2017-02-21 11:59:39]

Hi Marc,
would you please have a look at the follow-up CL for measuring data of the
DoodleFetcher?

[Marc Treib, 2017-02-21 12:17:16]

lgtm

FYI: Per recent announcement, it's okay to TBR=asvitkine for the histograms.xml
change. (You'll still need approval from data_use_measurement owners though.)

[fhorschig, 2017-02-21 13:12:30]

Description was changed from

==========
Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=690467
==========

to

==========
Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=690467
TBR=asvitkine
==========

[fhorschig, 2017-02-21 13:14:12]

On 2017/02/21 12:17:16, Marc Treib wrote:
> lgtm
> 
> FYI: Per recent announcement, it's okay to TBR=asvitkine for the
histograms.xml
> change. (You'll still need approval from data_use_measurement owners though.)

Thanks, that apparently slipped my attention. Added TBR line.

[fhorschig, 2017-02-21 13:14:26]

fhorschig@chromium.org changed reviewers:
+ ryansturm@chromium.org

[fhorschig, 2017-02-21 13:14:27]

Hi Ryan,
Could you please take a look at the enum entry I introduced to measure Doodle
fetching data?
(in components/data_use_measurement/core/data_use_user_data.*)

[RyanSturm, 2017-02-21 17:47:46]

lgtm

[fhorschig, 2017-02-21 17:50:26]

The CQ bit was checked by fhorschig@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-21 17:51:03]

Dry run: 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-02-21 20:00:27]

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

[commit-bot: I haz the power, 2017-02-21 20:00:27]

Dry run: This issue passed the CQ dry run.

[fhorschig, 2017-02-22 08:37:13]

The CQ bit was checked by fhorschig@chromium.org

[commit-bot: I haz the power, 2017-02-22 08:38:20]

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-02-22 08:46:13]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487752633291660,
"parent_rev": "7a0a5316f12d0873b3c17a29157b703cf4f1ae36", "commit_rev":
"c5b606c3188dac60dd3f8e29bafbb90d95089c3e"}

[commit-bot: I haz the power, 2017-02-22 08:46:50]

Description was changed from

==========
Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=690467
TBR=asvitkine
==========

to

==========
Add data use measurement to DoodleFetcher

In order to track the doodle fetches, introduce a new measurement.

BUG=690467
TBR=asvitkine

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

[commit-bot: I haz the power, 2017-02-22 08:46:52]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/c5b606c3188dac60dd3f8e29bafb...