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

Issue 2514283003: [tools] Add deep links to callstats.html (Closed)

Created:
4 years, 1 month ago by Camillo Bruni
Modified:
4 years, 1 month ago
Reviewers:
Michael Hablich
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[tools] Add deep links to callstats.html R=hablich@chromium.org NOTRY=true NOTREECHECKS=true Committed: https://crrev.com/e735c5d378d89d4204a4fbce0174a0b1506676ad Cr-Commit-Position: refs/heads/master@{#41167}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -2 lines) Patch
M tools/callstats.html View 8 chunks +76 lines, -2 lines 0 comments Download

Messages

Total messages: 13 (7 generated)
Camillo Bruni
PTAL
4 years, 1 month ago (2016-11-22 08:57:58 UTC) #1
Michael Hablich
On 2016/11/22 08:57:58, Camillo Bruni wrote: > PTAL lgtm
4 years, 1 month ago (2016-11-22 09:03:30 UTC) #2
Michael Hablich
I added notry and notreechecks. No need to test this with our bots.
4 years, 1 month ago (2016-11-22 09:08:42 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/2514283003/1
4 years, 1 month ago (2016-11-22 10:10:59 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-11-22 10:12:57 UTC) #11
commit-bot: I haz the power
4 years, 1 month ago (2016-11-22 10:13:17 UTC) #13
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/e735c5d378d89d4204a4fbce0174a0b1506676ad
Cr-Commit-Position: refs/heads/master@{#41167}

Powered by Google App Engine
This is Rietveld 408576698