Skip to content

Commit

Permalink
Predeclare globals required by the test framework to avoid walking th…
Browse files Browse the repository at this point in the history
…e prototype chain.

Otherwise http/tests/security/xss-DENIED-method-with-iframe-proto.html double-faults while trying to read those global variables that aren't declared.

BUG=

Review URL: https://codereview.chromium.org/405893003

git-svn-id: svn://svn.chromium.org/blink/trunk@178553 bbb929c8-8fbe-4397-9dbb-9b2b20218538
  • Loading branch information
verwaest@chromium.org committed Jul 21, 2014
1 parent b1cd3dc commit 0d80a8e
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions third_party/WebKit/LayoutTests/resources/js-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -726,6 +726,8 @@ function isSuccessfullyParsed()
debug('<br /><span class="pass">TEST COMPLETE</span>');
}

var wasPostTestScriptParsed, wasFinishJSTestCalled, jsTestIsAsync;

// It's possible for an async test to call finishJSTest() before js-test-post.js
// has been parsed.
function finishJSTest()
Expand Down

0 comments on commit 0d80a8e

Please sign in to comment.