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

Side by Side Diff: test/debugger/test-api.js

Issue 2510093002: [inspector] gracefully handle stack overflows in the inspector. (Closed)
Patch Set: fix asan Created 4 years 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 | « test/debugger/regress/regress-2318.js ('k') | test/mjsunit/mjsunit.status » ('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 2016 the V8 project authors. All rights reserved. 1 // Copyright 2016 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 "use strict"; 5 "use strict";
6 6
7 // If true, prints all messages sent and received by inspector. 7 // If true, prints all messages sent and received by inspector.
8 const printProtocolMessages = false; 8 const printProtocolMessages = false;
9 9
10 // The active wrapper instance. 10 // The active wrapper instance.
(...skipping 801 matching lines...) Expand 10 before | Expand all | Expand 10 after
812 case "exception": 812 case "exception":
813 case "promiseRejection": 813 case "promiseRejection":
814 debugEvent = this.DebugEvent.Exception; 814 debugEvent = this.DebugEvent.Exception;
815 break; 815 break;
816 default: 816 default:
817 // TODO(jgruber): More granularity. 817 // TODO(jgruber): More granularity.
818 debugEvent = this.DebugEvent.Break; 818 debugEvent = this.DebugEvent.Break;
819 break; 819 break;
820 } 820 }
821 821
822 if (!params.callFrames[0]) return;
823
822 // Skip break events in this file. 824 // Skip break events in this file.
823 if (params.callFrames[0].location.scriptId == this.thisScriptId) return; 825 if (params.callFrames[0].location.scriptId == this.thisScriptId) return;
824 826
825 // TODO(jgruber): Arguments as needed. 827 // TODO(jgruber): Arguments as needed.
826 let execState = { frames : params.callFrames, 828 let execState = { frames : params.callFrames,
827 prepareStep : this.execStatePrepareStep.bind(this), 829 prepareStep : this.execStatePrepareStep.bind(this),
828 evaluateGlobal : 830 evaluateGlobal :
829 (expr) => this.execStateEvaluateGlobal(expr), 831 (expr) => this.execStateEvaluateGlobal(expr),
830 frame : (index) => this.execStateFrame( 832 frame : (index) => this.execStateFrame(
831 index ? params.callFrames[index] 833 index ? params.callFrames[index]
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
883 debug.instance = new DebugWrapper(); 885 debug.instance = new DebugWrapper();
884 debug.instance.enable(); 886 debug.instance.enable();
885 } 887 }
886 return debug.instance; 888 return debug.instance;
887 }}); 889 }});
888 890
889 Object.defineProperty(debug, 'ScopeType', { get: function() { 891 Object.defineProperty(debug, 'ScopeType', { get: function() {
890 const instance = debug.Debug; 892 const instance = debug.Debug;
891 return instance.ScopeType; 893 return instance.ScopeType;
892 }}); 894 }});
OLDNEW
« no previous file with comments | « test/debugger/regress/regress-2318.js ('k') | test/mjsunit/mjsunit.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698