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

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-sync-content-nodejs.html

Issue 2542703002: DevTools: Editing of node.js should not add a new line after shebang. (Closed)
Patch Set: 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 | « no previous file | third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-sync-content-nodejs-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <html> 1 <html>
2 <head> 2 <head>
3 <script src="../inspector-test.js"></script> 3 <script src="../inspector-test.js"></script>
4 <script src="../debugger-test.js"></script> 4 <script src="../debugger-test.js"></script>
5 <script src="../isolated-filesystem-test.js"></script> 5 <script src="../isolated-filesystem-test.js"></script>
6 <script src="./persistence-test.js"></script> 6 <script src="./persistence-test.js"></script>
7 <script> 7 <script>
8 8
9 function test() 9 function test()
10 { 10 {
11 // Pretend we are running under V8 front-end. 11 // Pretend we are running under V8 front-end.
12 SDK.targetManager.mainTarget().setIsNodeJSForTest(); 12 SDK.targetManager.mainTarget().setIsNodeJSForTest();
13 13
14 var content = [ 14 var content = [
15 '', 15 '',
16 '',
16 'var express = require("express");', 17 'var express = require("express");',
17 '//TODO' 18 '//TODO'
18 ].join("\n"); 19 ].join("\n");
19 20
20 var fsContent = Persistence.Persistence._NodeShebang + content; 21 var fsContent = Persistence.Persistence._NodeShebang + content;
21 var nodeContent = Persistence.Persistence._NodePrefix + content + Persistenc e.Persistence._NodeSuffix; 22 var nodeContent = Persistence.Persistence._NodePrefix + content + Persistenc e.Persistence._NodeSuffix;
22 23
23 InspectorTest.addResult("Initial fileSystem content:"); 24 InspectorTest.addResult("Initial fileSystem content:");
24 InspectorTest.addResult(indent(fsContent)); 25 InspectorTest.addResult(indent(fsContent));
25 InspectorTest.addResult("\n Initial network content:"); 26 InspectorTest.addResult("\n Initial network content:");
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 { 89 {
89 return content.split("\n").map(line => " " + line).join("\n"); 90 return content.split("\n").map(line => " " + line).join("\n");
90 } 91 }
91 }; 92 };
92 </script> 93 </script>
93 </head> 94 </head>
94 <body onload="runTest()"> 95 <body onload="runTest()">
95 <p>Verify that syncing Node.js contents works fine.</p> 96 <p>Verify that syncing Node.js contents works fine.</p>
96 </body> 97 </body>
97 </html> 98 </html>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-sync-content-nodejs-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698