Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(69)

Issue 250590044: [perf_insights] Make sure all tags are alphanumeric (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 9 months ago by fmeawad
Modified:
8 years, 9 months ago
Reviewers:
dsinclair
CC:
tracing-review_chromium.org
Base URL:
https://github.com/google/trace-viewer.git@master
Visibility:
Public.

Description

[perf_insights] Make sure all tags are alphanumeric R=dsinclair@chromium.org Committed: https://chromium.googlesource.com/external/trace-viewer/+/2cf812ba2015bfcc392a2f401c253850b31060c7

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M perf_insights/perf_insights/upload.py View 2 chunks +2 lines, -1 line 0 comments Download

Messages

Total messages: 3
fmeawad
PTAL
8 years, 9 months ago (2015-07-22 17:50:24 UTC) #1
dsinclair
lgtm
8 years, 9 months ago (2015-07-22 17:58:20 UTC) #2
fmeawad
8 years, 9 months ago (2015-07-22 17:59:50 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2cf812ba2015bfcc392a2f401c253850b31060c7 (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b