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

Side by Side Diff: test/mjsunit/debug-evaluate.js

Issue 8888006: Make more JS files beter match the coding standard. Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Address review comments Created 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2008 the V8 project authors. All rights reserved. 1 // Copyright 2008 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
11 // with the distribution. 11 // with the distribution.
12 // * Neither the name of Google Inc. nor the names of its 12 // * Neither the name of Google Inc. nor the names of its
13 // contributors may be used to endorse or promote products derived 13 // contributors may be used to endorse or promote products derived
14 // from this software without specific prior written permission. 14 // from this software without specific prior written permission.
15 // 15 //
16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 27
28 // Flags: --expose-debug-as debug 28 // Flags: --expose-debug-as debug
29 // Get the Debug object exposed from the debug context global object. 29 // Get the Debug object exposed from the debug context global object.
30 Debug = debug.Debug 30 Debug = debug.Debug;
31 31
32 listenerComplete = false; 32 listenerComplete = false;
33 exception = false; 33 exception = false;
34 34
35 // The base part of all evaluate requests. 35 // The base part of all evaluate requests.
36 var base_request = '"seq":0,"type":"request","command":"evaluate"' 36 var base_request = '"seq":0,"type":"request","command":"evaluate"';
37 37
38 function safeEval(code) { 38 function safeEval(code) {
39 try { 39 try {
40 return eval('(' + code + ')'); 40 return eval('(' + code + ')');
41 } catch (e) { 41 } catch (e) {
42 assertEquals(void 0, e); 42 assertEquals(void 0, e);
43 return undefined; 43 return undefined;
44 } 44 }
45 } 45 }
46 46
47 function testRequest(dcp, arguments, success, result) { 47 function testRequest(dcp, arguments, success, result) {
48 // Generate request with the supplied arguments. 48 // Generate request with the supplied arguments.
49 var request; 49 var request;
50 if (arguments) { 50 if (arguments) {
51 request = '{' + base_request + ',"arguments":' + arguments + '}'; 51 request = '{' + base_request + ',"arguments":' + arguments + '}';
52 } else { 52 } else {
53 request = '{' + base_request + '}' 53 request = '{' + base_request + '}';
54 } 54 }
55 var response = safeEval(dcp.processDebugJSONRequest(request)); 55 var response = safeEval(dcp.processDebugJSONRequest(request));
56 if (success) { 56 if (success) {
57 assertTrue(response.success, request + ' -> ' + response.message); 57 assertTrue(response.success, request + ' -> ' + response.message);
58 assertEquals(result, response.body.value); 58 assertEquals(result, response.body.value);
59 } else { 59 } else {
60 assertFalse(response.success, request + ' -> ' + response.message); 60 assertFalse(response.success, request + ' -> ' + response.message);
61 } 61 }
62 assertEquals(response.running, "unspecified_running_state", 62 assertEquals(response.running, "unspecified_running_state",
63 request + ' -> expected not running'); 63 request + ' -> expected not running');
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 testRequest( 115 testRequest(
116 dcp, 116 dcp,
117 '{"expression":"this.longString","global":true}', 117 '{"expression":"this.longString","global":true}',
118 true, 118 true,
119 longString.substring(0, 80) + truncatedStringSuffix); 119 longString.substring(0, 80) + truncatedStringSuffix);
120 120
121 // Indicate that all was processed. 121 // Indicate that all was processed.
122 listenerComplete = true; 122 listenerComplete = true;
123 } 123 }
124 } catch (e) { 124 } catch (e) {
125 exception = e 125 exception = e;
126 }; 126 }
127 }; 127 }
128 128
129 // Add the debug event listener. 129 // Add the debug event listener.
130 Debug.setListener(listener); 130 Debug.setListener(listener);
131 131
132 function f() { 132 function f() {
133 var a = 3; 133 var a = 3;
134 }; 134 }
135 135
136 function g() { 136 function g() {
137 var a = 2; 137 var a = 2;
138 f(); 138 f();
139 }; 139 }
140 140
141 a = 1; 141 a = 1;
142 142
143 // String which is longer than 80 chars. 143 // String which is longer than 80 chars.
144 var longString = "1234567890_"; 144 var longString = "1234567890_";
145 for (var i = 0; i < 4; i++) { 145 for (var i = 0; i < 4; i++) {
146 longString += longString; 146 longString += longString;
147 } 147 }
148 148
149 // Set a break point at return in f and invoke g to hit the breakpoint. 149 // Set a break point at return in f and invoke g to hit the breakpoint.
150 Debug.setBreakPoint(f, 2, 0); 150 Debug.setBreakPoint(f, 2, 0);
151 g(); 151 g();
152 152
153 assertFalse(exception, "exception in listener") 153 assertFalse(exception, "exception in listener");
154 // Make sure that the debug event listener vas invoked. 154 // Make sure that the debug event listener vas invoked.
155 assertTrue(listenerComplete, "listener did not run to completion"); 155 assertTrue(listenerComplete, "listener did not run to completion");
OLDNEW
« no previous file with comments | « test/mjsunit/debug-enable-disable-breakpoints.js ('k') | test/mjsunit/debug-evaluate-arguments.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698