<?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>107987</bug_id>
          
          <creation_ts>2013-01-25 15:31:22 -0800</creation_ts>
          <short_desc>Make unload-reparent-sibling-frame.html wait for completion</short_desc>
          <delta_ts>2013-01-25 17:14:49 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>New Bugs</component>
          <version>528+ (Nightly build)</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="Elliott Sprehn">esprehn</reporter>
          <assigned_to name="Elliott Sprehn">esprehn</assigned_to>
          <cc>arv</cc>
    
    <cc>eric</cc>
    
    <cc>ojan</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>816955</commentid>
    <comment_count>0</comment_count>
    <who name="Elliott Sprehn">esprehn</who>
    <bug_when>2013-01-25 15:31:22 -0800</bug_when>
    <thetext>Make unload-reparent-sibling-frame.html wait for completion</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>816962</commentid>
    <comment_count>1</comment_count>
      <attachid>184819</attachid>
    <who name="Elliott Sprehn">esprehn</who>
    <bug_when>2013-01-25 15:35:44 -0800</bug_when>
    <thetext>Created attachment 184819
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>816998</commentid>
    <comment_count>2</comment_count>
      <attachid>184819</attachid>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2013-01-25 16:00:16 -0800</bug_when>
    <thetext>Comment on attachment 184819
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=184819&amp;action=review

This is kind of changing what this test is testing, no? As in, it used to also test that we&apos;d fire the onload events for both frames when they&apos;re reparented and the description seems to be that it&apos;s explicitly testing that. Adding waituntildone/notifydone makes sense, but I think you need to notifyDone in teh onload handlers.

&gt; LayoutTests/fast/frames/unload-reparent-sibling-frame.html:40
&gt; +            isSuccessfullyParsed();

