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

Side by Side Diff: test/cctest/test-debug.cc

Issue 1415463017: [debugger] Re-enable --always-opt in one debugger test. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 1 month 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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 6688 matching lines...) Expand 10 before | Expand all | Expand 10 after
6699 6699
6700 6700
6701 // Test that debug messages get processed when ProcessDebugMessages is called. 6701 // Test that debug messages get processed when ProcessDebugMessages is called.
6702 TEST(Backtrace) { 6702 TEST(Backtrace) {
6703 DebugLocalContext env; 6703 DebugLocalContext env;
6704 v8::Isolate* isolate = env->GetIsolate(); 6704 v8::Isolate* isolate = env->GetIsolate();
6705 v8::HandleScope scope(isolate); 6705 v8::HandleScope scope(isolate);
6706 6706
6707 v8::Debug::SetMessageHandler(BacktraceData::MessageHandler); 6707 v8::Debug::SetMessageHandler(BacktraceData::MessageHandler);
6708 6708
6709 // TODO(mstarzinger): This doesn't work with --always-opt because we don't
6710 // have correct source positions in optimized code. Enable once we have.
6711 i::FLAG_always_opt = false;
6712
6713 const int kBufferSize = 1000; 6709 const int kBufferSize = 1000;
6714 uint16_t buffer[kBufferSize]; 6710 uint16_t buffer[kBufferSize];
6715 const char* scripts_command = 6711 const char* scripts_command =
6716 "{\"seq\":0," 6712 "{\"seq\":0,"
6717 "\"type\":\"request\"," 6713 "\"type\":\"request\","
6718 "\"command\":\"backtrace\"}"; 6714 "\"command\":\"backtrace\"}";
6719 6715
6720 // Check backtrace from ProcessDebugMessages. 6716 // Check backtrace from ProcessDebugMessages.
6721 BacktraceData::frame_counter = -10; 6717 BacktraceData::frame_counter = -10;
6722 v8::Debug::SendCommand( 6718 v8::Debug::SendCommand(
(...skipping 927 matching lines...) Expand 10 before | Expand all | Expand 10 after
7650 CompileRun("function foo() {}; foo();"); 7646 CompileRun("function foo() {}; foo();");
7651 --after_compile_handler_depth; 7647 --after_compile_handler_depth;
7652 } 7648 }
7653 7649
7654 7650
7655 TEST(NoInterruptsInDebugListener) { 7651 TEST(NoInterruptsInDebugListener) {
7656 DebugLocalContext env; 7652 DebugLocalContext env;
7657 v8::Debug::SetDebugEventListener(NoInterruptsOnDebugEvent); 7653 v8::Debug::SetDebugEventListener(NoInterruptsOnDebugEvent);
7658 CompileRun("void(0);"); 7654 CompileRun("void(0);");
7659 } 7655 }
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