Bug 149391

Summary: WebKitTestRunner can leak user content script results between tests
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: Tools / TestsAssignee: Alexey Proskuryakov <ap>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, thorton
Priority: P2    
Version: Safari 9   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed fix none

Description Alexey Proskuryakov 2015-09-20 14:13:37 PDT
WebKitTestRunner removes user content before starting a new test. But this means that user content scripts run in about:blank that is loaded between tests, and can produce output. They can even install timers that will run when WKTR thinks that it's already running a new test.
Comment 1 Alexey Proskuryakov 2015-09-20 14:14:48 PDT
Steps to reproduce: run these tests in order, in a single process:

userscripts/window-onerror-for-isolated-world-1.html
userscripts/window-onerror-for-isolated-world-2.html
webgl/1.0.2/conformance/glsl/literals/float_literal.vert.html
Comment 2 Alexey Proskuryakov 2015-09-20 14:17:08 PDT
Created attachment 261613 [details]
proposed fix
Comment 3 WebKit Commit Bot 2015-09-20 17:29:00 PDT
Comment on attachment 261613 [details]
proposed fix

Clearing flags on attachment: 261613

Committed r190035: <http://trac.webkit.org/changeset/190035>
Comment 4 WebKit Commit Bot 2015-09-20 17:29:03 PDT
All reviewed patches have been landed.  Closing bug.