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

Issue 2630423003: Improve display of long isolate names on summary page (Closed)

Created:
3 years, 11 months ago by kevmoo
Modified:
3 years, 11 months ago
Reviewers:
turnidge, rmacnak, Cutch
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
M runtime/observatory/lib/src/elements/css/shared.css View 1 chunk +11 lines, -0 lines 0 comments Download
M runtime/observatory/lib/src/elements/isolate/summary.dart View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
kevmoo
3 years, 11 months ago (2017-01-16 23:13:37 UTC) #2
Cutch
lgtm but wait for turnidge
3 years, 11 months ago (2017-01-17 00:56:13 UTC) #3
turnidge
lgtm
3 years, 11 months ago (2017-01-17 21:33:36 UTC) #4
kevmoo
3 years, 11 months ago (2017-01-17 21:34:46 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
7c6b933e35ca493084f8253387633a706fd6a0db (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698