<?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>205925</bug_id>
          
          <creation_ts>2020-01-08 06:57:53 -0800</creation_ts>
          <short_desc>[Web Animations] Stop creating CSS Animations for &lt;noscript&gt; elements</short_desc>
          <delta_ts>2020-01-08 08:30: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>CSS</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=205929</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=205928</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Antoine Quint">graouts</reporter>
          <assigned_to name="Antoine Quint">graouts</assigned_to>
          <cc>cdumez</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>dbates</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>kangil.han</cc>
    
    <cc>koivisto</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1604075</commentid>
    <comment_count>0</comment_count>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2020-01-08 06:57:53 -0800</bug_when>
    <thetext>[Web Animations] Stop creating CSS Animations for &lt;noscript&gt; elements</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604077</commentid>
    <comment_count>1</comment_count>
      <attachid>387103</attachid>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2020-01-08 07:02:15 -0800</bug_when>
    <thetext>Created attachment 387103
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604078</commentid>
    <comment_count>2</comment_count>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2020-01-08 07:02:20 -0800</bug_when>
    <thetext>&lt;rdar://problem/58158479&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604089</commentid>
    <comment_count>3</comment_count>
      <attachid>387103</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2020-01-08 07:41:23 -0800</bug_when>
    <thetext>Comment on attachment 387103
Patch

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

&gt; Source/WebCore/ChangeLog:16
&gt; +        It makes no sense to create CSS Animations for a &lt;noscript&gt; element and it has the side effect of potential crashes.
&gt; +        Indeed, AnimationTimeline::updateCSSAnimationsForElement() may be called without a currentStyle and so we never have
&gt; +        a list of previously-applied animations to compare to the list of animations in afterChangeStyle. So on each call we
&gt; +        end up creating a new CSSAnimation and the previous animation for the same name is never explicitly removed from the
&gt; +        effect stack and is eventually destroyed and the WeakPtr for it in the stack ends up being null, which would cause a
&gt; +        crash under KeyframeEffectStack::ensureEffectsAreSorted().

You&apos;ll should add some other defense against this in animation code too as this patch likely doesn&apos;t cover all the paths where this can happen.

&gt; Source/WebCore/html/HTMLElement.cpp:736
&gt; -bool HTMLElement::rendererIsNeeded(const RenderStyle&amp; style)
&gt; +bool HTMLElement::rendererIsEverNeeded()

You can also switch HTMLInputElement::rendererIsNeeded to rendererIsEverNeeded, maybe others. Please check, they can probably all be used to get this crash.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604096</commentid>
    <comment_count>4</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2020-01-08 08:04:24 -0800</bug_when>
    <thetext>In fact most rendererIsNeeded functions can be turned into rendererIsEverNeeded. 

