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

Side by Side Diff: runtime/bin/vmservice/client/lib/src/elements/script_ref.dart

Issue 184233007: Refactor Observatory (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library script_ref_element; 5 library script_ref_element;
6 6
7 import 'package:polymer/polymer.dart'; 7 import 'package:polymer/polymer.dart';
8 import 'service_ref.dart'; 8 import 'service_ref.dart';
9 9
10 @CustomTag('script-ref') 10 @CustomTag('script-ref')
11 class ScriptRefElement extends ServiceRefElement { 11 class ScriptRefElement extends ServiceRefElement {
12 @published int line = -1; 12 @published int line = -1;
13 13
14 String get url { 14 String get objectId {
15 if ((app != null) && (ref != null)) { 15 if (line < 0) {
16 var baseUrl = relativeLink(ref['id']); 16 return super.objectId;
17 if (line < 0) {
18 return baseUrl;
19 } else {
20 return '$baseUrl?line=$line';
21 }
22 } 17 }
23 return ''; 18 // TODO(johnmccutchan): Add a ?line=XX invalidates the idea that this
19 // method returns an objectId.
20 return '${super.objectId}?line=$line';
24 } 21 }
25 22
26 String get hoverText { 23 String get hoverText {
27 if (ref == null) { 24 if (ref == null) {
28 return ''; 25 return '';
29 } 26 }
30 if (line < 0) { 27 if (line < 0) {
31 return ref['user_name']; 28 return ref['user_name'];
32 } else { 29 } else {
33 return "${ref['user_name']}:$line"; 30 return "${ref['user_name']}:$line";
34 } 31 }
35 } 32 }
36 33
37 String get name { 34 String get name {
38 if (ref == null) { 35 if (ref == null) {
39 return ''; 36 return '';
40 } 37 }
41 var scriptUrl = ref['user_name']; 38 var scriptUrl = ref['user_name'];
42 var shortScriptUrl = scriptUrl.substring(scriptUrl.lastIndexOf('/') + 1); 39 var shortScriptUrl = scriptUrl.substring(scriptUrl.lastIndexOf('/') + 1);
43 if (line < 0) { 40 if (line < 0) {
44 return shortScriptUrl; 41 return shortScriptUrl;
45 } else { 42 } else {
46 return "$shortScriptUrl:$line"; 43 return "$shortScriptUrl:$line";
47 } 44 }
48 } 45 }
49 46
50 ScriptRefElement.created() : super.created(); 47 ScriptRefElement.created() : super.created();
51 } 48 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698