Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(131)

Issue 3002973002: Refactor diagnostic spans to share a polymer behavior. (Closed)

Created:
3 years, 4 months ago by benjhayden
Modified:
3 years, 4 months ago
Reviewers:
eakuefner
CC:
catapult-reviews_chromium.org, tracing-review_chromium.org
Target Ref:
refs/heads/master
Project:
catapult
Visibility:
Public.

Description

Refactor diagnostic spans to share a polymer behavior. This facilitates changing how diagnostic spans are built and plumb more information into them such as DiagnosticMap key (name). No functional changes. Review-Url: https://codereview.chromium.org/3002973002 Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/8fc1f77cc8310160be3f66db27c0700f545fec39

Patch Set 1 : #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+429 lines, -441 lines) Patch
M tracing/trace_viewer.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M tracing/tracing/value/ui/breakdown_span.html View 2 chunks +252 lines, -265 lines 0 comments Download
M tracing/tracing/value/ui/breakdown_span_test.html View 1 chunk +2 lines, -2 lines 0 comments Download
M tracing/tracing/value/ui/collected_related_event_set_span.html View 2 chunks +11 lines, -18 lines 0 comments Download
M tracing/tracing/value/ui/date_range_span.html View 1 chunk +12 lines, -20 lines 0 comments Download
M tracing/tracing/value/ui/diagnostic_map_table.html View 2 chunks +2 lines, -2 lines 0 comments Download
M tracing/tracing/value/ui/diagnostic_span.html View 1 chunk +5 lines, -4 lines 0 comments Download
A tracing/tracing/value/ui/diagnostic_span_behavior.html View 1 chunk +44 lines, -0 lines 1 comment Download
M tracing/tracing/value/ui/generic_set_span.html View 3 chunks +11 lines, -17 lines 0 comments Download
M tracing/tracing/value/ui/related_event_set_span.html View 2 chunks +11 lines, -17 lines 0 comments Download
M tracing/tracing/value/ui/related_histogram_map_span.html View 2 chunks +40 lines, -43 lines 0 comments Download
M tracing/tracing/value/ui/related_histogram_set_span.html View 2 chunks +12 lines, -17 lines 0 comments Download
M tracing/tracing/value/ui/scalar_diagnostic_span.html View 1 chunk +10 lines, -17 lines 0 comments Download
M tracing/tracing/value/ui/unmergeable_diagnostic_set_span.html View 1 chunk +16 lines, -19 lines 0 comments Download

Messages

Total messages: 11 (7 generated)
benjhayden
PTAL
3 years, 4 months ago (2017-08-18 17:55:58 UTC) #5
eakuefner
lgtm https://codereview.chromium.org/3002973002/diff/20001/tracing/tracing/value/ui/diagnostic_span_behavior.html File tracing/tracing/value/ui/diagnostic_span_behavior.html (right): https://codereview.chromium.org/3002973002/diff/20001/tracing/tracing/value/ui/diagnostic_span_behavior.html#newcode3 tracing/tracing/value/ui/diagnostic_span_behavior.html:3: Copyright 2016 The Chromium Authors. All rights reserved. ...
3 years, 4 months ago (2017-08-18 23:24:30 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/3002973002/20001
3 years, 4 months ago (2017-08-19 04:56:33 UTC) #8
commit-bot: I haz the power
3 years, 4 months ago (2017-08-19 05:25:32 UTC) #11
Message was sent while issue was closed.
Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

Powered by Google App Engine
This is Rietveld 408576698