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

Side by Side Diff: runtime/vm/debugger.cc

Issue 1414493003: Remove some Isolate::current_zone() calls, as it gets the zone from mutator thread not the current … (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Zones and commentw Created 5 years, 2 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 | « runtime/vm/debugger.h ('k') | runtime/vm/isolate.cc » ('j') | 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) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 #include "vm/debugger.h" 5 #include "vm/debugger.h"
6 6
7 #include "include/dart_api.h" 7 #include "include/dart_api.h"
8 8
9 #include "vm/code_generator.h" 9 #include "vm/code_generator.h"
10 #include "vm/code_patcher.h" 10 #include "vm/code_patcher.h"
(...skipping 2384 matching lines...) Expand 10 before | Expand all | Expand 10 after
2395 } 2395 }
2396 return Array::MakeArray(field_list); 2396 return Array::MakeArray(field_list);
2397 } 2397 }
2398 2398
2399 2399
2400 void Debugger::CollectLibraryFields(const GrowableObjectArray& field_list, 2400 void Debugger::CollectLibraryFields(const GrowableObjectArray& field_list,
2401 const Library& lib, 2401 const Library& lib,
2402 const String& prefix, 2402 const String& prefix,
2403 bool include_private_fields) { 2403 bool include_private_fields) {
2404 DictionaryIterator it(lib); 2404 DictionaryIterator it(lib);
2405 Object& entry = Object::Handle(isolate_->current_zone()); 2405 Object& entry = Object::Handle(zone());
2406 Field& field = Field::Handle(zone()); 2406 Field& field = Field::Handle(zone());
2407 String& field_name = String::Handle(zone()); 2407 String& field_name = String::Handle(zone());
2408 PassiveObject& field_value = PassiveObject::Handle(isolate_->current_zone()); 2408 PassiveObject& field_value = PassiveObject::Handle(zone());
2409 while (it.HasNext()) { 2409 while (it.HasNext()) {
2410 entry = it.GetNext(); 2410 entry = it.GetNext();
2411 if (entry.IsField()) { 2411 if (entry.IsField()) {
2412 field ^= entry.raw(); 2412 field ^= entry.raw();
2413 ASSERT(field.is_static()); 2413 ASSERT(field.is_static());
2414 field_name = field.name(); 2414 field_name = field.name();
2415 if ((field_name.CharAt(0) == '_') && !include_private_fields) { 2415 if ((field_name.CharAt(0) == '_') && !include_private_fields) {
2416 // Skip library-private field. 2416 // Skip library-private field.
2417 continue; 2417 continue;
2418 } 2418 }
(...skipping 820 matching lines...) Expand 10 before | Expand all | Expand 10 after
3239 } 3239 }
3240 3240
3241 3241
3242 void Debugger::RegisterCodeBreakpoint(CodeBreakpoint* bpt) { 3242 void Debugger::RegisterCodeBreakpoint(CodeBreakpoint* bpt) {
3243 ASSERT(bpt->next() == NULL); 3243 ASSERT(bpt->next() == NULL);
3244 bpt->set_next(code_breakpoints_); 3244 bpt->set_next(code_breakpoints_);
3245 code_breakpoints_ = bpt; 3245 code_breakpoints_ = bpt;
3246 } 3246 }
3247 3247
3248 } // namespace dart 3248 } // namespace dart
OLDNEW
« no previous file with comments | « runtime/vm/debugger.h ('k') | runtime/vm/isolate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698