Reland Plumb trace canonicalUrl through TelemetryInfo.

Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

http://www/~benjhayden/763375.html

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:763375

Review-Url: https://chromiumcodereview.appspot.com/3005313002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/5f3e94e29a7b023d9cf4031e9f9fde9bc6e0f4e6

[benjhayden, 2017-09-08 21:32:27]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

Next CLs:
- format traceUrls generic-set-spans https://codereview.chromium.org/3008203002
- add traceUrls to CSVBuilder https://codereview.chromium.org/3005203002
- plumb traceUrls through ChartJsonConverter

BUG=catapult:#2431
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

Next CLs:
- format traceUrls generic-set-spans https://codereview.chromium.org/3008203002
- add traceUrls to CSVBuilder https://codereview.chromium.org/3005203002
- plumb traceUrls through ChartJsonConverter

BUG=catapult:#2431,chromium:#763375
==========

[benjhayden, 2017-09-08 21:33:32]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

Next CLs:
- format traceUrls generic-set-spans https://codereview.chromium.org/3008203002
- add traceUrls to CSVBuilder https://codereview.chromium.org/3005203002
- plumb traceUrls through ChartJsonConverter

BUG=catapult:#2431,chromium:#763375
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

This depends on a prospective change in chromium:
https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

[benjhayden, 2017-09-08 21:40:03]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

This depends on a prospective change in chromium:
https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on a prospective change in chromium:
https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

[benjhayden, 2017-09-11 16:59:27]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on a prospective change in chromium:
https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

[benjhayden, 2017-09-11 18:43:35]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

http://www/~benjhayden/763375.html

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

[benjhayden, 2017-09-11 18:43:56]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

http://www/~benjhayden/763375.html

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:#763375
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

http://www/~benjhayden/763375.html

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:763375
==========

[benjhayden, 2017-09-11 18:49:40]

benjhayden@chromium.org changed reviewers:
+ eakuefner@chromium.org, nednguyen@google.com, simonhatch@chromium.org

[benjhayden, 2017-09-11 18:50:32]

Patchset #1 (id:1) has been deleted

[eakuefner, 2017-09-11 20:58:57]

lgtm

[shatch, 2017-09-13 01:27:11]

lgtm

[benjhayden, 2017-09-13 04:57:18]

The CQ bit was checked by benjhayden@chromium.org

[commit-bot: I haz the power, 2017-09-13 04:57:25]

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-09-13 05:13:34]

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

[commit-bot: I haz the power, 2017-09-13 05:13:35]

Try jobs failed on following builders:
  Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20An...)

[benjhayden, 2017-09-13 06:00:50]

The CQ bit was checked by benjhayden@chromium.org

[benjhayden, 2017-09-13 06:00:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
eakuefner@chromium.org, simonhatch@chromium.org
Link to the patchset: https://codereview.chromium.org/3005313002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2017-09-13 06:00:55]

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-09-13 06:16:26]

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

[commit-bot: I haz the power, 2017-09-13 06:16:27]

Try jobs failed on following builders:
  Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20An...)

[benjhayden, 2017-09-13 20:47:00]

The CQ bit was checked by benjhayden@chromium.org

[benjhayden, 2017-09-13 20:47:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
eakuefner@chromium.org, simonhatch@chromium.org
Link to the patchset: https://codereview.chromium.org/3005313002/#ps100001
(title: " ")

[commit-bot: I haz the power, 2017-09-13 20:47:10]

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-09-13 21:19:06]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1505335620277490,
"parent_rev": "88d1196ef1548e6676e4ad4c5d2d5871e5a9e49f", "commit_rev":
"5f3e94e29a7b023d9cf4031e9f9fde9bc6e0f4e6"}

[commit-bot: I haz the power, 2017-09-13 21:19:11]

Description was changed from

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

http://www/~benjhayden/763375.html

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:763375
==========

to

==========
Reland Plumb trace canonicalUrl through TelemetryInfo.

Currently, trace canonical URLs are constructed after serializing the trace.
This prevents Histogram results from containing the canonical URL.
This CL constructs trace canonical URLs before serializing the trace so that
Histograms can contain the canonical URL.

http://www/~benjhayden/763375.html

This CL was originally landed in https://codereview.chromium.org/3007063002
and reverted in https://chromiumcodereview.appspot.com/3007313002/

This depends on https://chromium-review.googlesource.com/c/chromium/src/+/658263

Next CLs:
- plumb traceUrls through ChartJsonConverter
https://codereview.chromium.org/3008223002

BUG=catapult:#2431,chromium:763375

Review-Url: https://chromiumcodereview.appspot.com/3005313002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-09-13 21:19:12]

Committed patchset #5 (id:100001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...