Ideally you&apos;d include js-test-post.js. That ensures that the test doesn&apos;t appear to pass if you accidentally introduce a JS error. I don&apos;t think this actually does what it&apos;s intended to if don&apos;t include js-test-post.js (i.e. it&apos;s supposed to spit out an error if it&apos;s not successfully parsed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817002</commentid>
    <comment_count>3</comment_count>
      <attachid>184819</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2013-01-25 16:03:53 -0800</bug_when>
    <thetext>Comment on attachment 184819
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=184819&amp;action=review

&gt; LayoutTests/fast/frames/unload-reparent-sibling-frame.html:37
&gt; +        setTimeout(function() {

Why the timeout?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817011</commentid>
    <comment_count>4</comment_count>
    <who name="Elliott Sprehn">esprehn</who>
    <bug_when>2013-01-25 16:10:03 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 184819 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=184819&amp;action=review
&gt; 
&gt; This is kind of changing what this test is testing, no? As in, it used to also test that we&apos;d fire the onload events for both frames when they&apos;re reparented and the description seems to be that it&apos;s explicitly testing that. Adding waituntildone/notifydone makes sense, but I think you need to notifyDone in teh onload handlers.

We know it fired the onload event if the text in the dump contains the value of the srcdoc. You can&apos;t depend on the onload handlers because they may fire synchronously inside appendChild.

&gt; 
&gt; &gt; LayoutTests/fast/frames/unload-reparent-sibling-frame.html:40
&gt; &gt; +            isSuccessfullyParsed();
&gt; 
&gt; Ideally you&apos;d include js-test-post.js. That ensures that the test doesn&apos;t appear to pass if you accidentally introduce a JS error. I don&apos;t think this actually does what it&apos;s intended to if don&apos;t include js-test-post.js (i.e. it&apos;s supposed to spit out an error if it&apos;s not successfully parsed.

Nah, js-test-post.js in async mode just waits for you to call finishJSTest() and then calls isSuccessfullyParsed() which is exactly equivalent to this code. If I used post.js I&apos;d need to set jsTestIsAsync = true and then call finishJsTest() where I have isSuccessfullyParsed() right now and the output would be identical.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817018</commentid>
    <comment_count>5</comment_count>
    <who name="Elliott Sprehn">esprehn</who>
    <bug_when>2013-01-25 16:15:06 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 184819 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=184819&amp;action=review
&gt; 
&gt; &gt; LayoutTests/fast/frames/unload-reparent-sibling-frame.html:37
&gt; &gt; +        setTimeout(function() {
&gt; 
&gt; Why the timeout?

We have to wait until the current ChildFrameDisconnector::disconnect() call completes before looking at the loaded state of the documents.

The bug this test is for is that we&apos;re in the middle of iterating the list of frames to disconnect in ChildFrameDisconnector::disconnectFrameOwners(), and frame at index N moves the frame at index N+k to a different location in the document but with the same parentElement() by moving some ancestor, then when we get to frame N+k we disconnect it even though it&apos;s no longer in the subtree we should be disconnecting because the old isValid() method would have returned true.

See: https://trac.webkit.org/changeset/140856</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817033</commentid>
    <comment_count>6</comment_count>
      <attachid>184819</attachid>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2013-01-25 16:38:13 -0800</bug_when>
    <thetext>Comment on attachment 184819
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=184819&amp;action=review

&gt;&gt;&gt; LayoutTests/fast/frames/unload-reparent-sibling-frame.html:40
&gt;&gt;&gt; +            isSuccessfullyParsed();
&gt;&gt; 
&gt;&gt; Ideally you&apos;d include js-test-post.js. That ensures that the test doesn&apos;t appear to pass if you accidentally introduce a JS error. I don&apos;t think this actually does what it&apos;s intended to if don&apos;t include js-test-post.js (i.e. it&apos;s supposed to spit out an error if it&apos;s not successfully parsed.
&gt; 
&gt; Nah, js-test-post.js in async mode just waits for you to call finishJSTest() and then calls isSuccessfullyParsed() which is exactly equivalent to this code. If I used post.js I&apos;d need to set jsTestIsAsync = true and then call finishJsTest() where I have isSuccessfullyParsed() right now and the output would be identical.

You&apos;re right. That said, the more code we have that digs into the internals of the js-test functions, the harder it is to refactor them later (e.g. to get rid of js-test-post).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817037</commentid>
    <comment_count>7</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2013-01-25 16:39:05 -0800</bug_when>
    <thetext>(In reply to comment #4)
&gt; (In reply to comment #2)
&gt; &gt; (From update of attachment 184819 [details] [details])
&gt; &gt; View in context: https://bugs.webkit.org/attachment.cgi?id=184819&amp;action=review
&gt; &gt; 
&gt; &gt; This is kind of changing what this test is testing, no? As in, it used to also test that we&apos;d fire the onload events for both frames when they&apos;re reparented and the description seems to be that it&apos;s explicitly testing that. Adding waituntildone/notifydone makes sense, but I think you need to notifyDone in teh onload handlers.
&gt; 
&gt; We know it fired the onload event if the text in the dump contains the value of the srcdoc. You can&apos;t depend on the onload handlers because they may fire synchronously inside appendChild.

Isn&apos;t this a race condition? If the onload hasn&apos;t fired yet, the text dump will have the wrong output, no?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817042</commentid>
    <comment_count>8</comment_count>
      <attachid>184838</attachid>
    <who name="Elliott Sprehn">esprehn</who>
    <bug_when>2013-01-25 16:44:09 -0800</bug_when>
    <thetext>Created attachment 184838
Patch

Address review comments</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817043</commentid>
    <comment_count>9</comment_count>
    <who name="Elliott Sprehn">esprehn</who>
    <bug_when>2013-01-25 16:44:40 -0800</bug_when>
    <thetext>(In reply to comment #7)
&gt; ...
&gt; Isn&apos;t this a race condition? If the onload hasn&apos;t fired yet, the text dump will have the wrong output, no?

I think you&apos;re right, fixed in the new patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817066</commentid>
    <comment_count>10</comment_count>
      <attachid>184838</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-25 17:14:45 -0800</bug_when>
    <thetext>Comment on attachment 184838
Patch

Clearing flags on attachment: 184838

Committed r140885: &lt;http://trac.webkit.org/changeset/140885&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>817067</commentid>
    <comment_count>11</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-25 17:14:49 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>184819</attachid>
            <date>2013-01-25 15:35:44 -0800</date>
            <delta_ts>2013-01-25 16:44:05 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-107987-20130125183234.patch</filename>
            <type>text/plain</type>
            <size>2943</size>
            <attacher name="Elliott Sprehn">esprehn</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQwODY3CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNzUxMjBmOWRmOGM3MmM2MDJmMjhi
NWU5ZmNhMjBjZjgzMTA1OGNhNS4uNzRhOTZjZjlmZGU1ZmEwNTFiODhiMWRkMGI2NGJlOTMxNjBj
MWE1MCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEzLTAxLTI1ICBFbGxpb3R0IFNwcmVobiAg
PGVzcHJlaG5AY2hyb21pdW0ub3JnPgorCisgICAgICAgIE1ha2UgdW5sb2FkLXJlcGFyZW50LXNp
YmxpbmctZnJhbWUuaHRtbCB3YWl0IGZvciBjb21wbGV0aW9uCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDc5ODcKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXaGlsZSB0aGlzIHRlc3QgcnVucyBmaW5lIGxv
Y2FsbHkgYW5kIGluIHRoZSBjci1saW51eCBFV1MgaXQKKyAgICAgICAgYXBwZWFycyB0byBiZSBm
bGFreSBvbiBtYW55IG9mIHRoZSBvdGhlciBib3RzIGJlY2F1c2UgaXQgZG9lc24ndAorICAgICAg
ICB3YWl0IGZvciB0aGUgdGVzdCB0byBjb21wbGV0ZS4gSW5zdGVhZCBtYWtlIHRoZSB3YWl0aW5n
IGluIHRoZSB0ZXN0CisgICAgICAgIGV4cGxpY2l0LgorCisgICAgICAgICogZmFzdC9mcmFtZXMv
dW5sb2FkLXJlcGFyZW50LXNpYmxpbmctZnJhbWUuaHRtbDoKKwogMjAxMy0wMS0yNSAgTGV2aSBX
ZWludHJhdWIgIDxsZXZpd0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgVW5yZXZpZXdlZCBnYXJk
ZW5pbmcuIFVwZGF0aW5nIGJhc2VsaW5lcyBmb2xsb3dpbmcgcjE0MDE5Mi4KZGlmZiAtLWdpdCBh
L0xheW91dFRlc3RzL2Zhc3QvZnJhbWVzL3VubG9hZC1yZXBhcmVudC1zaWJsaW5nLWZyYW1lLmh0
bWwgYi9MYXlvdXRUZXN0cy9mYXN0L2ZyYW1lcy91bmxvYWQtcmVwYXJlbnQtc2libGluZy1mcmFt
ZS5odG1sCmluZGV4IDI3NjE0OGQ0MTFjMTk1NWIxMGVlYTlmNDBhZmVkNjM5YjMzYzM0NDAuLjkx
ZDM4NDNiM2Q5MjY2ZTBiZDA1YTQ5YjJhNTk1ZTIxYmI4NDkxZGUgMTAwNjQ0Ci0tLSBhL0xheW91
dFRlc3RzL2Zhc3QvZnJhbWVzL3VubG9hZC1yZXBhcmVudC1zaWJsaW5nLWZyYW1lLmh0bWwKKysr
IGIvTGF5b3V0VGVzdHMvZmFzdC9mcmFtZXMvdW5sb2FkLXJlcGFyZW50LXNpYmxpbmctZnJhbWUu
aHRtbApAQCAtNCw2ICs0LDggQEAKIAogPGRpdiBpZD0iY29udGFpbmVyIj4KICAgICA8ZGl2IGlk
PSJpbnNpZGUiPgorICAgICAgICA8aWZyYW1lIGlkPSJmcmFtZTEiIHNyY2RvYz0iZnJhbWUxIj48
L2lmcmFtZT4KKyAgICAgICAgPGlmcmFtZSBpZD0iZnJhbWUyIiBzcmNkb2M9ImZyYW1lMiI+PC9p
ZnJhbWU+CiAgICAgPC9kaXY+CiA8L2Rpdj4KIApAQCAtMTMsNiArMTUsNyBAQCBkZXNjcmlwdGlv
bignUmVwYXJlbnRlZCBzaWJsaW5nIGZyYW1lcyBmcm9tIHVubG9hZCBoYW5kbGVycyBzaG91bGQg
bG9hZC4nKTsKIGlmICh3aW5kb3cudGVzdFJ1bm5lcikgewogICAgIHRlc3RSdW5uZXIuZHVtcEFz
VGV4dCgpOwogICAgIHRlc3RSdW5uZXIuZHVtcENoaWxkRnJhbWVzQXNUZXh0KCk7CisgICAgdGVz
dFJ1bm5lci53YWl0VW50aWxEb25lKCk7CiB9CiAKIGZ1bmN0aW9uIG9uZnJhbWVsb2FkKHRleHQp
CkBAIC0yMCwyMCArMjMsMTAgQEAgZnVuY3Rpb24gb25mcmFtZWxvYWQodGV4dCkKICAgICB0aGlz
LmNvbnRlbnREb2N1bWVudC5ib2R5LmlubmVySFRNTCA9IHRoaXMuaWQ7CiB9CiAKLWZ1bmN0aW9u
IHJ1blRlc3QoKQoteyAgICAKLX0KLQogdmFyIGNvbnRhaW5lciA9IGRvY3VtZW50LmdldEVsZW1l
bnRCeUlkKCdjb250YWluZXInKTsKIHZhciBpbnNpZGUgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJ
ZCgnaW5zaWRlJyk7Ci0KLXZhciBmcmFtZTEgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50KCdpZnJh
bWUnKTsKLWZyYW1lMS5pZCA9ICdmcmFtZTEnOwotZnJhbWUxLm9ubG9hZCA9IG9uZnJhbWVsb2Fk
OwotCi12YXIgZnJhbWUyID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgnaWZyYW1lJyk7Ci1mcmFt
ZTIuaWQgPSAnZnJhbWUyJzsKLWZyYW1lMi5vbmxvYWQgPSBvbmZyYW1lbG9hZDsKK3ZhciBmcmFt
ZTEgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnZnJhbWUxJyk7Cit2YXIgZnJhbWUyID0gZG9j
dW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2ZyYW1lMicpOwogCiBvbmxvYWQgPSBmdW5jdGlvbigpIHsK
ICAgICBpbnNpZGUuYXBwZW5kQ2hpbGQoZnJhbWUxKTsKQEAgLTQxLDExICszNCwxNSBAQCBvbmxv
YWQgPSBmdW5jdGlvbigpIHsKIAogICAgIGZyYW1lMS5jb250ZW50V2luZG93Lm9udW5sb2FkID0g
ZnVuY3Rpb24oKSB7CiAgICAgICAgIGRvY3VtZW50LmJvZHkuYXBwZW5kQ2hpbGQoaW5zaWRlKTsK
KyAgICAgICAgc2V0VGltZW91dChmdW5jdGlvbigpIHsKKyAgICAgICAgICAgIHNob3VsZE5vdEJl
KCdmcmFtZTEuY29udGVudERvY3VtZW50JywgJ251bGwnKTsKKyAgICAgICAgICAgIHNob3VsZE5v
dEJlKCdmcmFtZTIuY29udGVudERvY3VtZW50JywgJ251bGwnKTsKKyAgICAgICAgICAgIGlzU3Vj
Y2Vzc2Z1bGx5UGFyc2VkKCk7CisgICAgICAgICAgICBpZiAod2luZG93LnRlc3RSdW5uZXIpCisg
ICAgICAgICAgICAgICAgdGVzdFJ1bm5lci5ub3RpZnlEb25lKCk7CisgICAgICAgIH0sIDApOwog
ICAgIH07CiAKICAgICBjb250YWluZXIucGFyZW50Tm9kZS5yZW1vdmVDaGlsZChjb250YWluZXIp
OwotICAgIHNob3VsZE5vdEJlKCdmcmFtZTEuY29udGVudERvY3VtZW50JywgJ251bGwnKTsKLSAg
ICBzaG91bGROb3RCZSgnZnJhbWUyLmNvbnRlbnREb2N1bWVudCcsICdudWxsJyk7Ci0gICAgaXNT
dWNjZXNzZnVsbHlQYXJzZWQoKTsKIH07CiA8L3NjcmlwdD4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>184838</attachid>
            <date>2013-01-25 16:44:09 -0800</date>
            <delta_ts>2013-01-25 17:14:45 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-107987-20130125194058.patch</filename>
            <type>text/plain</type>
            <size>3166</size>
            <attacher name="Elliott Sprehn">esprehn</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQwODY3CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNzUxMjBmOWRmOGM3MmM2MDJmMjhi
NWU5ZmNhMjBjZjgzMTA1OGNhNS4uNzRhOTZjZjlmZGU1ZmEwNTFiODhiMWRkMGI2NGJlOTMxNjBj
MWE1MCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEzLTAxLTI1ICBFbGxpb3R0IFNwcmVobiAg
PGVzcHJlaG5AY2hyb21pdW0ub3JnPgorCisgICAgICAgIE1ha2UgdW5sb2FkLXJlcGFyZW50LXNp
YmxpbmctZnJhbWUuaHRtbCB3YWl0IGZvciBjb21wbGV0aW9uCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDc5ODcKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXaGlsZSB0aGlzIHRlc3QgcnVucyBmaW5lIGxv
Y2FsbHkgYW5kIGluIHRoZSBjci1saW51eCBFV1MgaXQKKyAgICAgICAgYXBwZWFycyB0byBiZSBm
bGFreSBvbiBtYW55IG9mIHRoZSBvdGhlciBib3RzIGJlY2F1c2UgaXQgZG9lc24ndAorICAgICAg
ICB3YWl0IGZvciB0aGUgdGVzdCB0byBjb21wbGV0ZS4gSW5zdGVhZCBtYWtlIHRoZSB3YWl0aW5n
IGluIHRoZSB0ZXN0CisgICAgICAgIGV4cGxpY2l0LgorCisgICAgICAgICogZmFzdC9mcmFtZXMv
dW5sb2FkLXJlcGFyZW50LXNpYmxpbmctZnJhbWUuaHRtbDoKKwogMjAxMy0wMS0yNSAgTGV2aSBX
ZWludHJhdWIgIDxsZXZpd0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgVW5yZXZpZXdlZCBnYXJk
ZW5pbmcuIFVwZGF0aW5nIGJhc2VsaW5lcyBmb2xsb3dpbmcgcjE0MDE5Mi4KZGlmZiAtLWdpdCBh
L0xheW91dFRlc3RzL2Zhc3QvZnJhbWVzL3VubG9hZC1yZXBhcmVudC1zaWJsaW5nLWZyYW1lLmh0
bWwgYi9MYXlvdXRUZXN0cy9mYXN0L2ZyYW1lcy91bmxvYWQtcmVwYXJlbnQtc2libGluZy1mcmFt
ZS5odG1sCmluZGV4IDI3NjE0OGQ0MTFjMTk1NWIxMGVlYTlmNDBhZmVkNjM5YjMzYzM0NDAuLjNl
YWRhMzAyMWY3NzgxNDFiZTE4MTcyNmUwMjU5MjJlZmI1YzVhMzggMTAwNjQ0Ci0tLSBhL0xheW91
dFRlc3RzL2Zhc3QvZnJhbWVzL3VubG9hZC1yZXBhcmVudC1zaWJsaW5nLWZyYW1lLmh0bWwKKysr
IGIvTGF5b3V0VGVzdHMvZmFzdC9mcmFtZXMvdW5sb2FkLXJlcGFyZW50LXNpYmxpbmctZnJhbWUu
aHRtbApAQCAtNCw0OCArNCw1MCBAQAogCiA8ZGl2IGlkPSJjb250YWluZXIiPgogICAgIDxkaXYg
aWQ9Imluc2lkZSI+CisgICAgICAgIDxpZnJhbWUgaWQ9ImZyYW1lMSIgc3JjZG9jPSJmcmFtZTEi
PjwvaWZyYW1lPgorICAgICAgICA8aWZyYW1lIGlkPSJmcmFtZTIiIHNyY2RvYz0iZnJhbWUyIj48
L2lmcmFtZT4KICAgICA8L2Rpdj4KIDwvZGl2PgogCiA8c2NyaXB0PgogZGVzY3JpcHRpb24oJ1Jl
cGFyZW50ZWQgc2libGluZyBmcmFtZXMgZnJvbSB1bmxvYWQgaGFuZGxlcnMgc2hvdWxkIGxvYWQu
Jyk7CiAKK3ZhciBqc1Rlc3RJc0FzeW5jID0gdHJ1ZTsKK3ZhciBjb3VudCA9IDI7Cit2YXIgY29u
dGFpbmVyID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2NvbnRhaW5lcicpOwordmFyIGluc2lk
ZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdpbnNpZGUnKTsKK3ZhciBmcmFtZTEgPSBkb2N1
bWVudC5nZXRFbGVtZW50QnlJZCgnZnJhbWUxJyk7Cit2YXIgZnJhbWUyID0gZG9jdW1lbnQuZ2V0
RWxlbWVudEJ5SWQoJ2ZyYW1lMicpOworCiBpZiAod2luZG93LnRlc3RSdW5uZXIpIHsKICAgICB0
ZXN0UnVubmVyLmR1bXBBc1RleHQoKTsKICAgICB0ZXN0UnVubmVyLmR1bXBDaGlsZEZyYW1lc0Fz
VGV4dCgpOwogfQogCi1mdW5jdGlvbiBvbmZyYW1lbG9hZCh0ZXh0KQotewotICAgIHRoaXMuY29u
dGVudERvY3VtZW50LmJvZHkuaW5uZXJIVE1MID0gdGhpcy5pZDsKLX0KLQogZnVuY3Rpb24gcnVu
VGVzdCgpCi17ICAgIAoreworICAgIGlmICgtLWNvdW50KSByZXR1cm47CisgICAgLy8gV2FpdCB1
bnRpbCBhbGwgZnJhbWUgZGlzY29ubmVjdGlvbiBjYWxscyBhcmUgY29tcGxldGUgdG8gZW5zdXJl
IGluRG9jdW1lbnQKKyAgICAvLyBmcmFtZXMgcmVtYWluIGxvYWRlZC4KKyAgICBzZXRUaW1lb3V0
KGZ1bmN0aW9uKCkgeworICAgICAgICBzaG91bGROb3RCZSgnZnJhbWUxLmNvbnRlbnREb2N1bWVu
dCcsICdudWxsJyk7CisgICAgICAgIHNob3VsZE5vdEJlKCdmcmFtZTIuY29udGVudERvY3VtZW50
JywgJ251bGwnKTsKKyAgICAgICAgZmluaXNoSlNUZXN0KCk7CisgICAgfSwgMCk7CiB9CiAKLXZh
ciBjb250YWluZXIgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnY29udGFpbmVyJyk7Ci12YXIg
aW5zaWRlID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2luc2lkZScpOwotCi12YXIgZnJhbWUx
ID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgnaWZyYW1lJyk7Ci1mcmFtZTEuaWQgPSAnZnJhbWUx
JzsKLWZyYW1lMS5vbmxvYWQgPSBvbmZyYW1lbG9hZDsKLQotdmFyIGZyYW1lMiA9IGRvY3VtZW50
LmNyZWF0ZUVsZW1lbnQoJ2lmcmFtZScpOwotZnJhbWUyLmlkID0gJ2ZyYW1lMic7Ci1mcmFtZTIu
b25sb2FkID0gb25mcmFtZWxvYWQ7Ci0KIG9ubG9hZCA9IGZ1bmN0aW9uKCkgewogICAgIGluc2lk
ZS5hcHBlbmRDaGlsZChmcmFtZTEpOwogICAgIGluc2lkZS5hcHBlbmRDaGlsZChmcmFtZTIpOwog
CiAgICAgZnJhbWUxLmNvbnRlbnRXaW5kb3cub251bmxvYWQgPSBmdW5jdGlvbigpIHsKKyAgICAg
ICAgZnJhbWUxLm9ubG9hZCA9IHJ1blRlc3Q7CisgICAgICAgIGZyYW1lMi5vbmxvYWQgPSBydW5U
ZXN0OwogICAgICAgICBkb2N1bWVudC5ib2R5LmFwcGVuZENoaWxkKGluc2lkZSk7CiAgICAgfTsK
IAogICAgIGNvbnRhaW5lci5wYXJlbnROb2RlLnJlbW92ZUNoaWxkKGNvbnRhaW5lcik7Ci0gICAg
c2hvdWxkTm90QmUoJ2ZyYW1lMS5jb250ZW50RG9jdW1lbnQnLCAnbnVsbCcpOwotICAgIHNob3Vs
ZE5vdEJlKCdmcmFtZTIuY29udGVudERvY3VtZW50JywgJ251bGwnKTsKLSAgICBpc1N1Y2Nlc3Nm
dWxseVBhcnNlZCgpOwogfTsKIDwvc2NyaXB0PgorCis8c2NyaXB0IHNyYz0iLi4vanMvcmVzb3Vy
Y2VzL2pzLXRlc3QtcG9zdC5qcyI+PC9zY3JpcHQ+Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>