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

Side by Side Diff: src/debug/debug.cc

Issue 1731883003: Encode interpreter::SourcePositionTable as variable-length ints. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Really fix leak. SourcePositionTableBuilder is member of a zone-allocated object. Created 4 years, 10 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 | src/heap/heap.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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/debug/debug.h" 5 #include "src/debug/debug.h"
6 6
7 #include "src/api.h" 7 #include "src/api.h"
8 #include "src/arguments.h" 8 #include "src/arguments.h"
9 #include "src/bootstrapper.h" 9 #include "src/bootstrapper.h"
10 #include "src/code-stubs.h" 10 #include "src/code-stubs.h"
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 } else { 164 } else {
165 type = NOT_DEBUG_BREAK; 165 type = NOT_DEBUG_BREAK;
166 } 166 }
167 return BreakLocation(debug_info_, type, code_offset(), position(), 167 return BreakLocation(debug_info_, type, code_offset(), position(),
168 statement_position()); 168 statement_position());
169 } 169 }
170 170
171 BreakLocation::BytecodeArrayIterator::BytecodeArrayIterator( 171 BreakLocation::BytecodeArrayIterator::BytecodeArrayIterator(
172 Handle<DebugInfo> debug_info, BreakLocatorType type) 172 Handle<DebugInfo> debug_info, BreakLocatorType type)
173 : Iterator(debug_info), 173 : Iterator(debug_info),
174 source_position_iterator_( 174 source_position_iterator_(debug_info->abstract_code()
175 debug_info->abstract_code()->GetBytecodeArray()), 175 ->GetBytecodeArray()
176 ->source_position_table()),
176 break_locator_type_(type), 177 break_locator_type_(type),
177 start_position_(debug_info->shared()->start_position()) { 178 start_position_(debug_info->shared()->start_position()) {
178 if (!Done()) Next(); 179 if (!Done()) Next();
179 } 180 }
180 181
181 void BreakLocation::BytecodeArrayIterator::Next() { 182 void BreakLocation::BytecodeArrayIterator::Next() {
182 DisallowHeapAllocation no_gc; 183 DisallowHeapAllocation no_gc;
183 DCHECK(!Done()); 184 DCHECK(!Done());
184 bool first = break_index_ == -1; 185 bool first = break_index_ == -1;
185 while (!Done()) { 186 while (!Done()) {
(...skipping 2430 matching lines...) Expand 10 before | Expand all | Expand 10 after
2616 } 2617 }
2617 2618
2618 2619
2619 void LockingCommandMessageQueue::Clear() { 2620 void LockingCommandMessageQueue::Clear() {
2620 base::LockGuard<base::Mutex> lock_guard(&mutex_); 2621 base::LockGuard<base::Mutex> lock_guard(&mutex_);
2621 queue_.Clear(); 2622 queue_.Clear();
2622 } 2623 }
2623 2624
2624 } // namespace internal 2625 } // namespace internal
2625 } // namespace v8 2626 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | src/heap/heap.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698