LayoutTests/inspector/sources/debugger/navigator-view.html - Issue 368643002: DevTools: Make addFileForURL more robust

Side by Side Diff: LayoutTests/inspector/sources/debugger/navigator-view.html

Issue 368643002: DevTools: Make addFileForURL more robust (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Compacting the malformed URL Created 6 years, 5 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 | Annotate | Revision Log

OLD	NEW
1 <html>	1 <html>

2 <head>	2 <head>

3 <script src="../../../http/tests/inspector/inspector-test.js"></script>	3 <script src="../../../http/tests/inspector/inspector-test.js"></script>

4 <script src="../../../http/tests/inspector/debugger-test.js"></script>	4 <script src="../../../http/tests/inspector/debugger-test.js"></script>

5 <script src="../../../http/tests/inspector/workspace-test.js"></script>	5 <script src="../../../http/tests/inspector/workspace-test.js"></script>

6	6

7 <script>	7 <script>

8 function test()	8 function test()

9 {	9 {

10 InspectorTest.createWorkspace(true);	10 InspectorTest.createWorkspace(true);

(...skipping 48 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
59 dumpNavigator();	59 dumpNavigator();

60	60

61 InspectorTest.addResult("Adding content scripts and some random resources:") ;	61 InspectorTest.addResult("Adding content scripts and some random resources:") ;

62 addUISourceCode(rootURL + "foo/bar/contentScript2.js?a=1", true);	62 addUISourceCode(rootURL + "foo/bar/contentScript2.js?a=1", true);

63 addUISourceCode(rootURL + "foo/bar/contentScript.js?a=2", true);	63 addUISourceCode(rootURL + "foo/bar/contentScript.js?a=2", true);

64 addUISourceCode(rootURL + "foo/bar/contentScript.js?a=1", true);	64 addUISourceCode(rootURL + "foo/bar/contentScript.js?a=1", true);

65 addUISourceCode("http://example.com/", false);	65 addUISourceCode("http://example.com/", false);

66 addUISourceCode("http://example.com/?a=b", false);	66 addUISourceCode("http://example.com/?a=b", false);

67 addUISourceCode("?a=b", false);	67 addUISourceCode("?a=b", false);

68 addUISourceCode("very_looooooooooooooooooooooooooooooooooooooooooooooooooooo oooooooooooooooooooooooooooooooooooooooong_url", false);	68 addUISourceCode("very_looooooooooooooooooooooooooooooooooooooooooooooooooooo oooooooooooooooooooooooooooooooooooooooong_url", false);

	69 addUISourceCode("%malformedURI", false);
	aandrey 2014/07/02 12:33:05 could you also test a real world example, like: r could you also test a real world example, like: rootURL + "foo/bar/contentScript.js?a=10% vsevik 2014/07/02 12:43:40 I think we only need one URL for this test: "http: I think we only need one URL for this test: "http://example.com/foo?bar=100%&baz=a%20b". This will emphasizes the fact that decodeURI doesn't work for this case. alph 2014/07/02 13:14:34 Done. Show quoted text On 2014/07/02 12:43:40, vsevik wrote: > I think we only need one URL for this test: > "http://example.com/foo?bar=100%&baz=a%20b". > This will emphasizes the fact that decodeURI doesn't work for this case. Done.
69 dumpNavigator();	70 dumpNavigator();

70	71

71 InspectorTest.addResult("Revealing first resource:");	72 InspectorTest.addResult("Revealing first resource:");

72 revealUISourceCode(uiSourceCodes[0]);	73 revealUISourceCode(uiSourceCodes[0]);

73 dumpNavigator();	74 dumpNavigator();

74	75

75 // Here we keep http://localhost:8080/LayoutTests/inspector/debugger2/ folde r collapsed while adding resources into it.	76 // Here we keep http://localhost:8080/LayoutTests/inspector/debugger2/ folde r collapsed while adding resources into it.

76 InspectorTest.addResult("Adding some resources to change the way debugger fo lder looks like, first:");	77 InspectorTest.addResult("Adding some resources to change the way debugger fo lder looks like, first:");

77 var rootURL2 = "http://localhost:8080/LayoutTests/inspector/debugger2/";	78 var rootURL2 = "http://localhost:8080/LayoutTests/inspector/debugger2/";

78 addUISourceCode(rootURL2 + "foo/bar/script.js", false);	79 addUISourceCode(rootURL2 + "foo/bar/script.js", false);

(...skipping 27 matching lines...) Expand all Loading...
106 </script>	107 </script>

107	108

108 </head>	109 </head>

109 <body onload="runTest()">	110 <body onload="runTest()">

110 <p>	111 <p>

111 Tests scripts panel file selectors.	112 Tests scripts panel file selectors.

112 </p>	113 </p>

113 </body>	114 </body>

114	115

115 </html>	116 </html>

OLD	NEW