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

Unified Diff: runtime/bin/vmservice/client/lib/src/elements/function_view.dart

Issue 376293003: Observatory: Promote function to ServiceObject (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: add deployed/ Created 6 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: runtime/bin/vmservice/client/lib/src/elements/function_view.dart
diff --git a/runtime/bin/vmservice/client/lib/src/elements/function_view.dart b/runtime/bin/vmservice/client/lib/src/elements/function_view.dart
index 4463a34f90ddc45262482f801b564e23348203df..c953393f75e8fd503b7364d910c7ce55dc4d656f 100644
--- a/runtime/bin/vmservice/client/lib/src/elements/function_view.dart
+++ b/runtime/bin/vmservice/client/lib/src/elements/function_view.dart
@@ -11,77 +11,9 @@ import 'package:polymer/polymer.dart';
@CustomTag('function-view')
class FunctionViewElement extends ObservatoryElement {
- @published ServiceMap function;
+ @published ServiceFunction function;
FunctionViewElement.created() : super.created();
- // TODO(turnidge): Once we create a Function object, these fields
- // should move there.
- @published String qualifiedName;
- @published String kind;
-
- String _getQualifiedName(ServiceMap function) {
- var parent = (function != null && function['parent'] != null
- ? function['parent'] : null);
- if (parent != null) {
- return "${_getQualifiedName(parent)}.${function['user_name']}";
- }
- Class cls = (function != null &&
- function['owner'] != null &&
- function['owner'].serviceType == 'Class'
- ? function['owner'] : null);
- if (cls != null) {
- return "${cls.name}.${function['user_name']}";
- }
- return "${function['user_name']}";
- }
-
- void functionChanged(oldValue) {
- notifyPropertyChange(#qualifiedName, 0, 1);
- notifyPropertyChange(#kind, 0, 1);
- qualifiedName = _getQualifiedName(function);
- switch(function['kind']) {
- case 'kRegularFunction':
- kind = 'function';
- break;
- case 'kClosureFunction':
- kind = 'closure function';
- break;
- case 'kSignatureFunction':
- kind = 'signature function';
- break;
- case 'kGetterFunction':
- kind = 'getter function';
- break;
- case 'kSetterFunction':
- kind = 'setter function';
- break;
- case 'kConstructor':
- kind = 'constructor';
- break;
- case 'kImplicitGetterFunction':
- kind = 'implicit getter function';
- break;
- case 'kImplicitSetterFunction':
- kind = 'implicit setter function';
- break;
- case 'kStaticInitializer':
- kind = 'static initializer';
- break;
- case 'kMethodExtractor':
- kind = 'method extractor';
- break;
- case 'kNoSuchMethodDispatcher':
- kind = 'noSuchMethod dispatcher';
- break;
- case 'kInvokeFieldDispatcher':
- kind = 'invoke field dispatcher';
- break;
- default:
- kind = 'UNKNOWN';
- break;
- }
- }
-
void refresh(var done) {
function.reload().whenComplete(done);
}

Powered by Google App Engine
This is Rietveld 408576698