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

Side by Side Diff: runtime/observatory/lib/src/service/object.dart

Issue 1732303003: Fix analysis bots (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 part of service; 5 part of service;
6 6
7 // Some value smaller than the object ring, so requesting a large array 7 // Some value smaller than the object ring, so requesting a large array
8 // doesn't result in an expired ref because the elements lapped it in the 8 // doesn't result in an expired ref because the elements lapped it in the
9 // object ring. 9 // object ring.
10 const int kDefaultFieldLimit = 100; 10 const int kDefaultFieldLimit = 100;
(...skipping 373 matching lines...) Expand 10 before | Expand all | Expand 10 after
384 384
385 void _update(ObservableMap map, bool mapIsRef) { 385 void _update(ObservableMap map, bool mapIsRef) {
386 if (map['class'] != null) { 386 if (map['class'] != null) {
387 // Sent with refs for some types. Load it if available, but don't clobber 387 // Sent with refs for some types. Load it if available, but don't clobber
388 // it with null for kinds that only send if for full responses. 388 // it with null for kinds that only send if for full responses.
389 clazz = map['class']; 389 clazz = map['class'];
390 } 390 }
391 391
392 // Load the full class object if the isolate is runnable. 392 // Load the full class object if the isolate is runnable.
393 if (clazz != null) { 393 if (clazz != null) {
394 if (clazz.owner.runnable) { 394 if (clazz.isolate.runnable) {
395 clazz.load(); 395 clazz.load();
396 } 396 }
397 } 397 }
398 398
399 if (mapIsRef) { 399 if (mapIsRef) {
400 return; 400 return;
401 } 401 }
402 size = map['size']; 402 size = map['size'];
403 } 403 }
404 } 404 }
(...skipping 3602 matching lines...) Expand 10 before | Expand all | Expand 10 after
4007 var v = list[i]; 4007 var v = list[i];
4008 if ((v is ObservableMap) && _isServiceMap(v)) { 4008 if ((v is ObservableMap) && _isServiceMap(v)) {
4009 list[i] = owner.getFromMap(v); 4009 list[i] = owner.getFromMap(v);
4010 } else if (v is ObservableList) { 4010 } else if (v is ObservableList) {
4011 _upgradeObservableList(v, owner); 4011 _upgradeObservableList(v, owner);
4012 } else if (v is ObservableMap) { 4012 } else if (v is ObservableMap) {
4013 _upgradeObservableMap(v, owner); 4013 _upgradeObservableMap(v, owner);
4014 } 4014 }
4015 } 4015 }
4016 } 4016 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698