<?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>140935</bug_id>
          
          <creation_ts>2015-01-27 06:32:37 -0800</creation_ts>
          <short_desc>[GTK] Resize the redirected XComposite window again after leaving accelerated compositing mode</short_desc>
          <delta_ts>2015-01-29 23:38:18 -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>WebKitGTK</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>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>berto</cc>
    
    <cc>commit-queue</cc>
    
    <cc>gustavo</cc>
    
    <cc>mrobinson</cc>
    
    <cc>svillar</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1064494</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-01-27 06:32:37 -0800</bug_when>
    <thetext>Since r178414 we don&apos;t resize the redirected XComposite window until we enter accelerated compositing mode, but after leaving it the redirected window keeps its size. We should resize to 1x1 again and free the XPixmap and cairo surface to save that memory.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1064495</commentid>
    <comment_count>1</comment_count>
      <attachid>245440</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-01-27 06:37:39 -0800</bug_when>
    <thetext>Created attachment 245440
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1064496</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2015-01-27 06:39:38 -0800</bug_when>
    <thetext>Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1065192</commentid>
    <comment_count>3</comment_count>
      <attachid>245440</attachid>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2015-01-29 02:18:39 -0800</bug_when>
    <thetext>Comment on attachment 245440
Patch

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

Awesome fix.

&gt; Source/WebKit2/UIProcess/gtk/RedirectedXCompositeWindow.cpp:219
&gt; +    XResizeWindow(m_display, m_window, std::max(1, size.width()), std::max(1, size.height()));

Perhaps a comment explaining that 0x0 windows are not allowed by X...

&gt; Source/WebKit2/UIProcess/gtk/RedirectedXCompositeWindow.cpp:259
&gt; +        std::max(1, m_size.width()), std::max(1, m_size.height())));

Ditto.

Orthogonal to this we should probably assert in ::surface() if we aren&apos;t in AC.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1065509</commentid>
    <comment_count>4</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-01-29 23:29:23 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; Comment on attachment 245440 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=245440&amp;action=review
&gt; 
&gt; Awesome fix.

Thanks.

&gt; &gt; Source/WebKit2/UIProcess/gtk/RedirectedXCompositeWindow.cpp:219
&gt; &gt; +    XResizeWindow(m_display, m_window, std::max(1, size.width()), std::max(1, size.height()));
&gt; 
&gt; Perhaps a comment explaining that 0x0 windows are not allowed by X...

Sure.

&gt; &gt; Source/WebKit2/UIProcess/gtk/RedirectedXCompositeWindow.cpp:259
&gt; &gt; +        std::max(1, m_size.width()), std::max(1, m_size.height())));
&gt; 
&gt; Ditto.

I&apos;ve realized we don&apos;t need this, since we are going to assert for empty sizes here.

&gt; Orthogonal to this we should probably assert in ::surface() if we aren&apos;t in
&gt; AC.

Sure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1065512</commentid>
    <comment_count>5</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-01-29 23:38:18 -0800</bug_when>
    <thetext>Committed r179397: &lt;http://trac.webkit.org/changeset/179397&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>245440</attachid>
            <date>2015-01-27 06:37:39 -0800</date>
            <delta_ts>2015-01-29 02:18:39 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>wk2-exit-ac-mode.diff</filename>
            <type>text/plain</type>
            <size>7495</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCA1OWIwZDU2Li4zMGFkYjIzIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMzkg
QEAKIDIwMTUtMDEtMjcgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgogCisgICAgICAgIFtHVEtdIFJlc2l6ZSB0aGUgcmVkaXJlY3RlZCBYQ29tcG9zaXRlIGFnYWlu
IGFmdGVyIGxlYXZpbmcgYWNjZWxlcmF0ZWQgY29tcG9zaXRpbmcgbW9kZQorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTQwOTM1CisKKyAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgU2luY2UgcjE3ODQxNCB3ZSBkb24n
dCByZXNpemUgdGhlIHJlZGlyZWN0ZWQgWENvbXBvc2l0ZSB3aW5kb3cKKyAgICAgICAgdW50aWwg
d2UgZW50ZXIgYWNjZWxlcmF0ZWQgY29tcG9zaXRpbmcgbW9kZSwgYnV0IGFmdGVyIGxlYXZpbmcg
aXQKKyAgICAgICAgdGhlIHJlZGlyZWN0ZWQgd2luZG93IGtlZXBzIGl0cyBzaXplLiBXZSBzaG91
bGQgcmVzaXplIGl0IHRvIDF4MQorICAgICAgICBhZ2FpbiBhbmQgZnJlZSB0aGUgWFBpeG1hcCBh
bmQgY2Fpcm8gc3VyZmFjZSB0byBzYXZlIHRoYXQgbWVtb3J5CisgICAgICAgIHdoaWxlIG5vdCBp
biBhY2NlbGVyYXRlZCBjb21wb3NpdGluZyBtb2RlLgorCisgICAgICAgICogVUlQcm9jZXNzL0FQ
SS9ndGsvUGFnZUNsaWVudEltcGwuY3BwOgorICAgICAgICAoV2ViS2l0OjpQYWdlQ2xpZW50SW1w
bDo6ZW50ZXJBY2NlbGVyYXRlZENvbXBvc2l0aW5nTW9kZSk6IE5vdGlmeQorICAgICAgICB0aGUg
V2ViS2l0V2ViVmlld0Jhc2UuCisgICAgICAgIChXZWJLaXQ6OlBhZ2VDbGllbnRJbXBsOjpleGl0
QWNjZWxlcmF0ZWRDb21wb3NpdGluZ01vZGUpOiBEaXR0by4KKyAgICAgICAgKiBVSVByb2Nlc3Mv
QVBJL2d0ay9XZWJLaXRXZWJWaWV3QmFzZS5jcHA6CisgICAgICAgICh3ZWJraXRXZWJWaWV3QmFz
ZUVudGVyQWNjZWxlcmF0ZWRDb21wb3NpdGluZ01vZGUpOiBSZXNpemUgdGhlCisgICAgICAgIFhD
b21wb3NpdGUgd2luZG93IHRvIHRoZSBjdXJyZW50IGRyYXdpbmcgYXJlYSBzaXplLgorICAgICAg
ICAod2Via2l0V2ViVmlld0Jhc2VFeGl0QWNjZWxlcmF0ZWRDb21wb3NpdGluZ01vZGUpOiBSZXNp
emUgdGhlCisgICAgICAgIFhDb21wb3NpdGUgd2luZG93IHRvIGFuZCBlbXB0eSBzaXplLgorICAg
ICAgICAqIFVJUHJvY2Vzcy9BUEkvZ3RrL1dlYktpdFdlYlZpZXdCYXNlUHJpdmF0ZS5oOgorICAg
ICAgICAqIFVJUHJvY2Vzcy9ndGsvUmVkaXJlY3RlZFhDb21wb3NpdGVXaW5kb3cuY3BwOgorICAg
ICAgICAoV2ViS2l0OjpSZWRpcmVjdGVkWENvbXBvc2l0ZVdpbmRvdzo6UmVkaXJlY3RlZFhDb21w
b3NpdGVXaW5kb3cpOgorICAgICAgICBEbyBub3QgaW5pdGlhbGl6ZSBtX3NpemUgc2luY2Ugd2Ug
bm93IGhhbmRsZSB0aGUgZW1wdHkgc2l6ZSBhcyBhCisgICAgICAgIGVzcGVjaWFsIGNhc2UuIE1h
a2Ugc3VyZSB3ZSBhbHdheXMgY3JlYXRlIHRoZSBYIHdpbmRvdyB3aXRoIGF0CisgICAgICAgIGxl
YXN0IDF4MSBzaXplLgorICAgICAgICAoV2ViS2l0OjpSZWRpcmVjdGVkWENvbXBvc2l0ZVdpbmRv
dzo6cmVzaXplKTogUmVzaXplIHRoZSB3aW5kb3cgdG8KKyAgICAgICAgYXQgbGVhc3QgMXgxLCBi
dXQgd2hlbiBlbiBlbXB0eSBzaXplIGlzIGdpdmVuLCBjYWxsCisgICAgICAgIGNsZWFudXBQaXht
YXBBbmRQaXhtYXBTdXJmYWNlKCkgdG8gcmVsZWFzZSB0aG9zZSByZXNvdXJjZXMuCisgICAgICAg
IChXZWJLaXQ6OlJlZGlyZWN0ZWRYQ29tcG9zaXRlV2luZG93OjpzdXJmYWNlKTogQ3JlYXRlIHRo
ZSBjYWlybworICAgICAgICBzdXJmYWNlIHdpdGggYXQgbGVhc3QgMXgxIHNpemUuCisKKzIwMTUt
MDEtMjcgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgorCiAgICAg
ICAgIFJFR1JFU1NJT04ocjE2Njk3NSk6IFtHVEtdIFBhZ2Ugb3ZlcmxheXMgYXJlIG5vdCBkcmF3
biBhbnltb3JlIGFmdGVyIHIxNjY5NzUKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTEzMTQzMwogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL2d0ay9QYWdlQ2xpZW50SW1wbC5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nl
c3MvQVBJL2d0ay9QYWdlQ2xpZW50SW1wbC5jcHAKaW5kZXggNmJmODE3NS4uOTI5OTYwOCAxMDA2
NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9ndGsvUGFnZUNsaWVudEltcGwu
Y3BwCisrKyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZ3RrL1BhZ2VDbGllbnRJbXBs
LmNwcApAQCAtMjQ1LDEyICsyNDUsMTIgQEAgdm9pZCBQYWdlQ2xpZW50SW1wbDo6c2V0VGV4dElu
ZGljYXRvckFuaW1hdGlvblByb2dyZXNzKGZsb2F0KQogCiB2b2lkIFBhZ2VDbGllbnRJbXBsOjpl
bnRlckFjY2VsZXJhdGVkQ29tcG9zaXRpbmdNb2RlKGNvbnN0IExheWVyVHJlZUNvbnRleHQmKQog
ewotICAgIG5vdEltcGxlbWVudGVkKCk7CisgICAgd2Via2l0V2ViVmlld0Jhc2VFbnRlckFjY2Vs
ZXJhdGVkQ29tcG9zaXRpbmdNb2RlKFdFQktJVF9XRUJfVklFV19CQVNFKG1fdmlld1dpZGdldCkp
OwogfQogCiB2b2lkIFBhZ2VDbGllbnRJbXBsOjpleGl0QWNjZWxlcmF0ZWRDb21wb3NpdGluZ01v
ZGUoKQogewotICAgIG5vdEltcGxlbWVudGVkKCk7CisgICAgd2Via2l0V2ViVmlld0Jhc2VFeGl0
QWNjZWxlcmF0ZWRDb21wb3NpdGluZ01vZGUoV0VCS0lUX1dFQl9WSUVXX0JBU0UobV92aWV3V2lk
Z2V0KSk7CiB9CiAKIHZvaWQgUGFnZUNsaWVudEltcGw6OnVwZGF0ZUFjY2VsZXJhdGVkQ29tcG9z
aXRpbmdNb2RlKGNvbnN0IExheWVyVHJlZUNvbnRleHQmKQpkaWZmIC0tZ2l0IGEvU291cmNlL1dl
YktpdDIvVUlQcm9jZXNzL0FQSS9ndGsvV2ViS2l0V2ViVmlld0Jhc2UuY3BwIGIvU291cmNlL1dl
YktpdDIvVUlQcm9jZXNzL0FQSS9ndGsvV2ViS2l0V2ViVmlld0Jhc2UuY3BwCmluZGV4IGJkNTY4
MTQuLmE3OTMzZTcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZ3Rr
L1dlYktpdFdlYlZpZXdCYXNlLmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJ
L2d0ay9XZWJLaXRXZWJWaWV3QmFzZS5jcHAKQEAgLTEzMDgsMyArMTMwOCwyOSBAQCB2b2lkIHdl
YmtpdFdlYlZpZXdCYXNlUmVzZXRDbGlja0NvdW50ZXIoV2ViS2l0V2ViVmlld0Jhc2UqIHdlYmtp
dFdlYlZpZXdCYXNlKQogewogICAgIHdlYmtpdFdlYlZpZXdCYXNlLT5wcml2LT5jbGlja0NvdW50
ZXIucmVzZXQoKTsKIH0KKwordm9pZCB3ZWJraXRXZWJWaWV3QmFzZUVudGVyQWNjZWxlcmF0ZWRD
b21wb3NpdGluZ01vZGUoV2ViS2l0V2ViVmlld0Jhc2UqIHdlYmtpdFdlYlZpZXdCYXNlKQorewor
I2lmIFVTRShURVhUVVJFX01BUFBFUl9HTCkgJiYgUExBVEZPUk0oWDExKQorICAgIFdlYktpdFdl
YlZpZXdCYXNlUHJpdmF0ZSogcHJpdiA9IHdlYmtpdFdlYlZpZXdCYXNlLT5wcml2OworICAgIGlm
ICghcHJpdi0+cmVkaXJlY3RlZFdpbmRvdykKKyAgICAgICAgcmV0dXJuOworICAgIERyYXdpbmdB
cmVhUHJveHlJbXBsKiBkcmF3aW5nQXJlYSA9IHN0YXRpY19jYXN0PERyYXdpbmdBcmVhUHJveHlJ
bXBsKj4ocHJpdi0+cGFnZVByb3h5LT5kcmF3aW5nQXJlYSgpKTsKKyAgICBpZiAoIWRyYXdpbmdB
cmVhKQorICAgICAgICByZXR1cm47CisgICAgcHJpdi0+cmVkaXJlY3RlZFdpbmRvdy0+cmVzaXpl
KGRyYXdpbmdBcmVhLT5zaXplKCkpOworI2Vsc2UKKyAgICBVTlVTRURfUEFSQU0od2Via2l0V2Vi
Vmlld0Jhc2UpOworI2VuZGlmCit9CisKK3ZvaWQgd2Via2l0V2ViVmlld0Jhc2VFeGl0QWNjZWxl
cmF0ZWRDb21wb3NpdGluZ01vZGUoV2ViS2l0V2ViVmlld0Jhc2UqIHdlYmtpdFdlYlZpZXdCYXNl
KQoreworI2lmIFVTRShURVhUVVJFX01BUFBFUl9HTCkgJiYgUExBVEZPUk0oWDExKQorICAgIFdl
YktpdFdlYlZpZXdCYXNlUHJpdmF0ZSogcHJpdiA9IHdlYmtpdFdlYlZpZXdCYXNlLT5wcml2Owor
ICAgIGlmIChwcml2LT5yZWRpcmVjdGVkV2luZG93KQorICAgICAgICBwcml2LT5yZWRpcmVjdGVk
V2luZG93LT5yZXNpemUoSW50U2l6ZSgpKTsKKyNlbHNlCisgICAgVU5VU0VEX1BBUkFNKHdlYmtp
dFdlYlZpZXdCYXNlKTsKKyNlbmRpZgorfQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvVUlQ
cm9jZXNzL0FQSS9ndGsvV2ViS2l0V2ViVmlld0Jhc2VQcml2YXRlLmggYi9Tb3VyY2UvV2ViS2l0
Mi9VSVByb2Nlc3MvQVBJL2d0ay9XZWJLaXRXZWJWaWV3QmFzZVByaXZhdGUuaAppbmRleCAwOTli
M2NiLi5hNWYzNTU1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2d0
ay9XZWJLaXRXZWJWaWV3QmFzZVByaXZhdGUuaAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nl
c3MvQVBJL2d0ay9XZWJLaXRXZWJWaWV3QmFzZVByaXZhdGUuaApAQCAtNjksNiArNjksOCBAQCB2
b2lkIHdlYmtpdFdlYlZpZXdCYXNlQWRkQXV0aGVudGljYXRpb25EaWFsb2coV2ViS2l0V2ViVmll
d0Jhc2UqLCBHdGtXaWRnZXQqIGF1dAogdm9pZCB3ZWJraXRXZWJWaWV3QmFzZUNhbmNlbEF1dGhl
bnRpY2F0aW9uRGlhbG9nKFdlYktpdFdlYlZpZXdCYXNlKik7CiB2b2lkIHdlYmtpdFdlYlZpZXdC
YXNlQWRkV2ViSW5zcGVjdG9yKFdlYktpdFdlYlZpZXdCYXNlKiwgR3RrV2lkZ2V0KiBpbnNwZWN0
b3IsIFdlYktpdDo6QXR0YWNobWVudFNpZGUpOwogdm9pZCB3ZWJraXRXZWJWaWV3QmFzZVJlc2V0
Q2xpY2tDb3VudGVyKFdlYktpdFdlYlZpZXdCYXNlKik7Cit2b2lkIHdlYmtpdFdlYlZpZXdCYXNl
RW50ZXJBY2NlbGVyYXRlZENvbXBvc2l0aW5nTW9kZShXZWJLaXRXZWJWaWV3QmFzZSopOwordm9p
ZCB3ZWJraXRXZWJWaWV3QmFzZUV4aXRBY2NlbGVyYXRlZENvbXBvc2l0aW5nTW9kZShXZWJLaXRX
ZWJWaWV3QmFzZSopOwogCiAjaWYgRU5BQkxFKERSQUdfU1VQUE9SVCkKIFdlYktpdDo6RHJhZ0Fu
ZERyb3BIYW5kbGVyJiB3ZWJraXRXZWJWaWV3QmFzZURyYWdBbmREcm9wSGFuZGxlcihXZWJLaXRX
ZWJWaWV3QmFzZSopOwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL2d0ay9S
ZWRpcmVjdGVkWENvbXBvc2l0ZVdpbmRvdy5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3Mv
Z3RrL1JlZGlyZWN0ZWRYQ29tcG9zaXRlV2luZG93LmNwcAppbmRleCBlYTViZTU1Li5lOTdlODFk
IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvZ3RrL1JlZGlyZWN0ZWRYQ29t
cG9zaXRlV2luZG93LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvZ3RrL1JlZGly
ZWN0ZWRYQ29tcG9zaXRlV2luZG93LmNwcApAQCAtMTM1LDcgKzEzNSw2IEBAIHN0ZDo6dW5pcXVl
X3B0cjxSZWRpcmVjdGVkWENvbXBvc2l0ZVdpbmRvdz4gUmVkaXJlY3RlZFhDb21wb3NpdGVXaW5k
b3c6OmNyZWF0ZShHCiAKIFJlZGlyZWN0ZWRYQ29tcG9zaXRlV2luZG93OjpSZWRpcmVjdGVkWENv
bXBvc2l0ZVdpbmRvdyhHZGtXaW5kb3cqIHBhcmVudFdpbmRvdywgc3RkOjpmdW5jdGlvbjx2b2lk
KCk+IGRhbWFnZU5vdGlmeSkKICAgICA6IG1fZGlzcGxheShHREtfRElTUExBWV9YRElTUExBWShn
ZGtfd2luZG93X2dldF9kaXNwbGF5KHBhcmVudFdpbmRvdykpKQotICAgICwgbV9zaXplKDEsIDEp
CiAgICAgLCBtX3dpbmRvdygwKQogICAgICwgbV9wYXJlbnRXaW5kb3coMCkKICAgICAsIG1fcGl4
bWFwKDApCkBAIC0xNzEsNyArMTcwLDkgQEAgUmVkaXJlY3RlZFhDb21wb3NpdGVXaW5kb3c6OlJl
ZGlyZWN0ZWRYQ29tcG9zaXRlV2luZG93KEdka1dpbmRvdyogcGFyZW50V2luZG93LAogICAgIHdp
bmRvd0F0dHJpYnV0ZXMub3ZlcnJpZGVfcmVkaXJlY3QgPSBGYWxzZTsKICAgICBtX3dpbmRvdyA9
IFhDcmVhdGVXaW5kb3cobV9kaXNwbGF5LAogICAgICAgICBtX3BhcmVudFdpbmRvdywKLSAgICAg
ICAgMCwgMCwgbV9zaXplLndpZHRoKCksIG1fc2l6ZS5oZWlnaHQoKSwKKyAgICAgICAgMCwgMCwK
KyAgICAgICAgc3RkOjptYXgoMSwgbV9zaXplLndpZHRoKCkpLAorICAgICAgICBzdGQ6Om1heCgx
LCBtX3NpemUuaGVpZ2h0KCkpLAogICAgICAgICAwLAogICAgICAgICBDb3B5RnJvbVBhcmVudCwK
ICAgICAgICAgSW5wdXRPdXRwdXQsCkBAIC0yMTUsMTEgKzIxNiwxMyBAQCB2b2lkIFJlZGlyZWN0
ZWRYQ29tcG9zaXRlV2luZG93OjpyZXNpemUoY29uc3QgSW50U2l6ZSYgc2l6ZSkKICAgICBpZiAo
c2l6ZSA9PSBtX3NpemUpCiAgICAgICAgIHJldHVybjsKIAotICAgIFhSZXNpemVXaW5kb3cobV9k
aXNwbGF5LCBtX3dpbmRvdywgc2l6ZS53aWR0aCgpLCBzaXplLmhlaWdodCgpKTsKKyAgICBYUmVz
aXplV2luZG93KG1fZGlzcGxheSwgbV93aW5kb3csIHN0ZDo6bWF4KDEsIHNpemUud2lkdGgoKSks
IHN0ZDo6bWF4KDEsIHNpemUuaGVpZ2h0KCkpKTsKICAgICBYRmx1c2gobV9kaXNwbGF5KTsKIAog
ICAgIG1fc2l6ZSA9IHNpemU7CiAgICAgbV9uZWVkc05ld1BpeG1hcEFmdGVyUmVzaXplID0gdHJ1
ZTsKKyAgICBpZiAobV9zaXplLmlzRW1wdHkoKSkKKyAgICAgICAgY2xlYW51cFBpeG1hcEFuZFBp
eG1hcFN1cmZhY2UoKTsKIH0KIAogdm9pZCBSZWRpcmVjdGVkWENvbXBvc2l0ZVdpbmRvdzo6Y2xl
YW51cFBpeG1hcEFuZFBpeG1hcFN1cmZhY2UoKQpAQCAtMjUyLDcgKzI1NSw4IEBAIGNhaXJvX3N1
cmZhY2VfdCogUmVkaXJlY3RlZFhDb21wb3NpdGVXaW5kb3c6OnN1cmZhY2UoKQogICAgICAgICBy
ZXR1cm4gbnVsbHB0cjsKICAgICB9CiAKLSAgICBSZWZQdHI8Y2Fpcm9fc3VyZmFjZV90PiBuZXdT
dXJmYWNlID0gYWRvcHRSZWYoY2Fpcm9feGxpYl9zdXJmYWNlX2NyZWF0ZShtX2Rpc3BsYXksIG5l
d1BpeG1hcCwgd2luZG93QXR0cmlidXRlcy52aXN1YWwsIG1fc2l6ZS53aWR0aCgpLCBtX3NpemUu
aGVpZ2h0KCkpKTsKKyAgICBSZWZQdHI8Y2Fpcm9fc3VyZmFjZV90PiBuZXdTdXJmYWNlID0gYWRv
cHRSZWYoY2Fpcm9feGxpYl9zdXJmYWNlX2NyZWF0ZShtX2Rpc3BsYXksIG5ld1BpeG1hcCwgd2lu
ZG93QXR0cmlidXRlcy52aXN1YWwsCisgICAgICAgIHN0ZDo6bWF4KDEsIG1fc2l6ZS53aWR0aCgp
KSwgc3RkOjptYXgoMSwgbV9zaXplLmhlaWdodCgpKSkpOwogCiAgICAgLy8gTnZpZGlhIGRyaXZl
cnMgc2VlbSB0byBwcmVwYXJlIHRoZWlyIHJlZGlyZWN0ZWQgd2luZG93IHBpeG1hcCBhc3luY2hy
b25vdXNseSwgc28gZm9yIGEgZmV3IGZyYWN0aW9ucwogICAgIC8vIG9mIGEgc2Vjb25kIGFmdGVy
IGVhY2ggcmVzaXplLCB3aGlsZSBkb2luZyBjb250aW51b3VzIHJlc2l6aW5nICh3aGljaCBjb25z
dGFudGx5IGRlc3Ryb3lzIGFuZCBjcmVhdGVzCg==
</data>
<flag name="review"
          id="270383"
          type_id="1"
          status="+"
          setter="svillar"
    />
          </attachment>
      

    </bug>

</bugzilla>