<?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>55198</bug_id>
          
          <creation_ts>2011-02-24 17:01:24 -0800</creation_ts>
          <short_desc>LayoutTests/http/tests/media/video-play-suspend is now hanging on chromium</short_desc>
          <delta_ts>2011-03-01 11:53:50 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>0</everconfirmed>
          <reporter name="Steve Lacey">sjl</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>eric.carlson</cc>
    
    <cc>jamesr</cc>
    
    <cc>jer.noble</cc>
    
    <cc>scherkus</cc>
    
    <cc>tonyg</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>357584</commentid>
    <comment_count>0</comment_count>
    <who name="Steve Lacey">sjl</who>
    <bug_when>2011-02-24 17:01:24 -0800</bug_when>
    <thetext>This just started hanging for chrome due to a cl of mine (http://codereview.chromium.org/6581012/) which causes the MediaPlayer states to transition from &apos;loading -&gt; loaded -&gt;idle&apos; instead of &apos;loading -&gt; idle&apos;. This appears to be the correct spec thing to do as a progress event is now fired when the media has completed loading (which it wasn&apos;t before), but a side effect is that the &apos;suspend&apos; event is now not being fired as the network doesn&apos;t actually suspend (indicated by a &apos;loading -&gt; idle&apos; transition).

I ran the test under safari and the test also hangs there too.

So it seems to me that the test is actually not testing for correct behaviour.

Thoughts?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>357674</commentid>
    <comment_count>1</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-02-24 21:11:23 -0800</bug_when>
    <thetext>(In reply to comment #0)
&gt; This just started hanging for chrome due to a cl of mine (http://codereview.chromium.org/6581012/) which causes the MediaPlayer states to transition from &apos;loading -&gt; loaded -&gt;idle&apos; instead of &apos;loading -&gt; idle&apos;. This appears to be the correct spec thing to do as a progress event is now fired when the media has completed loading (which it wasn&apos;t before), but a side effect is that the &apos;suspend&apos; event is now not being fired as the network doesn&apos;t actually suspend (indicated by a &apos;loading -&gt; idle&apos; transition).
&gt; 
&gt; I ran the test under safari and the test also hangs there too.
&gt; 
&gt; So it seems to me that the test is actually not testing for correct behaviour.
&gt; 
&gt; Thoughts?

&apos;suspend&apos; is to be fired when the UA stops downloading for some reason:

    When a media element&apos;s download has been suspended, the user agent must set the 
    networkState to NETWORK_IDLE and queue a task to fire a simple event named suspend 
    at the element.

so HTMLMediaElement fires it when the media engine signals a network state change from &apos;loading&apos; or &apos;loaded&apos; to MediaPlayer::Idle.  

I think the problem is in HTMLMediaElement::setNetworkState, where MediaPlayer::Loaded results in m_networkState being set to NETWORK_IDLE without firing a &apos;suspended&apos; event. I think it should fire a &apos;suspend&apos; event just before the final &apos;progress&apos; event, though we will have to see if it causes other problems.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>357968</commentid>
    <comment_count>2</comment_count>
    <who name="Steve Lacey">sjl</who>
    <bug_when>2011-02-25 09:30:56 -0800</bug_when>
    <thetext>(In reply to comment #1)
&gt; (In reply to comment #0)
&gt; &gt; This just started hanging for chrome due to a cl of mine (http://codereview.chromium.org/6581012/) which causes the MediaPlayer states to transition from &apos;loading -&gt; loaded -&gt;idle&apos; instead of &apos;loading -&gt; idle&apos;. This appears to be the correct spec thing to do as a progress event is now fired when the media has completed loading (which it wasn&apos;t before), but a side effect is that the &apos;suspend&apos; event is now not being fired as the network doesn&apos;t actually suspend (indicated by a &apos;loading -&gt; idle&apos; transition).
&gt; &gt; 
&gt; &gt; I ran the test under safari and the test also hangs there too.
&gt; &gt; 
&gt; &gt; So it seems to me that the test is actually not testing for correct behaviour.
&gt; &gt; 
&gt; &gt; Thoughts?
&gt; 
&gt; &apos;suspend&apos; is to be fired when the UA stops downloading for some reason:
&gt; 
&gt;     When a media element&apos;s download has been suspended, the user agent must set the 
&gt;     networkState to NETWORK_IDLE and queue a task to fire a simple event named suspend 
&gt;     at the element.
&gt; 
&gt; so HTMLMediaElement fires it when the media engine signals a network state change from &apos;loading&apos; or &apos;loaded&apos; to MediaPlayer::Idle.  
&gt; 
&gt; I think the problem is in HTMLMediaElement::setNetworkState, where MediaPlayer::Loaded results in m_networkState being set to NETWORK_IDLE without firing a &apos;suspended&apos; event. I think it should fire a &apos;suspend&apos; event just before the final &apos;progress&apos; event, though we will have to see if it causes other problems.

I also see this in the spec for the suspend event:

      The user agent is intentionally not currently fetching media data, but does not have the entire
      media resource downloaded.

Which implies to me that you get a suspend event if download stops but there&apos;s more data to download.

The sentences before and after the paragraph you list also imply (to me) that if download is suspended but there&apos;s more to download, that&apos;s when you get a suspend event. There&apos;s no wordage that implies you get a suspend event in any other situation...

Confusion (in my mind at least :-) abounds.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>357973</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-02-25 09:35:44 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; (In reply to comment #1)
&gt; 
&gt; I also see this in the spec for the suspend event:
&gt; 
&gt;       The user agent is intentionally not currently fetching media data, but does not have the entire
&gt;       media resource downloaded.
&gt; 
&gt; Which implies to me that you get a suspend event if download stops but there&apos;s more data to download.

  Indeed, it does say that in the summary. I guess I must have read the spec more carefully when I wrote setNetworkState - good thing! 

&gt; 
&gt; Confusion (in my mind at least :-) abounds.

So propose a fix.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>358015</commentid>
    <comment_count>4</comment_count>
    <who name="Steve Lacey">sjl</who>
    <bug_when>2011-02-25 10:31:46 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; (In reply to comment #1)
&gt; &gt; 
&gt; &gt; I also see this in the spec for the suspend event:
&gt; &gt; 
&gt; &gt;       The user agent is intentionally not currently fetching media data, but does not have the entire
&gt; &gt;       media resource downloaded.
&gt; &gt; 
&gt; &gt; Which implies to me that you get a suspend event if download stops but there&apos;s more data to download.
&gt; 
&gt;   Indeed, it does say that in the summary. I guess I must have read the spec more carefully when I wrote setNetworkState - good thing! 
&gt; 
&gt; &gt; 
&gt; &gt; Confusion (in my mind at least :-) abounds.
&gt; 
&gt; So propose a fix.

(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; (In reply to comment #1)
&gt; &gt; 
&gt; &gt; I also see this in the spec for the suspend event:
&gt; &gt; 
&gt; &gt;       The user agent is intentionally not currently fetching media data, but does not have the entire
&gt; &gt;       media resource downloaded.
&gt; &gt; 
&gt; &gt; Which implies to me that you get a suspend event if download stops but there&apos;s more data to download.
&gt; 
&gt;   Indeed, it does say that in the summary. I guess I must have read the spec more carefully when I wrote setNetworkState - good thing! 
&gt; 
&gt; &gt; 
&gt; &gt; Confusion (in my mind at least :-) abounds.
&gt; 
&gt; So propose a fix.

My suggestion would be to create a new layout test basically the same as video-play-suspend, but called video-play-progress and does the same thing but expects a progress event.

I&apos;ll start working on this.

Then mod video-play-suspend to cause a stall and expect a suspend event. I&apos;m not sure how to do this or do it in a non-flakey manner as it would rely on the networking internals of the port.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359377</commentid>
    <comment_count>5</comment_count>
      <attachid>84127</attachid>
    <who name="Steve Lacey">sjl</who>
    <bug_when>2011-02-28 14:49:08 -0800</bug_when>
    <thetext>Created attachment 84127
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359383</commentid>
    <comment_count>6</comment_count>
    <who name="Steve Lacey">sjl</who>
    <bug_when>2011-02-28 14:52:03 -0800</bug_when>
    <thetext>The attached patch:

- removes video-play-suspend as it was basically an incorrect test. I don&apos;t believe it&apos;s possible to test this deterministically as it depends on port player internals
- introduces video-play-progress that tests that at least one progress event is fired. Ideally we&apos;d test that progress events are fired every 350ms (flaky...) and that a final progress event is fired after all media is loaded (but it&apos;s unclear how to determine this state).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359952</commentid>
    <comment_count>7</comment_count>
      <attachid>84127</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-03-01 09:39:57 -0800</bug_when>
    <thetext>Comment on attachment 84127
Patch

Will you also please add comments to bugs 37034, 37035, and 37036 to clarify what they should do since the description talks about the &apos;suspend&apos; event?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359958</commentid>
    <comment_count>8</comment_count>
      <attachid>84127</attachid>
    <who name="Steve Lacey">sjl</who>
    <bug_when>2011-03-01 09:45:17 -0800</bug_when>
    <thetext>Comment on attachment 84127
Patch

Will comment on those bugs when this goes in.

Could I get a cq+? Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359969</commentid>
    <comment_count>9</comment_count>
      <attachid>84127</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-03-01 09:56:22 -0800</bug_when>
    <thetext>Comment on attachment 84127
Patch

FYI - if you set cq? when requesting a review, the reviewer will generally cq+ when marking r+.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360071</commentid>
    <comment_count>10</comment_count>
      <attachid>84127</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-03-01 11:53:45 -0800</bug_when>
    <thetext>Comment on attachment 84127
Patch

Clearing flags on attachment: 84127

Committed r80017: &lt;http://trac.webkit.org/changeset/80017&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360073</commentid>
    <comment_count>11</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-03-01 11:53:50 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>84127</attachid>
            <date>2011-02-28 14:49:08 -0800</date>
            <delta_ts>2011-03-01 11:53:44 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-55198-20110228144907.patch</filename>
            <type>text/plain</type>
            <size>7207</size>
            <attacher name="Steve Lacey">sjl</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogNzk5MjEKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5n
ZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwppbmRleCA3MjRlN2RkMDMyNmU1NDhjODcxNzlh
Njk3ZjRiYThiMDU1OTJmODFmLi44Y2RmZDFiOGY3ZTgwZWVhZDI3MTM3NzE2MjJiNTljY2I2MDQy
YjI2IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMv
Q2hhbmdlTG9nCkBAIC0xLDMgKzEsMjAgQEAKKzIwMTEtMDItMjggIFN0ZXZlIExhY2V5ICA8c2ps
QGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBDcmVhdGUgbGF5b3V0IHRlc3QgZm9yIHByb2dyZXNzIGV2ZW50IGFuZCByZW1vdmUg
dGVzdCBmb3Igc3VzcGVuZCBldmVudC4KKworICAgICAgICBMYXlvdXRUZXN0cy9odHRwL3Rlc3Rz
L21lZGlhL3ZpZGVvLXBsYXktc3VzcGVuZCBpcyBub3cgaGFuZ2luZyBvbiBjaHJvbWl1bQorICAg
ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTUxOTgKKworICAg
ICAgICAqIGh0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1wcm9ncmVzcy1leHBlY3RlZC50eHQ6
IEFkZGVkLgorICAgICAgICAqIGh0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1wcm9ncmVzcy5o
dG1sOiBBZGRlZC4KKyAgICAgICAgKiBodHRwL3Rlc3RzL21lZGlhL3ZpZGVvLXBsYXktc3VzcGVu
ZC1leHBlY3RlZC50eHQ6IFJlbW92ZWQuCisgICAgICAgICogaHR0cC90ZXN0cy9tZWRpYS92aWRl
by1wbGF5LXN1c3BlbmQuaHRtbDogUmVtb3ZlZC4KKyAgICAgICAgKiBwbGF0Zm9ybS9ndGsvU2tp
cHBlZDoKKyAgICAgICAgKiBwbGF0Zm9ybS9tYWMvU2tpcHBlZDoKKyAgICAgICAgKiBwbGF0Zm9y
bS93aW4vU2tpcHBlZDoKKwogMjAxMS0wMi0yOCAgSnVuZ3NoaWsgU2hpbiAgPGpzaGluQGNocm9t
aXVtLm9yZz4KIAogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NTUxMjEKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVkaWEvdmlkZW8t
cGxheS1wcm9ncmVzcy1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL21lZGlh
L3ZpZGVvLXBsYXktcHJvZ3Jlc3MtZXhwZWN0ZWQudHh0Cm5ldyBmaWxlIG1vZGUgMTAwNjQ0Cmlu
ZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLjljZDEyMWE5N2Vh
ZTAyYWZlNjgxN2M0Y2RjZDMzNTNhNTE2YWFjN2UKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRU
ZXN0cy9odHRwL3Rlc3RzL21lZGlhL3ZpZGVvLXBsYXktcHJvZ3Jlc3MtZXhwZWN0ZWQudHh0CkBA
IC0wLDAgKzEsNyBAQAorVGVzdCB0aGF0IGF0IGxlYXN0IG9uZSBwcm9ncmVzcyBldmVudCBpcyBm
aXJlZCBhZnRlciBzdGFydGluZyB0byBsb2FkIHRoZSB2aWRlby4KKworUlVOKHZpZGVvLnBsYXko
KSkKK0VWRU5UKGxvYWRzdGFydCkKK0VWRU5UKHByb2dyZXNzKQorRU5EIE9GIFRFU1QKKwpkaWZm
IC0tZ2l0IGEvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9tZWRpYS92aWRlby1wbGF5LXByb2dyZXNz
Lmh0bWwgYi9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL21lZGlhL3ZpZGVvLXBsYXktcHJvZ3Jlc3Mu
aHRtbApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw
MDAwMDAwMDAwMDAwMDAwLi5kNDRkZWU0NWQ0NTA2ZGRhZTUzNTg5NWNiNjcxNGQ2MzAwNDExMWIw
Ci0tLSAvZGV2L251bGwKKysrIGIvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9tZWRpYS92aWRlby1w
bGF5LXByb2dyZXNzLmh0bWwKQEAgLTAsMCArMSwyNiBAQAorPGh0bWw+CisgIDxoZWFkPgorICAg
ICAgPHNjcmlwdCBzcmM9Li4vLi4vbWVkaWEtcmVzb3VyY2VzL21lZGlhLWZpbGUuanM+PC9zY3Jp
cHQ+CisgICAgICA8c2NyaXB0IHNyYz0uLi8uLi9tZWRpYS1yZXNvdXJjZXMvdmlkZW8tdGVzdC5q
cz48L3NjcmlwdD4KKyAgICAgIDxzY3JpcHQ+CisgICAgICAgICAgZnVuY3Rpb24gc3RhcnQoKSB7
CisgICAgICAgICAgICAgIGZpbmRNZWRpYUVsZW1lbnQoKTsKKworICAgICAgICAgICAgICB2YXIg
cHJvZ3Jlc3NDb3VudCA9IDA7CisKKyAgICAgICAgICAgICAgLy8gTWFrZSBzdXJlIHdlJ3ZlIGF0
IGxlYXN0IHJlYWNoZWQgTkVUV09SS19MT0FESU5HIGJlZm9yZSB3YWl0aW5nIGZvciBwcm9ncmVz
cy4KKyAgICAgICAgICAgICAgd2FpdEZvckV2ZW50KCdsb2Fkc3RhcnQnLCBmdW5jdGlvbigpIHsK
KyAgICAgICAgICAgICAgICAgIHdhaXRGb3JFdmVudEFuZEVuZCgncHJvZ3Jlc3MnKQorICAgICAg
ICAgICAgICB9ICk7CisKKyAgICAgICAgICAgICAgdmFyIG1lZGlhRmlsZSA9IGZpbmRNZWRpYUZp
bGUoInZpZGVvIiwgInJlc291cmNlcy90ZXN0Iik7CisgICAgICAgICAgICAgIHZpZGVvLnNyYyA9
ICJodHRwOi8vMTI3LjAuMC4xOjgwMDAvIiArIG1lZGlhRmlsZTsKKyAgICAgICAgICAgICAgcnVu
KCJ2aWRlby5wbGF5KCkiKTsKKyAgICAgICAgICB9CisgICAgICA8L3NjcmlwdD4KKyAgICA8L2hl
YWQ+CisgICAgPGJvZHkgb25sb2FkPSJzdGFydCgpIj4KKyAgICAgIDx2aWRlbz48L3ZpZGVvPgor
ICAgICAgPHA+VGVzdCB0aGF0IGF0IGxlYXN0IG9uZSBwcm9ncmVzcyBldmVudCBpcyBmaXJlZCBh
ZnRlciBzdGFydGluZyB0byBsb2FkIHRoZSB2aWRlby48L3A+CisgICAgPC9ib2R5PgorPC9odG1s
PgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9tZWRpYS92aWRlby1wbGF5LXN1
c3BlbmQtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9tZWRpYS92aWRlby1w
bGF5LXN1c3BlbmQtZXhwZWN0ZWQudHh0CmRlbGV0ZWQgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAw
ODJlMmYzNzQwNDgzNTUxMzE4YjgzYjg0MzFkNjJmYWFkMGVmZmJiLi4wMDAwMDAwMDAwMDAwMDAw
MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwCi0tLSBhL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVk
aWEvdmlkZW8tcGxheS1zdXNwZW5kLWV4cGVjdGVkLnR4dAorKysgL2Rldi9udWxsCkBAIC0xLDgg
KzAsMCBAQAotVGVzdCB0aGF0IHRpbWV1cGRhdGUgZXZlbnRzIGFyZSBzZW50IHdoZW4gbWVkaWEg
bG9hZGluZyBzdXNwZW5kcyBpdHNlbGYuCi0KLVJVTih2aWRlby5wbGF5KCkpCi1FVkVOVChsb2Fk
c3RhcnQpCi1FVkVOVChzdXNwZW5kKQotRVZFTlQodGltZXVwZGF0ZSkKLUVORCBPRiBURVNUCi0K
ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1zdXNw
ZW5kLmh0bWwgYi9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL21lZGlhL3ZpZGVvLXBsYXktc3VzcGVu
ZC5odG1sCmRlbGV0ZWQgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwOTJlOTA3ODk1ZDZlY2I0MDg4
MTMyMmQzYmU1Mjk3ZTYxNTljMGY1Li4wMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw
MDAwMDAwCi0tLSBhL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1zdXNw
ZW5kLmh0bWwKKysrIC9kZXYvbnVsbApAQCAtMSwzNiArMCwwIEBACi08aHRtbD4KLSAgPGhlYWQ+
Ci0gICAgICA8c2NyaXB0IHNyYz0uLi8uLi9tZWRpYS1yZXNvdXJjZXMvbWVkaWEtZmlsZS5qcz48
L3NjcmlwdD4KLSAgICAgIDxzY3JpcHQgc3JjPS4uLy4uL21lZGlhLXJlc291cmNlcy92aWRlby10
ZXN0LmpzPjwvc2NyaXB0PgotICAgICAgPHNjcmlwdD4KLSAgICAgICAgICBmdW5jdGlvbiBzdGFy
dCgpIHsKLSAgICAgICAgICAgICAgZmluZE1lZGlhRWxlbWVudCgpOwotCi0gICAgICAgICAgICAg
IHZhciBzdXNwZW5kQ291bnQgPSAwOwotCi0gICAgICAgICAgICAgIC8vIE1ha2Ugc3VyZSB3ZSd2
ZSBhdCBsZWFzdCByZWFjaGVkIE5FVFdPUktfTE9BRElORyBiZWZvcmUgd2FpdGluZyBmb3Igc3Vz
cGVuZC4KLSAgICAgICAgICAgICAgd2FpdEZvckV2ZW50KCdsb2Fkc3RhcnQnLCBmdW5jdGlvbigp
IHsKLSAgICAgICAgICAgICAgICAgIC8vIHN1c3BlbmQgZXZlbnQgbWVhbnMgdGhlIHVzZXIgYWdl
bnQgaGFzIGludGVudGlvbmFsbHkgcGF1c2VkIG5ldHdvcmsgdXNhZ2UsCi0gICAgICAgICAgICAg
ICAgICAvLyBob3dldmVyIHdlIHNob3VsZCBzdGlsbCByZWNlaXZlIHRpbWV1cGRhdGUgZXZlbnRz
LgotICAgICAgICAgICAgICAgICAgbWVkaWFFbGVtZW50LmFkZEV2ZW50TGlzdGVuZXIoJ3N1c3Bl
bmQnLCBmdW5jdGlvbiAoKSB7Ci0gICAgICAgICAgICAgICAgICAgICAgLy8gdGltZXVwZGF0ZSBl
dmVudHMgYXJlIGZpcmVkIGFzIHBsYXliYWNrIHByb2dyZXNzZXMgc28gb25seSB2ZXJpZnkgdGhh
dCBhdCBsZWFzdCBvbmUKLSAgICAgICAgICAgICAgICAgICAgICAvLyBldmVudCBpcyBmaXJlZAot
ICAgICAgICAgICAgICAgICAgICAgICsrc3VzcGVuZENvdW50OwotICAgICAgICAgICAgICAgICAg
ICAgIGlmIChzdXNwZW5kQ291bnQgPT0gMSkgewotICAgICAgICAgICAgICAgICAgICAgICAgICBj
b25zb2xlV3JpdGUoIkVWRU5UKHN1c3BlbmQpIik7Ci0gICAgICAgICAgICAgICAgICAgICAgICAg
IHdhaXRGb3JFdmVudEFuZEVuZCgndGltZXVwZGF0ZScpOwotICAgICAgICAgICAgICAgICAgICAg
IH0KLSAgICAgICAgICAgICAgICAgIH0gKTsKLSAgICAgICAgICAgICAgfSApOwotCi0gICAgICAg
ICAgICAgIHZhciBtZWRpYUZpbGUgPSBmaW5kTWVkaWFGaWxlKCJ2aWRlbyIsICJyZXNvdXJjZXMv
dGVzdCIpOwotICAgICAgICAgICAgICB2aWRlby5zcmMgPSAiaHR0cDovLzEyNy4wLjAuMTo4MDAw
LyIgKyBtZWRpYUZpbGU7Ci0gICAgICAgICAgICAgIHJ1bigidmlkZW8ucGxheSgpIik7Ci0gICAg
ICAgICAgfQotICAgICAgPC9zY3JpcHQ+Ci0gICAgPC9oZWFkPgotICAgIDxib2R5IG9ubG9hZD0i
c3RhcnQoKSI+Ci0gICAgICA8dmlkZW8+PC92aWRlbz4KLSAgICAgIDxwPlRlc3QgdGhhdCB0aW1l
dXBkYXRlIGV2ZW50cyBhcmUgc2VudCB3aGVuIG1lZGlhIGxvYWRpbmcgc3VzcGVuZHMgaXRzZWxm
LjwvcD4KLSAgICA8L2JvZHk+Ci08L2h0bWw+CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0
Zm9ybS9ndGsvU2tpcHBlZCBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2d0ay9Ta2lwcGVkCmluZGV4
IGFmMDM3NmI1Mjk3MGUwZTQxZTUzMTRlZTA2ZDI3MTJlY2IzZmZiNmYuLjhmMTliYWYxYWU1MDUw
NjRmNmRlMWY3YzQ2ZDBiYjJiZjZiNGY2MWEgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL3BsYXRm
b3JtL2d0ay9Ta2lwcGVkCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2d0ay9Ta2lwcGVkCkBA
IC03NzEsMTAgKzc3MSw2IEBAIGZhc3QvbG9hZGVyL3N0YXRlb2JqZWN0cy9yZXBsYWNlc3RhdGUt
aW4taWZyYW1lLmh0bWwKICMgZm9yIGEgcmVhbCBmaXguCiBlZGl0aW5nL3NlbGVjdGlvbi9leHRl
bmQtYWZ0ZXItbW91c2Utc2VsZWN0aW9uLmh0bWwKIAotIyBNZWRpYVBsYXllclByaXZhdGVHU3Ry
ZWFtZXIgZG9lc24ndCByZXR1cm4gdG8gTkVUV09SS19JRExFIGFuZCBoZW5jZSBkb2Vzbid0IGZp
cmUgYSBzdXNwZW5kIGV2ZW50Ci0jIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD0zNzAzNAotaHR0cC90ZXN0cy9tZWRpYS92aWRlby1wbGF5LXN1c3BlbmQuaHRtbAotCiAj
IE5lZWQgdG8gYWRkIGZ1bmN0aW9uYWxpdHkgdG8gRHVtcFJlbmRlclRyZWUgdG8gaGFuZGxlIHNj
cm9sbGJhciBwb2xpY2UgY2hhbmdlcwogZmFzdC9vdmVyZmxvdy9zY3JvbGxiYXItcmVzdG9yZWQt
YW5kLXRoZW4tbG9ja2VkLmh0bWwKIApkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0v
bWFjL1NraXBwZWQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvU2tpcHBlZAppbmRleCA0ODc4
ZjYxMjViODlhYzZlMTU3ZWYyZWFlNjlkZDNlYmM0MWYzMGZjLi4zODJiMjU4ZTNmZGY4YjliNjU3
ZmFkMjhhZmRhNzFiN2QzOTJkMTQ2IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9t
YWMvU2tpcHBlZAorKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvU2tpcHBlZApAQCAtNDQs
MTAgKzQ0LDYgQEAgZmFzdC9yZXBhaW50L2JvZHktYmFja2dyb3VuZC1pbWFnZS5odG1sCiAjIDxy
ZGFyOi8vcHJvYmxlbS82NzEwNjI1PiBUZXN0IG1lZGlhL3ZpZGVvLWVycm9yLWFib3J0Lmh0bWwg
ZG9lc24ndCB3b3JrCiBodHRwL3Rlc3RzL21lZGlhL3ZpZGVvLWVycm9yLWFib3J0Lmh0bWwKIAot
IyBNZWRpYVBsYXllclByaXZhdGVRVEtpdCBkb2Vzbid0IHJldHVybiB0byBORVRXT1JLX0lETEUg
YW5kIGhlbmNlIGRvZXNuJ3QgZmlyZSBhIHN1c3BlbmQgZXZlbnQKLSMgaHR0cHM6Ly9idWdzLndl
YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM3MDM2Ci1odHRwL3Rlc3RzL21lZGlhL3ZpZGVvLXBs
YXktc3VzcGVuZC5odG1sCi0KICMgVGhpcyBwb3J0IGRvZXNuJ3Qgc3VwcG9ydCBOb3RpZmljYXRp
b25zLgogZmFzdC9ub3RpZmljYXRpb25zCiBodHRwL3Rlc3RzL25vdGlmaWNhdGlvbnMKZGlmZiAt
LWdpdCBhL0xheW91dFRlc3RzL3BsYXRmb3JtL3dpbi9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxh
dGZvcm0vd2luL1NraXBwZWQKaW5kZXggM2ZlNTVmOTgxNzM5ZmUxNTUzNDBlY2M4OGRiM2Y5YzVi
YzE2MDNkYS4uNTA0NGRkOTA3Y2RmN2QxODM4ODkwNDEzZTEzODQxMjZlMmUxMzgyNCAxMDA2NDQK
LS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vd2luL1NraXBwZWQKKysrIGIvTGF5b3V0VGVzdHMv
cGxhdGZvcm0vd2luL1NraXBwZWQKQEAgLTc0NiwxMCArNzQ2LDYgQEAgZmFzdC9kb20vR2VvbG9j
YXRpb24KIGZhc3Qvbm90aWZpY2F0aW9ucwogaHR0cC90ZXN0cy9ub3RpZmljYXRpb25zCiAKLSMg
TWVkaWFQbGF5ZXJQcml2YXRlUXVpY2tUaW1lV2luIGRvZXNuJ3QgcmV0dXJuIHRvIE5FVFdPUktf
SURMRSBhbmQgaGVuY2UgZG9lc24ndCBmaXJlIGEgc3VzcGVuZCBldmVudAotIyBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzcwMzUKLWh0dHAvdGVzdHMvbWVkaWEvdmlk
ZW8tcGxheS1zdXNwZW5kLmh0bWwKLQogIyBUaGVzZSBtZWRpYSB0ZXN0cyByZWdyZXNzZWQgb24g
dGhlIGJvdHMuICBTa2lwcGluZyB0byBtYWtlIHRoZSBib3RzIGdyZWVuIGFnYWluLgogIyBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjgyODEKIGh0dHAvdGVzdHMvbWVk
aWEvdmlkZW8tcGxheS1zdGFsbC5odG1sCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>