Doesn&apos;t need to be done in this patch though.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604100</commentid>
    <comment_count>5</comment_count>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2020-01-08 08:27:49 -0800</bug_when>
    <thetext>(In reply to Antti Koivisto from comment #3)
&gt; Comment on attachment 387103 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=387103&amp;action=review
&gt; 
&gt; &gt; Source/WebCore/ChangeLog:16
&gt; &gt; +        It makes no sense to create CSS Animations for a &lt;noscript&gt; element and it has the side effect of potential crashes.
&gt; &gt; +        Indeed, AnimationTimeline::updateCSSAnimationsForElement() may be called without a currentStyle and so we never have
&gt; &gt; +        a list of previously-applied animations to compare to the list of animations in afterChangeStyle. So on each call we
&gt; &gt; +        end up creating a new CSSAnimation and the previous animation for the same name is never explicitly removed from the
&gt; &gt; +        effect stack and is eventually destroyed and the WeakPtr for it in the stack ends up being null, which would cause a
&gt; &gt; +        crash under KeyframeEffectStack::ensureEffectsAreSorted().
&gt; 
&gt; You&apos;ll should add some other defense against this in animation code too as
&gt; this patch likely doesn&apos;t cover all the paths where this can happen.

It&apos;s more than a defense that is needed, the problem is that `AnimationTimeline` doesn&apos;t know to expect several CSS Animations with the same animation-name. I will address this as part of a different bug as soon as possible. I raised https://bugs.webkit.org/show_bug.cgi?id=205928.

&gt; &gt; Source/WebCore/html/HTMLElement.cpp:736
&gt; &gt; -bool HTMLElement::rendererIsNeeded(const RenderStyle&amp; style)
&gt; &gt; +bool HTMLElement::rendererIsEverNeeded()
&gt; 
&gt; You can also switch HTMLInputElement::rendererIsNeeded to
&gt; rendererIsEverNeeded, maybe others. Please check, they can probably all be
&gt; used to get this crash.

I will add more rendererIsNeeded overrides in a followup, I raised</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604101</commentid>
    <comment_count>6</comment_count>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2020-01-08 08:28:20 -0800</bug_when>
    <thetext>(In reply to Antoine Quint from comment #5)
&gt; I will add more rendererIsNeeded overrides in a followup, I raised

… https://bugs.webkit.org/show_bug.cgi?id=205929</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1604102</commentid>
    <comment_count>7</comment_count>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2020-01-08 08:30:49 -0800</bug_when>
    <thetext>Committed r254201: &lt;https://trac.webkit.org/changeset/254201&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>387103</attachid>
            <date>2020-01-08 07:02:15 -0800</date>
            <delta_ts>2020-01-08 07:41:23 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-205925-20200108160213.patch</filename>
            <type>text/plain</type>
            <size>7720</size>
            <attacher name="Antoine Quint">graouts</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjU0MTMzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOTZkNzVhZDAyOWIyYWVh
MzUzN2YyN2JmOGJjYTgwOTM1MGU1NTZkYS4uY2RjNTE1ZDdkNjc3OWU0YTE1MTE0MzM1YzMyNDk0
ODgyN2YwZDNhYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDMzIEBACisyMDIwLTAxLTA4ICBBbnRv
aW5lIFF1aW50ICA8Z3Jhb3V0c0BhcHBsZS5jb20+CisKKyAgICAgICAgW1dlYiBBbmltYXRpb25z
XSBTdG9wIGNyZWF0aW5nIENTUyBBbmltYXRpb25zIGZvciA8bm9zY3JpcHQ+IGVsZW1lbnRzCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDU5MjUKKyAg
ICAgICAgPHJkYXI6Ly9wcm9ibGVtLzU4MTU4NDc5PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFRlc3Q6IHdlYmFuaW1hdGlvbnMvbm8tY3NzLWFuaW1h
dGlvbi1vbi1ub3NjcmlwdC5odG1sCisKKyAgICAgICAgSXQgbWFrZXMgbm8gc2Vuc2UgdG8gY3Jl
YXRlIENTUyBBbmltYXRpb25zIGZvciBhIDxub3NjcmlwdD4gZWxlbWVudCBhbmQgaXQgaGFzIHRo
ZSBzaWRlIGVmZmVjdCBvZiBwb3RlbnRpYWwgY3Jhc2hlcy4KKyAgICAgICAgSW5kZWVkLCBBbmlt
YXRpb25UaW1lbGluZTo6dXBkYXRlQ1NTQW5pbWF0aW9uc0ZvckVsZW1lbnQoKSBtYXkgYmUgY2Fs
bGVkIHdpdGhvdXQgYSBjdXJyZW50U3R5bGUgYW5kIHNvIHdlIG5ldmVyIGhhdmUKKyAgICAgICAg
YSBsaXN0IG9mIHByZXZpb3VzbHktYXBwbGllZCBhbmltYXRpb25zIHRvIGNvbXBhcmUgdG8gdGhl
IGxpc3Qgb2YgYW5pbWF0aW9ucyBpbiBhZnRlckNoYW5nZVN0eWxlLiBTbyBvbiBlYWNoIGNhbGwg
d2UKKyAgICAgICAgZW5kIHVwIGNyZWF0aW5nIGEgbmV3IENTU0FuaW1hdGlvbiBhbmQgdGhlIHBy
ZXZpb3VzIGFuaW1hdGlvbiBmb3IgdGhlIHNhbWUgbmFtZSBpcyBuZXZlciBleHBsaWNpdGx5IHJl
bW92ZWQgZnJvbSB0aGUKKyAgICAgICAgZWZmZWN0IHN0YWNrIGFuZCBpcyBldmVudHVhbGx5IGRl
c3Ryb3llZCBhbmQgdGhlIFdlYWtQdHIgZm9yIGl0IGluIHRoZSBzdGFjayBlbmRzIHVwIGJlaW5n
IG51bGwsIHdoaWNoIHdvdWxkIGNhdXNlIGEKKyAgICAgICAgY3Jhc2ggdW5kZXIgS2V5ZnJhbWVF
ZmZlY3RTdGFjazo6ZW5zdXJlRWZmZWN0c0FyZVNvcnRlZCgpLgorCisgICAgICAgIFdlIG5vdyBw
cmV2ZW50IGVsZW1lbnRzIHN1Y2ggYXMgPG5vc2NyaXB0PiBmcm9tIGJlaW5nIGNvbnNpZGVyZWQg
Zm9yIENTUyBBbmltYXRpb25zIGluIFRyZWVSZXNvbHZlcjo6cmVzb2x2ZUVsZW1lbnQoKS4KKwor
ICAgICAgICAqIGRvbS9FbGVtZW50LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkVsZW1lbnQ6OnJl
bmRlcmVySXNOZWVkZWQpOgorICAgICAgICAqIGRvbS9FbGVtZW50Lmg6CisgICAgICAgIChXZWJD
b3JlOjpFbGVtZW50OjpyZW5kZXJlcklzRXZlck5lZWRlZCk6CisgICAgICAgICogaHRtbC9IVE1M
RWxlbWVudC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpIVE1MRWxlbWVudDo6cmVuZGVyZXJJc0V2
ZXJOZWVkZWQpOgorICAgICAgICAoV2ViQ29yZTo6SFRNTEVsZW1lbnQ6OnJlbmRlcmVySXNOZWVk
ZWQpOiBEZWxldGVkLgorICAgICAgICAqIGh0bWwvSFRNTEVsZW1lbnQuaDoKKyAgICAgICAgKiBz
dHlsZS9TdHlsZVRyZWVSZXNvbHZlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpTdHlsZTo6VHJl
ZVJlc29sdmVyOjpyZXNvbHZlRWxlbWVudCk6CisKIDIwMjAtMDEtMDcgIHlvdWVubiBmYWJsZXQg
IDx5b3Vlbm5AYXBwbGUuY29tPgogCiAgICAgICAgIEltcGxlbWVudCBNZWRpYVJlY29yZGVyIGJh
Y2tlbmQgaW4gR1BVUHJvY2VzcwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvZG9tL0VsZW1l
bnQuY3BwIGIvU291cmNlL1dlYkNvcmUvZG9tL0VsZW1lbnQuY3BwCmluZGV4IGYzNGZlMTM4OTRi
MTYwMmVkMTk5MjczYjBmMDlmYzA4ZmFkMTM2ODYuLmI4NDI5N2VjZDY5MjNkY2JiMWFkYWVjMmQ0
YWQyZmYzMzQwNDMzYzYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2RvbS9FbGVtZW50LmNw
cAorKysgYi9Tb3VyY2UvV2ViQ29yZS9kb20vRWxlbWVudC5jcHAKQEAgLTIxMTQsNyArMjExNCw3
IEBAIGNvbnN0IEF0b21TdHJpbmcmIEVsZW1lbnQ6OmltYWdlU291cmNlVVJMKCkgY29uc3QKIAog
Ym9vbCBFbGVtZW50OjpyZW5kZXJlcklzTmVlZGVkKGNvbnN0IFJlbmRlclN0eWxlJiBzdHlsZSkK
IHsKLSAgICByZXR1cm4gc3R5bGUuZGlzcGxheSgpICE9IERpc3BsYXlUeXBlOjpOb25lICYmIHN0
eWxlLmRpc3BsYXkoKSAhPSBEaXNwbGF5VHlwZTo6Q29udGVudHM7CisgICAgcmV0dXJuIHJlbmRl
cmVySXNFdmVyTmVlZGVkKCkgJiYgc3R5bGUuZGlzcGxheSgpICE9IERpc3BsYXlUeXBlOjpOb25l
ICYmIHN0eWxlLmRpc3BsYXkoKSAhPSBEaXNwbGF5VHlwZTo6Q29udGVudHM7CiB9CiAKIFJlbmRl
clB0cjxSZW5kZXJFbGVtZW50PiBFbGVtZW50OjpjcmVhdGVFbGVtZW50UmVuZGVyZXIoUmVuZGVy
U3R5bGUmJiBzdHlsZSwgY29uc3QgUmVuZGVyVHJlZVBvc2l0aW9uJikKZGlmZiAtLWdpdCBhL1Nv
dXJjZS9XZWJDb3JlL2RvbS9FbGVtZW50LmggYi9Tb3VyY2UvV2ViQ29yZS9kb20vRWxlbWVudC5o
CmluZGV4IGZhOGNiNGRmOWRkNjY1NzM4NGVjMjdlZDU1OWU2MGE4NTQ2NDVjNGUuLjdlYzlkOTY5
YzU5NjdmOWIxYWQ0MjgxNDFjNGRjNTI2OTkyZjFlN2IgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJD
b3JlL2RvbS9FbGVtZW50LmgKKysrIGIvU291cmNlL1dlYkNvcmUvZG9tL0VsZW1lbnQuaApAQCAt
Mjg1LDYgKzI4NSw3IEBAIHB1YmxpYzoKIAogICAgIHZpcnR1YWwgUmVuZGVyUHRyPFJlbmRlckVs
ZW1lbnQ+IGNyZWF0ZUVsZW1lbnRSZW5kZXJlcihSZW5kZXJTdHlsZSYmLCBjb25zdCBSZW5kZXJU
cmVlUG9zaXRpb24mKTsKICAgICB2aXJ0dWFsIGJvb2wgcmVuZGVyZXJJc05lZWRlZChjb25zdCBS
ZW5kZXJTdHlsZSYpOworICAgIHZpcnR1YWwgYm9vbCByZW5kZXJlcklzRXZlck5lZWRlZCgpIHsg
cmV0dXJuIHRydWU7IH0KIAogICAgIFdFQkNPUkVfRVhQT1JUIFNoYWRvd1Jvb3QqIHNoYWRvd1Jv
b3QoKSBjb25zdDsKICAgICBTaGFkb3dSb290KiBzaGFkb3dSb290Rm9yQmluZGluZ3MoSlNDOjpK
U0dsb2JhbE9iamVjdCYpIGNvbnN0OwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaHRtbC9I
VE1MRWxlbWVudC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxFbGVtZW50LmNwcAppbmRl
eCAwYTBjYjA5ODE3YTRjY2RiYjE1ODE5N2RlNjg5M2ZkZWQ4NTMxZTgxLi42OGNmNTUzYWI0OTBh
ZTRkZjA3YzdkZmI2MmI3ZGRkMDY2OTA2MGVhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9o
dG1sL0hUTUxFbGVtZW50LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxFbGVtZW50
LmNwcApAQCAtNzMzLDcgKzczMyw3IEBAIHZvaWQgSFRNTEVsZW1lbnQ6OnNldFRyYW5zbGF0ZShi
b29sIGVuYWJsZSkKICAgICBzZXRBdHRyaWJ1dGVXaXRob3V0U3luY2hyb25pemF0aW9uKHRyYW5z
bGF0ZUF0dHIsIGVuYWJsZSA/ICJ5ZXMiIDogIm5vIik7CiB9CiAKLWJvb2wgSFRNTEVsZW1lbnQ6
OnJlbmRlcmVySXNOZWVkZWQoY29uc3QgUmVuZGVyU3R5bGUmIHN0eWxlKQorYm9vbCBIVE1MRWxl
bWVudDo6cmVuZGVyZXJJc0V2ZXJOZWVkZWQoKQogewogICAgIGlmIChoYXNUYWdOYW1lKG5vc2Ny
aXB0VGFnKSkgewogICAgICAgICBSZWZQdHI8RnJhbWU+IGZyYW1lID0gZG9jdW1lbnQoKS5mcmFt
ZSgpOwpAQCAtNzQ0LDcgKzc0NCw3IEBAIGJvb2wgSFRNTEVsZW1lbnQ6OnJlbmRlcmVySXNOZWVk
ZWQoY29uc3QgUmVuZGVyU3R5bGUmIHN0eWxlKQogICAgICAgICBpZiAoZnJhbWUgJiYgZnJhbWUt
PmxvYWRlcigpLnN1YmZyYW1lTG9hZGVyKCkuYWxsb3dQbHVnaW5zKCkpCiAgICAgICAgICAgICBy
ZXR1cm4gZmFsc2U7CiAgICAgfQotICAgIHJldHVybiBTdHlsZWRFbGVtZW50OjpyZW5kZXJlcklz
TmVlZGVkKHN0eWxlKTsKKyAgICByZXR1cm4gU3R5bGVkRWxlbWVudDo6cmVuZGVyZXJJc0V2ZXJO
ZWVkZWQoKTsKIH0KIAogUmVuZGVyUHRyPFJlbmRlckVsZW1lbnQ+IEhUTUxFbGVtZW50OjpjcmVh
dGVFbGVtZW50UmVuZGVyZXIoUmVuZGVyU3R5bGUmJiBzdHlsZSwgY29uc3QgUmVuZGVyVHJlZVBv
c2l0aW9uJikKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2h0bWwvSFRNTEVsZW1lbnQuaCBi
L1NvdXJjZS9XZWJDb3JlL2h0bWwvSFRNTEVsZW1lbnQuaAppbmRleCBmMTAyMGJiYTE3YjRlMGZh
ZWZmMThkYWMxYTg4YTIwMTkzNzI2ZmYxLi4zOTVkYjFlNjJlZWRkMWQ5MDBlOWU3YzNlYzg4OGNi
MDBjYjJhNGEzIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxFbGVtZW50LmgK
KysrIGIvU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRWxlbWVudC5oCkBAIC03MCw4ICs3MCw4IEBA
IHB1YmxpYzoKIAogICAgIHZvaWQgYWNjZXNzS2V5QWN0aW9uKGJvb2wgc2VuZE1vdXNlRXZlbnRz
KSBvdmVycmlkZTsKIAotICAgIGJvb2wgcmVuZGVyZXJJc05lZWRlZChjb25zdCBSZW5kZXJTdHls
ZSYpIG92ZXJyaWRlOwogICAgIFJlbmRlclB0cjxSZW5kZXJFbGVtZW50PiBjcmVhdGVFbGVtZW50
UmVuZGVyZXIoUmVuZGVyU3R5bGUmJiwgY29uc3QgUmVuZGVyVHJlZVBvc2l0aW9uJikgb3ZlcnJp
ZGU7CisgICAgYm9vbCByZW5kZXJlcklzRXZlck5lZWRlZCgpIGZpbmFsOwogCiAgICAgV0VCQ09S
RV9FWFBPUlQgdmlydHVhbCBIVE1MRm9ybUVsZW1lbnQqIGZvcm0oKSBjb25zdDsKIApkaWZmIC0t
Z2l0IGEvU291cmNlL1dlYkNvcmUvc3R5bGUvU3R5bGVUcmVlUmVzb2x2ZXIuY3BwIGIvU291cmNl
L1dlYkNvcmUvc3R5bGUvU3R5bGVUcmVlUmVzb2x2ZXIuY3BwCmluZGV4IGE2OTc2NTVmMDQ4NzBh
YjU0MDUwYTI0NWFhNmIyZDE5YjM3OTMwYWUuLjdhMzBjNmNiOWRjYjFiMTJjZTk3NTIyZjU2OGE3
YmYxYzhmMDllMTIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3N0eWxlL1N0eWxlVHJlZVJl
c29sdmVyLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9zdHlsZS9TdHlsZVRyZWVSZXNvbHZlci5j
cHAKQEAgLTE5OCw2ICsxOTgsOSBAQCBFbGVtZW50VXBkYXRlcyBUcmVlUmVzb2x2ZXI6OnJlc29s
dmVFbGVtZW50KEVsZW1lbnQmIGVsZW1lbnQpCiAgICAgICAgIHJldHVybiB7IH07CiAgICAgfQog
CisgICAgaWYgKCFlbGVtZW50LnJlbmRlcmVySXNFdmVyTmVlZGVkKCkpCisgICAgICAgIHJldHVy
biB7IH07CisKICAgICBhdXRvIG5ld1N0eWxlID0gc3R5bGVGb3JFbGVtZW50KGVsZW1lbnQsIHBh
cmVudCgpLnN0eWxlKTsKIAogICAgIGlmICghYWZmZWN0c1JlbmRlcmVkU3VidHJlZShlbGVtZW50
LCAqbmV3U3R5bGUpKQpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nIGIvTGF5b3V0
VGVzdHMvQ2hhbmdlTG9nCmluZGV4IGNiYWQ1YjE4YjA2MjljYzgwY2JjMjkyMjk2ZTU0NmQxOTQy
YjFjNWEuLmIxNTU2ZmMxMzBmMjU0MzhlZmU0N2RjYmNlNWU2MDM4M2U4MWIwYTkgMTAwNjQ0Ci0t
LSBhL0xheW91dFRlc3RzL0NoYW5nZUxvZworKysgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKQEAg
LTEsMyArMSwxNiBAQAorMjAyMC0wMS0wOCAgQW50b2luZSBRdWludCAgPGdyYW91dHNAYXBwbGUu
Y29tPgorCisgICAgICAgIFtXZWIgQW5pbWF0aW9uc10gU3RvcCBjcmVhdGluZyBDU1MgQW5pbWF0
aW9ucyBmb3IgPG5vc2NyaXB0PiBlbGVtZW50cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjA1OTI1CisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS81ODE1
ODQ3OT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBB
ZGQgYSBuZXcgdGVzdCB0aGF0IGNoZWNrcyB0aGF0IHNldHRpbmcgdGhlIGBhbmltYXRpb25gIHBy
b3BlcnR5IG9uIGEgPG5vc2NyaXB0PiBlbGVtZW50IGRvZXMgbm90IHlpZWxkIHRoZSBjcmVhdGlv
biBvZiBhIENTU0FuaW1hdGlvbiBvYmplY3QuCisKKyAgICAgICAgKiB3ZWJhbmltYXRpb25zL25v
LWNzcy1hbmltYXRpb24tb24tbm9zY3JpcHQtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAg
KiB3ZWJhbmltYXRpb25zL25vLWNzcy1hbmltYXRpb24tb24tbm9zY3JpcHQuaHRtbDogQWRkZWQu
CisKIDIwMjAtMDEtMDcgIFBhYmxvIFNhYXZlZHJhICA8cHNhYXZlZHJhQGlnYWxpYS5jb20+CiAK
ICAgICAgICAgUmUtc3luYyB3ZWItcGxhdGZvcm0tdGVzdHMvY29udGVudC1zZWN1cml0eS1wb2xp
Y3kgZnJvbSB1cHN0cmVhbQpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvd2ViYW5pbWF0aW9ucy9u
by1jc3MtYW5pbWF0aW9uLW9uLW5vc2NyaXB0LWV4cGVjdGVkLnR4dCBiL0xheW91dFRlc3RzL3dl
YmFuaW1hdGlvbnMvbm8tY3NzLWFuaW1hdGlvbi1vbi1ub3NjcmlwdC1leHBlY3RlZC50eHQKbmV3
IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw
MDAwMDAwMC4uY2M3ODg0ZWZmYmJjOWYwNmZhYWEzM2Q1ZmZiMmVlYWJkM2JiNjJjZQotLS0gL2Rl
di9udWxsCisrKyBiL0xheW91dFRlc3RzL3dlYmFuaW1hdGlvbnMvbm8tY3NzLWFuaW1hdGlvbi1v
bi1ub3NjcmlwdC1leHBlY3RlZC50eHQKQEAgLTAsMCArMSwzIEBACisKK1BBU1MgQSBDU1MgQW5p
bWF0aW9uIGNhbm5vdCBiZSBjcmVhdGVkIG9uIGEgPG5vc2NyaXB0PiBlbGVtZW50LiAKKwpkaWZm
IC0tZ2l0IGEvTGF5b3V0VGVzdHMvd2ViYW5pbWF0aW9ucy9uby1jc3MtYW5pbWF0aW9uLW9uLW5v
c2NyaXB0Lmh0bWwgYi9MYXlvdXRUZXN0cy93ZWJhbmltYXRpb25zL25vLWNzcy1hbmltYXRpb24t
b24tbm9zY3JpcHQuaHRtbApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAw
MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwLi40MjE0MWVhM2VkZWExMzcxYzIxNGYyNmNhOTEx
MDQ4MjgwN2E5YmMzCi0tLSAvZGV2L251bGwKKysrIGIvTGF5b3V0VGVzdHMvd2ViYW5pbWF0aW9u
cy9uby1jc3MtYW5pbWF0aW9uLW9uLW5vc2NyaXB0Lmh0bWwKQEAgLTAsMCArMSwyOCBAQAorPCFE
T0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorPHN0eWxlPgorCitub3NjcmlwdCB7CisgICAg
YW5pbWF0aW9uOiBhbmltYXRpb24gMXMgaW5maW5pdGU7Cit9CisKK0BrZXlmcmFtZXMgYW5pbWF0
aW9uIHsKKyAgICB0byB7IG1hcmdpbi1sZWZ0OiAiMTAwcHgiIH07Cit9CisKKzwvc3R5bGU+Cis8
L2hlYWQ+Cis8Ym9keT4KKzxzY3JpcHQgc3JjPSIuLi9yZXNvdXJjZXMvdGVzdGhhcm5lc3MuanMi
Pjwvc2NyaXB0PgorPHNjcmlwdCBzcmM9Ii4uL3Jlc291cmNlcy90ZXN0aGFybmVzc3JlcG9ydC5q
cyI+PC9zY3JpcHQ+Cis8bm9zY3JpcHQ+SGVsbG8gV29ybGQ8L25vc2NyaXB0PgorPHNjcmlwdD4K
KwordGVzdCgoKSA9PiB7CisgICAgYXNzZXJ0X2VxdWFscyhkb2N1bWVudC5xdWVyeVNlbGVjdG9y
KCJub3NjcmlwdCIpLmdldEFuaW1hdGlvbnMoKS5sZW5ndGgsIDApOworfSwgIkEgQ1NTIEFuaW1h
dGlvbiBjYW5ub3QgYmUgY3JlYXRlZCBvbiBhIDxub3NjcmlwdD4gZWxlbWVudC4iKTsKKworPC9z
Y3JpcHQ+Cis8L2JvZHk+Cis8L2h0bWw+Cg==
</data>
<flag name="review"
          id="402868"
          type_id="1"
          status="+"
          setter="koivisto"
    />
          </attachment>
      

    </bug>

</bugzilla>