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

Issue 2204973003: Converted Observatory class-ref & library-ref elements (Closed)

Created:
4 years, 4 months ago by cbernaschina
Modified:
4 years, 4 months ago
Reviewers:
turnidge, rmacnak, Cutch
CC:
reviews_dartlang.org, turnidge, rmacnak, Cutch, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 6

Patch Set 2 : Fixed indentation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+251 lines, -218 lines) Patch
M runtime/observatory/lib/elements.dart View 8 chunks +12 lines, -2 lines 0 comments Download
M runtime/observatory/lib/elements.html View 2 chunks +2 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/class_ref.dart View 1 chunk +43 lines, -19 lines 0 comments Download
D runtime/observatory/lib/src/elements/class_ref.html View 1 chunk +0 lines, -33 lines 0 comments Download
A + runtime/observatory/lib/src/elements/class_ref_as_value.dart View 1 chunk +4 lines, -5 lines 0 comments Download
A + runtime/observatory/lib/src/elements/class_ref_as_value.html View 3 chunks +5 lines, -8 lines 0 comments Download
A + runtime/observatory/lib/src/elements/class_ref_wrapper.dart View 2 chunks +12 lines, -12 lines 0 comments Download
M runtime/observatory/lib/src/elements/class_tree.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/class_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/code_ref.dart View 1 chunk +1 line, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/context_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/css/shared.css View 2 chunks +24 lines, -0 lines 0 comments Download
M runtime/observatory/lib/src/elements/field_view.html View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/function_ref.dart View 2 chunks +2 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/function_ref_wrapper.dart View 1 chunk +2 lines, -0 lines 0 comments Download
M runtime/observatory/lib/src/elements/function_view.html View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/heap_map.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/heap_profile.dart View 2 chunks +2 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/heap_profile.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/heap_snapshot.dart View 3 chunks +3 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/heap_snapshot.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/icdata_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/instance_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/instructions_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/isolate_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/library_ref.dart View 1 chunk +44 lines, -19 lines 0 comments Download
D runtime/observatory/lib/src/elements/library_ref.html View 1 chunk +0 lines, -38 lines 0 comments Download
A + runtime/observatory/lib/src/elements/library_ref_as_value.dart View 1 chunk +4 lines, -6 lines 0 comments Download
A runtime/observatory/lib/src/elements/library_ref_as_value.html View 1 chunk +30 lines, -0 lines 0 comments Download
A + runtime/observatory/lib/src/elements/library_ref_wrapper.dart View 2 chunks +12 lines, -12 lines 0 comments Download
M runtime/observatory/lib/src/elements/library_view.html View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/megamorphiccache_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/object_common.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/object_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/objectpool_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/lib/src/elements/objectstore_view.html View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/observatory/lib/src/elements/service_ref.dart View 1 2 chunks +33 lines, -20 lines 0 comments Download
M runtime/observatory/lib/src/elements/vm_view.html View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/observatory/observatory_sources.gypi View 2 chunks +6 lines, -2 lines 0 comments Download
A + runtime/observatory/tests/observatory_ui/class_ref/element_test.dart View 1 chunk +6 lines, -6 lines 0 comments Download
A + runtime/observatory/tests/observatory_ui/class_ref/element_test.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + runtime/observatory/tests/observatory_ui/library_ref/element_test.dart View 1 chunk +6 lines, -6 lines 0 comments Download
A + runtime/observatory/tests/observatory_ui/library_ref/element_test.html View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
cbernaschina
4 years, 4 months ago (2016-08-02 21:13:49 UTC) #2
Cutch
https://codereview.chromium.org/2204973003/diff/1/runtime/observatory/lib/src/elements/library_ref_as_value.dart File runtime/observatory/lib/src/elements/library_ref_as_value.dart (right): https://codereview.chromium.org/2204973003/diff/1/runtime/observatory/lib/src/elements/library_ref_as_value.dart#newcode13 runtime/observatory/lib/src/elements/library_ref_as_value.dart:13: class LibraryRefAsValueElement extends ServiceRefElement { any reason why this ...
4 years, 4 months ago (2016-08-03 00:32:03 UTC) #3
cbernaschina
https://codereview.chromium.org/2204973003/diff/1/runtime/observatory/lib/src/elements/library_ref_as_value.dart File runtime/observatory/lib/src/elements/library_ref_as_value.dart (right): https://codereview.chromium.org/2204973003/diff/1/runtime/observatory/lib/src/elements/library_ref_as_value.dart#newcode13 runtime/observatory/lib/src/elements/library_ref_as_value.dart:13: class LibraryRefAsValueElement extends ServiceRefElement { On 2016/08/03 00:32:03, Cutch ...
4 years, 4 months ago (2016-08-03 22:12:12 UTC) #4
Cutch
lgtm
4 years, 4 months ago (2016-08-04 00:00:19 UTC) #5
cbernaschina
4 years, 4 months ago (2016-08-04 00:48:51 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
937d281f7fcef11931fc9bdeb75e16d760b8465c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698