<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>149391</bug_id>
          
          <creation_ts>2015-09-20 14:13:37 -0700</creation_ts>
          <short_desc>WebKitTestRunner can leak user content script results between tests</short_desc>
          <delta_ts>2015-09-20 17:29:03 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>Safari 9</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alexey Proskuryakov">ap</reporter>
          <assigned_to name="Alexey Proskuryakov">ap</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>thorton</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1127520</commentid>
    <comment_count>0</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2015-09-20 14:13:37 -0700</bug_when>
    <thetext>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&apos;s already running a new test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1127521</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2015-09-20 14:14:48 -0700</bug_when>
    <thetext>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</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1127522</commentid>
    <comment_count>2</comment_count>
      <attachid>261613</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2015-09-20 14:17:08 -0700</bug_when>
    <thetext>Created attachment 261613
proposed fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1127541</commentid>
    <comment_count>3</comment_count>
      <attachid>261613</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2015-09-20 17:29:00 -0700</bug_when>
    <thetext>Comment on attachment 261613
proposed fix

Clearing flags on attachment: 261613

Committed r190035: &lt;http://trac.webkit.org/changeset/190035&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1127542</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2015-09-20 17:29:03 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>261613</attachid>
            <date>2015-09-20 14:17:08 -0700</date>
            <delta_ts>2015-09-20 17:29:00 -0700</delta_ts>
            <desc>proposed fix</desc>
            <filename>UserContent.txt</filename>
            <type>text/plain</type>
            <size>2420</size>
            <attacher name="Alexey Proskuryakov">ap</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDE5MDAzMykKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE4IEBACisyMDE1LTA5LTIwICBBbGV4ZXkgUHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29t
PgorCisgICAgICAgIFdlYktpdFRlc3RSdW5uZXIgY2FuIGxlYWsgdXNlciBjb250ZW50IHNjcmlw
dCByZXN1bHRzIGJldHdlZW4gdGVzdHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTE0OTM5MQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgIE1vdmUgV0tCdW5kbGVQYWdlUmVtb3ZlQWxsVXNlckNvbnRlbnQgY2Fs
bCBzbyB0aGF0IGl0J3MgZXhlY3V0ZWQgYmVmb3JlIGxvYWRpbmcKKyAgICAgICAgYWJvdXQ6Ymxh
bmssIGFuZCB0aHVzIGFib3V0OmJsYW5rIGRvZXNuJ3QgaGF2ZSB1bmludGVuZGVkIHNpZGUgZWZm
ZWN0cy4KKworICAgICAgICAqIFdlYktpdFRlc3RSdW5uZXIvSW5qZWN0ZWRCdW5kbGUvSW5qZWN0
ZWRCdW5kbGUuY3BwOgorICAgICAgICAoV1RSOjpJbmplY3RlZEJ1bmRsZTo6YmVnaW5UZXN0aW5n
KToKKyAgICAgICAgKiBXZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL0luamVjdGVkQnVu
ZGxlUGFnZS5jcHA6CisgICAgICAgIChXVFI6OkluamVjdGVkQnVuZGxlUGFnZTo6cmVzZXRBZnRl
clRlc3QpOgorCiAyMDE1LTA5LTIwICBBbGV4ZXkgUHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29t
PgogCiAgICAgICAgIFtNYWMsIGlPU10gQWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIgZG9lc24ndCB1
bmluc3RhbGwgZ2xvYmFsIG5vdGlmaWNhdGlvbiBoYW5kbGVyCkluZGV4OiBUb29scy9XZWJLaXRU
ZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL0luamVjdGVkQnVuZGxlLmNwcAo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSBUb29scy9XZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL0luamVjdGVkQnVuZGxlLmNw
cAkocmV2aXNpb24gMTkwMDMxKQorKysgVG9vbHMvV2ViS2l0VGVzdFJ1bm5lci9JbmplY3RlZEJ1
bmRsZS9JbmplY3RlZEJ1bmRsZS5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTI4MSw4ICsyODEsNiBA
QCB2b2lkIEluamVjdGVkQnVuZGxlOjpiZWdpblRlc3RpbmcoV0tEaWN0CiAgICAgV0tCdW5kbGVT
ZXRQbHVnaW5zRW5hYmxlZChtX2J1bmRsZSwgbV9wYWdlR3JvdXAsIHRydWUpOwogICAgIFdLQnVu
ZGxlU2V0UG9wdXBCbG9ja2luZ0VuYWJsZWQobV9idW5kbGUsIG1fcGFnZUdyb3VwLCBmYWxzZSk7
CiAKLSAgICBXS0J1bmRsZVBhZ2VSZW1vdmVBbGxVc2VyQ29udGVudChwYWdlKCktPnBhZ2UoKSk7
Ci0KICNpZiBQTEFURk9STShJT1MpCiAgICAgV0tCdW5kbGVQYWdlU2V0VXNlVGVzdGluZ1ZpZXdw
b3J0Q29uZmlndXJhdGlvbihwYWdlKCktPnBhZ2UoKSwgIWJvb2xlYW5Gb3JLZXkoc2V0dGluZ3Ms
ICJVc2VGbGV4aWJsZVZpZXdwb3J0IikpOwogI2VuZGlmCkluZGV4OiBUb29scy9XZWJLaXRUZXN0
UnVubmVyL0luamVjdGVkQnVuZGxlL0luamVjdGVkQnVuZGxlUGFnZS5jcHAKPT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot
LS0gVG9vbHMvV2ViS2l0VGVzdFJ1bm5lci9JbmplY3RlZEJ1bmRsZS9JbmplY3RlZEJ1bmRsZVBh
Z2UuY3BwCShyZXZpc2lvbiAxOTAwMzEpCisrKyBUb29scy9XZWJLaXRUZXN0UnVubmVyL0luamVj
dGVkQnVuZGxlL0luamVjdGVkQnVuZGxlUGFnZS5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTQyNyw2
ICs0MjcsOSBAQCB2b2lkIEluamVjdGVkQnVuZGxlUGFnZTo6cmVzZXRBZnRlclRlc3QoCiAgICAg
SlNHbG9iYWxDb250ZXh0UmVmIGNvbnRleHQgPSBXS0J1bmRsZUZyYW1lR2V0SmF2YVNjcmlwdENv
bnRleHQoZnJhbWUpOwogICAgIFdlYkNvcmVUZXN0U3VwcG9ydDo6cmVzZXRJbnRlcm5hbHNPYmpl
Y3QoY29udGV4dCk7CiAgICAgYXNzaWduZWRVcmxzQ2FjaGUuY2xlYXIoKTsKKworICAgIC8vIFVz
ZXIgc2NyaXB0cyBuZWVkIHRvIGJlIHJlbW92ZWQgYWZ0ZXIgdGhlIHRlc3QgYW5kIGJlZm9yZSBs
b2FkaW5nIGFib3V0OmJsYW5rLCBhcyBvdGhlcndpc2UgdGhleSB3b3VsZCBydW4gaW4gYWJvdXQ6
YmxhbmssIGFuZCBwb3RlbnRpYWxseSBsZWFrIHJlc3VsdHMgaW50byBhIHN1YnNlcXVlc3QgdGVz
dC4KKyAgICBXS0J1bmRsZVBhZ2VSZW1vdmVBbGxVc2VyQ29udGVudChtX3BhZ2UpOwogfQogCiAv
LyBMb2FkZXIgQ2xpZW50IENhbGxiYWNrcwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>