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

Unified Diff: LayoutTests/inspector/debugger/debugger-script-preprocessor.html

Issue 13575004: Apply script preprocessor to Web page scripts only. (Closed) Base URL: https://chromium.googlesource.com/external/WebKit_trimmed.git@master
Patch Set: rebase Created 7 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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/inspector/debugger/debugger-script-preprocessor.html
diff --git a/LayoutTests/inspector/debugger/debugger-script-preprocessor.html b/LayoutTests/inspector/debugger/debugger-script-preprocessor.html
index b70250173734cbfd4b3b1fe9f4e472c49222fd44..dd2de0d7720432fb6018de9f2062c5f4bc1926d9 100644
--- a/LayoutTests/inspector/debugger/debugger-script-preprocessor.html
+++ b/LayoutTests/inspector/debugger/debugger-script-preprocessor.html
@@ -6,8 +6,9 @@
function load()
{
- eval("function dynamic" + "Script1() {}");
- eval("function dynamic" + "Script2() {}");
+ eval("function dynamic" + "Script1() {}");
+ eval("function dynamic" + "Script2() {}");
+ eval("function dynamic" + "RuntimeError() {}");
runTest();
}
@@ -15,25 +16,39 @@ function test()
{
function preprocessor(script, name)
{
+ function makeResponse(script, msg, name) {
+ // write into the console to verify that no internal scripts are preprocessed.
+ return script + ';\nconsole.log(\"' + msg + '\");\n//# sourceURL=' + name;
+ }
if (script.indexOf("dynamic" + "Script1") !== -1)
- return script + "//# sourceURL=dynamicScript1";
+ return makeResponse(script, 'eval works', "dynamic"+"Script1");
if (script.indexOf("dynamic" + "Script2") !== -1) {
try {
- var w = eval("window");
- return script + "//# sourceURL=FAIL_window_should_not_be_there";
+ var w = eval("window.location.href");
+ return makeResponse(script, 'verify href: ' + w, "dynamic"+"Script2");
} catch (e) {
- return script + "//# sourceURL=dynamicScript2";
+ return makeResponse(script, 'Did not find window href', "dynamic"+"Script2");
}
}
- // Verify that the |name| argument is correct. Note: if name is not passed in
- // the results will be a script with a sourceURL equal to the original file name.
- return script + "//# sourceURL=" + name + ".js";
+ if (script.indexOf("dynamic" + "RuntimeError") !== -1) {
+ throw new Error("fake internal preprocessor error");
+ }
+ if (!this.String) {
+ throw new Error("preprocessor environment is missing built-ins");
+ }
+
+ // Verify that the |name| argument is correct.
+ name = name.replace(/\.html/, '_html') + '.js';
+ return makeResponse(script, 'preprocessed ' + name, name);
}
+ var failingPreprocessor = "(function() {throw new Error(\"failingPreprocessor throws\");}())"
+ var syntaxError = "(function preprocessor(script, name) {\nsyntax error\n}\n)";
+
InspectorTest.startDebuggerTest(step1);
function step1()
- {
+ {
InspectorTest.reloadPage(step2, undefined, "(" + preprocessor + ")");
}
@@ -45,8 +60,23 @@ function test()
}
var scripts = InspectorTest.queryScripts(accept);
for (var i = 0; i < scripts.length; ++i)
- InspectorTest.addResult(WebInspector.displayNameForURL(scripts[i].sourceURL));
+ InspectorTest.addResult("sourceURLs[" + i +"]=" + WebInspector.displayNameForURL(scripts[i].sourceURL));
+
+ InspectorTest.reloadPage(step3, undefined, "(" + failingPreprocessor + ")");
+ }
+ function step3()
+ {
+ InspectorTest.reloadPage(step4, undefined, "var notAFunctionOrSyntaxError;");
+ }
+
+ function step4()
+ {
+ InspectorTest.reloadPage(step5, undefined, syntaxError);
+ }
+
+ function step5()
+ {
InspectorTest.reloadPage(InspectorTest.completeDebuggerTest.bind(InspectorTest));
}
}

Powered by Google App Engine
This is Rietveld 408576698