<?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>126570</bug_id>
          
          <creation_ts>2014-01-07 02:55:40 -0800</creation_ts>
          <short_desc>REGRESSION(r161381): [GTK] Rendering is broken in GTK after r161381</short_desc>
          <delta_ts>2014-01-07 11:57:44 -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>Platform</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, Regression</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>abucur</cc>
    
    <cc>changseok</cc>
    
    <cc>commit-queue</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>glenn</cc>
    
    <cc>jinwoo7.song</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>ryuan.choi</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>thorton</cc>
    
    <cc>WebkitBugTracker</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>964936</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-07 02:55:40 -0800</bug_when>
    <thetext>Both GtkLauncher and MiniBrowser don&apos;t render anything after r161381. 

http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/43685/steps/layout-test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>964991</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-07 06:46:47 -0800</bug_when>
    <thetext>I think the problem has to do with the LayoutRect/IntRect conversion in RenderLayer.cpp. Using LayoutRect::infiniteRect() for the clips in RenderLayer.cpp fixes the problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965037</commentid>
    <comment_count>2</comment_count>
      <attachid>220532</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-07 09:18:10 -0800</bug_when>
    <thetext>Created attachment 220532
Patch

Using LayoutRect::infinteRect() instead of IntRect::infinteRect() when a LayoutRect is expected fixes the problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965041</commentid>
    <comment_count>3</comment_count>
    <who name="Andrei Bucur">abucur</who>
    <bug_when>2014-01-07 09:30:16 -0800</bug_when>
    <thetext>This may not be related, but while working on the CSS Regions visual overflow patch we noticed the cairo based GraphicsContext was not able to clip using the infiniteRect(). It seems to cause an integer overflow in cairo and it clips using a completely wrong rectangle.

It&apos;s probably a good idea to make infiniteRect platform specific, based on the limitations of the underlying graphics library.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965044</commentid>
    <comment_count>4</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-07 09:38:25 -0800</bug_when>
    <thetext>I don&apos;t know the details, but the Int/FloatRect::infiniteRect() is not defined the same way than the LayoutRect one, see:

LayoutRect:

LayoutUnit::nearlyMin() / 2, LayoutUnit::nearlyMin() / 2, LayoutUnit::nearlyMax(), LayoutUnit::nearlyMax()

which substituting nearlyMin/Max would be something like

(std::numeric_limits&lt;int&gt;::min() / 2) / 2, (std::numeric_limits&lt;int&gt;::min() / 2) / 2, std::numeric_limits&lt;int&gt;::max() / 2, std::numeric_limits&lt;int&gt;::max() / 2

IntRect:

-std::numeric_limits&lt;int&gt;::max() / 2, -std::numeric_limits&lt;int&gt;::max() / 2, std::numeric_limits&lt;int&gt;::max(), std::numeric_limits&lt;int&gt;::max()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965100</commentid>
    <comment_count>5</comment_count>
      <attachid>220532</attachid>
    <who name="Tim Horton">thorton</who>
    <bug_when>2014-01-07 11:53:08 -0800</bug_when>
    <thetext>Comment on attachment 220532
Patch

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

&gt; Source/WebCore/ChangeLog:9
&gt; +        Use LayoutRect::infiniteRect() instead of IntRect::infiniteRect()
&gt; +        when a LayoutRect is expected.

Whoops :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965103</commentid>
    <comment_count>6</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-07 11:57:44 -0800</bug_when>
    <thetext>Committed r161443: &lt;http://trac.webkit.org/changeset/161443&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>220532</attachid>
            <date>2014-01-07 09:18:10 -0800</date>
            <delta_ts>2014-01-07 11:53:08 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>wkcore-infinite-rect.diff</filename>
            <type>text/plain</type>
            <size>7851</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCA3ZTA1M2JiLi41NWQ1N2MxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjMg
QEAKKzIwMTQtMDEtMDcgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgorCisgICAgICAgIFJFR1JFU1NJT04ocjE2MTM4MSk6IFtHVEtdIFJlbmRlcmluZyBpcyBicm9r
ZW4gaW4gR1RLIGFmdGVyIHIxNjEzODEKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTEyNjU3MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgIFVzZSBMYXlvdXRSZWN0OjppbmZpbml0ZVJlY3QoKSBpbnN0ZWFkIG9m
IEludFJlY3Q6OmluZmluaXRlUmVjdCgpCisgICAgICAgIHdoZW4gYSBMYXlvdXRSZWN0IGlzIGV4
cGVjdGVkLgorCisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlckZsb3dUaHJlYWQuY3BwOgorICAg
ICAgICAoV2ViQ29yZTo6UmVuZGVyRmxvd1RocmVhZDo6ZnJhZ21lbnRzQm91bmRpbmdCb3gpOgor
ICAgICAgICAqIHJlbmRlcmluZy9SZW5kZXJMYXllci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpS
ZW5kZXJMYXllcjo6Y29sbGVjdEZyYWdtZW50cyk6CisgICAgICAgIChXZWJDb3JlOjpSZW5kZXJM
YXllcjo6Y2FsY3VsYXRlQ2xpcFJlY3RzKToKKyAgICAgICAgKiByZW5kZXJpbmcvUmVuZGVyTGF5
ZXJCYWNraW5nLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OmNsaXBCb3gpOgorICAgICAgICAqIHJl
bmRlcmluZy9SZW5kZXJMYXllckNvbXBvc2l0b3IuY3BwOgorICAgICAgICAoV2ViQ29yZTo6UmVu
ZGVyTGF5ZXJDb21wb3NpdG9yOjpjbGlwcGVkQnlBbmNlc3Rvcik6CisKIDIwMTQtMDEtMDcgIE1h
cmsgUm93ZSAgPG1yb3dlQGFwcGxlLmNvbT4KIAogICAgICAgICA8aHR0cHM6Ly93ZWJraXQub3Jn
L2IvMTI2NTYyPiBET01Qcm9ncmVzc0V2ZW50IGhhcyB1bnNwZWNpZmllZCBhdmFpbGFiaWxpdHkK
ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93VGhyZWFkLmNw
cCBiL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93VGhyZWFkLmNwcAppbmRleCA1
Zjc4ZThhLi4xMzhjOWVhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVu
ZGVyRmxvd1RocmVhZC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckZs
b3dUaHJlYWQuY3BwCkBAIC0xMDg5LDcgKzEwODksNyBAQCBMYXlvdXRSZWN0IFJlbmRlckZsb3dU
aHJlYWQ6OmZyYWdtZW50c0JvdW5kaW5nQm94KGNvbnN0IExheW91dFJlY3QmIGxheWVyQm91bmRp
bgogICAgIGZvciAoYXV0byBpdGVyID0gbV9yZWdpb25MaXN0LmJlZ2luKCksIGVuZCA9IG1fcmVn
aW9uTGlzdC5lbmQoKTsgaXRlciAhPSBlbmQ7ICsraXRlcikgewogICAgICAgICBSZW5kZXJSZWdp
b24qIHJlZ2lvbiA9ICppdGVyOwogICAgICAgICBMYXllckZyYWdtZW50cyBmcmFnbWVudHM7Ci0g
ICAgICAgIHJlZ2lvbi0+Y29sbGVjdExheWVyRnJhZ21lbnRzKGZyYWdtZW50cywgbGF5ZXJCb3Vu
ZGluZ0JveCwgSW50UmVjdDo6aW5maW5pdGVSZWN0KCkpOworICAgICAgICByZWdpb24tPmNvbGxl
Y3RMYXllckZyYWdtZW50cyhmcmFnbWVudHMsIGxheWVyQm91bmRpbmdCb3gsIExheW91dFJlY3Q6
OmluZmluaXRlUmVjdCgpKTsKICAgICAgICAgZm9yIChzaXplX3QgaSA9IDA7IGkgPCBmcmFnbWVu
dHMuc2l6ZSgpOyArK2kpIHsKICAgICAgICAgICAgIGNvbnN0IExheWVyRnJhZ21lbnQmIGZyYWdt
ZW50ID0gZnJhZ21lbnRzLmF0KGkpOwogICAgICAgICAgICAgTGF5b3V0UmVjdCBmcmFnbWVudFJl
Y3QobGF5ZXJCb3VuZGluZ0JveCk7CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJp
bmcvUmVuZGVyTGF5ZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxheWVy
LmNwcAppbmRleCA1NGM5MWJiLi4xODg5NjAxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9y
ZW5kZXJpbmcvUmVuZGVyTGF5ZXIuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9S
ZW5kZXJMYXllci5jcHAKQEAgLTQyODAsNyArNDI4MCw3IEBAIHZvaWQgUmVuZGVyTGF5ZXI6OmNv
bGxlY3RGcmFnbWVudHMoTGF5ZXJGcmFnbWVudHMmIGZyYWdtZW50cywgY29uc3QgUmVuZGVyTGF5
ZXIqCiAgICAgQ2xpcFJlY3QgYmFja2dyb3VuZFJlY3RJbkZsb3dUaHJlYWQ7CiAgICAgQ2xpcFJl
Y3QgZm9yZWdyb3VuZFJlY3RJbkZsb3dUaHJlYWQ7CiAgICAgQ2xpcFJlY3Qgb3V0bGluZVJlY3RJ
bkZsb3dUaHJlYWQ7Ci0gICAgY2FsY3VsYXRlUmVjdHMocGFnaW5hdGlvbkNsaXBSZWN0c0NvbnRl
eHQsIEludFJlY3Q6OmluZmluaXRlUmVjdCgpLCBsYXllckJvdW5kc0luRmxvd1RocmVhZCwgYmFj
a2dyb3VuZFJlY3RJbkZsb3dUaHJlYWQsIGZvcmVncm91bmRSZWN0SW5GbG93VGhyZWFkLAorICAg
IGNhbGN1bGF0ZVJlY3RzKHBhZ2luYXRpb25DbGlwUmVjdHNDb250ZXh0LCBMYXlvdXRSZWN0Ojpp
bmZpbml0ZVJlY3QoKSwgbGF5ZXJCb3VuZHNJbkZsb3dUaHJlYWQsIGJhY2tncm91bmRSZWN0SW5G
bG93VGhyZWFkLCBmb3JlZ3JvdW5kUmVjdEluRmxvd1RocmVhZCwKICAgICAgICAgb3V0bGluZVJl
Y3RJbkZsb3dUaHJlYWQsICZvZmZzZXRXaXRoaW5QYWdpbmF0ZWRMYXllcik7CiAgICAgCiAgICAg
Ly8gVGFrZSBvdXIgYm91bmRpbmcgYm94IHdpdGhpbiB0aGUgZmxvdyB0aHJlYWQgYW5kIGNsaXAg
aXQuCkBAIC01MzQ2LDcgKzUzNDYsNyBAQCB2b2lkIFJlbmRlckxheWVyOjpjYWxjdWxhdGVDbGlw
UmVjdHMoY29uc3QgQ2xpcFJlY3RzQ29udGV4dCYgY2xpcFJlY3RzQ29udGV4dCwgQwogewogICAg
IGlmICghcGFyZW50KCkpIHsKICAgICAgICAgLy8gVGhlIHJvb3QgbGF5ZXIncyBjbGlwIHJlY3Qg
aXMgYWx3YXlzIGluZmluaXRlLgotICAgICAgICBjbGlwUmVjdHMucmVzZXQoSW50UmVjdDo6aW5m
aW5pdGVSZWN0KCkpOworICAgICAgICBjbGlwUmVjdHMucmVzZXQoTGF5b3V0UmVjdDo6aW5maW5p
dGVSZWN0KCkpOwogICAgICAgICByZXR1cm47CiAgICAgfQogICAgIApAQCAtNTM3Miw3ICs1Mzcy
LDcgQEAgdm9pZCBSZW5kZXJMYXllcjo6Y2FsY3VsYXRlQ2xpcFJlY3RzKGNvbnN0IENsaXBSZWN0
c0NvbnRleHQmIGNsaXBSZWN0c0NvbnRleHQsIEMKICAgICAgICAgICAgIHBhcmVudExheWVyLT5j
YWxjdWxhdGVDbGlwUmVjdHMocGFyZW50Q29udGV4dCwgY2xpcFJlY3RzKTsKICAgICAgICAgfQog
ICAgIH0gZWxzZQotICAgICAgICBjbGlwUmVjdHMucmVzZXQoSW50UmVjdDo6aW5maW5pdGVSZWN0
KCkpOworICAgICAgICBjbGlwUmVjdHMucmVzZXQoTGF5b3V0UmVjdDo6aW5maW5pdGVSZWN0KCkp
OwogCiAgICAgLy8gQSBmaXhlZCBvYmplY3QgaXMgZXNzZW50aWFsbHkgdGhlIHJvb3Qgb2YgaXRz
IGNvbnRhaW5pbmcgYmxvY2sgaGllcmFyY2h5LCBzbyB3aGVuCiAgICAgLy8gd2UgZW5jb3VudGVy
IHN1Y2ggYW4gb2JqZWN0LCB3ZSByZXNldCBvdXIgY2xpcCByZWN0cyB0byB0aGUgZml4ZWRDbGlw
UmVjdC4KQEAgLTU0NjUsNyArNTQ2NSw3IEBAIENsaXBSZWN0IFJlbmRlckxheWVyOjpiYWNrZ3Jv
dW5kQ2xpcFJlY3QoY29uc3QgQ2xpcFJlY3RzQ29udGV4dCYgY2xpcFJlY3RzQ29udGV4CiAgICAg
UmVuZGVyVmlldyYgdmlldyA9IHJlbmRlcmVyKCkudmlldygpOwogCiAgICAgLy8gTm90ZTogaW5m
aW5pdGUgY2xpcFJlY3RzIHNob3VsZCBub3QgYmUgc2Nyb2xsZWQgaGVyZSwgb3RoZXJ3aXNlIHRo
ZXkgd2lsbCBhY2NpZGVudGFsbHkgbm8gbG9uZ2VyIGJlIGNvbnNpZGVyZWQgaW5maW5pdGUuCi0g
ICAgaWYgKHBhcmVudFJlY3RzLmZpeGVkKCkgJiYgJmNsaXBSZWN0c0NvbnRleHQucm9vdExheWVy
LT5yZW5kZXJlcigpID09ICZ2aWV3ICYmIGJhY2tncm91bmRDbGlwUmVjdCAhPSBJbnRSZWN0Ojpp
bmZpbml0ZVJlY3QoKSkKKyAgICBpZiAocGFyZW50UmVjdHMuZml4ZWQoKSAmJiAmY2xpcFJlY3Rz
Q29udGV4dC5yb290TGF5ZXItPnJlbmRlcmVyKCkgPT0gJnZpZXcgJiYgYmFja2dyb3VuZENsaXBS
ZWN0ICE9IExheW91dFJlY3Q6OmluZmluaXRlUmVjdCgpKQogICAgICAgICBiYWNrZ3JvdW5kQ2xp
cFJlY3QubW92ZSh2aWV3LmZyYW1lVmlldygpLnNjcm9sbE9mZnNldEZvckZpeGVkUG9zaXRpb24o
KSk7CiAKICAgICByZXR1cm4gYmFja2dyb3VuZENsaXBSZWN0OwpAQCAtNTU5NiwxMCArNTU5Niwx
MCBAQCBMYXlvdXRSZWN0IFJlbmRlckxheWVyOjpsb2NhbENsaXBSZWN0KCkgY29uc3QKICAgICBM
YXlvdXRSZWN0IGxheWVyQm91bmRzOwogICAgIENsaXBSZWN0IGJhY2tncm91bmRSZWN0LCBmb3Jl
Z3JvdW5kUmVjdCwgb3V0bGluZVJlY3Q7CiAgICAgQ2xpcFJlY3RzQ29udGV4dCBjbGlwUmVjdHND
b250ZXh0KGNsaXBwaW5nUm9vdExheWVyLCAwLCBQYWludGluZ0NsaXBSZWN0cyk7Ci0gICAgY2Fs
Y3VsYXRlUmVjdHMoY2xpcFJlY3RzQ29udGV4dCwgSW50UmVjdDo6aW5maW5pdGVSZWN0KCksIGxh
eWVyQm91bmRzLCBiYWNrZ3JvdW5kUmVjdCwgZm9yZWdyb3VuZFJlY3QsIG91dGxpbmVSZWN0KTsK
KyAgICBjYWxjdWxhdGVSZWN0cyhjbGlwUmVjdHNDb250ZXh0LCBMYXlvdXRSZWN0OjppbmZpbml0
ZVJlY3QoKSwgbGF5ZXJCb3VuZHMsIGJhY2tncm91bmRSZWN0LCBmb3JlZ3JvdW5kUmVjdCwgb3V0
bGluZVJlY3QpOwogCiAgICAgTGF5b3V0UmVjdCBjbGlwUmVjdCA9IGJhY2tncm91bmRSZWN0LnJl
Y3QoKTsKLSAgICBpZiAoY2xpcFJlY3QgPT0gSW50UmVjdDo6aW5maW5pdGVSZWN0KCkpCisgICAg
aWYgKGNsaXBSZWN0ID09IExheW91dFJlY3Q6OmluZmluaXRlUmVjdCgpKQogICAgICAgICByZXR1
cm4gY2xpcFJlY3Q7CiAKICAgICBMYXlvdXRQb2ludCBjbGlwcGluZ1Jvb3RPZmZzZXQ7CkBAIC01
NzkyLDcgKzU3OTIsNyBAQCBMYXlvdXRSZWN0IFJlbmRlckxheWVyOjpjYWxjdWxhdGVMYXllckJv
dW5kcyhjb25zdCBSZW5kZXJMYXllciogYW5jZXN0b3JMYXllciwgYwogCiAgICAgaWYgKGZsYWdz
ICYgVXNlTG9jYWxDbGlwUmVjdElmUG9zc2libGUpIHsKICAgICAgICAgTGF5b3V0UmVjdCBsb2Nh
bENsaXBSZWN0ID0gdGhpcy0+bG9jYWxDbGlwUmVjdCgpOwotICAgICAgICBpZiAobG9jYWxDbGlw
UmVjdCAhPSBJbnRSZWN0OjppbmZpbml0ZVJlY3QoKSkgeworICAgICAgICBpZiAobG9jYWxDbGlw
UmVjdCAhPSBMYXlvdXRSZWN0OjppbmZpbml0ZVJlY3QoKSkgewogICAgICAgICAgICAgaWYgKChm
bGFncyAmIEluY2x1ZGVTZWxmVHJhbnNmb3JtKSAmJiBwYWludHNXaXRoVHJhbnNmb3JtKFBhaW50
QmVoYXZpb3JOb3JtYWwpKQogICAgICAgICAgICAgICAgIGxvY2FsQ2xpcFJlY3QgPSB0cmFuc2Zv
cm0oKS0+bWFwUmVjdChsb2NhbENsaXBSZWN0KTsKIApAQCAtNjk1NSwxMiArNjk1NSwxMiBAQCB2
b2lkIFJlbmRlckxheWVyOjpwYWludEZsb3dUaHJlYWRJZlJlZ2lvbihHcmFwaGljc0NvbnRleHQq
IGNvbnRleHQsIGNvbnN0IExheWVyUAogICAgIH0KIAogICAgIC8vIE9wdGltaXplIGNsaXBwaW5n
IGZvciB0aGUgc2luZ2xlIGZyYWdtZW50IGNhc2UuCi0gICAgaWYgKCFyZWdpb25DbGlwUmVjdC5p
c0VtcHR5KCkgJiYgcmVnaW9uQ2xpcFJlY3QgIT0gSW50UmVjdDo6aW5maW5pdGVSZWN0KCkpCisg
ICAgaWYgKCFyZWdpb25DbGlwUmVjdC5pc0VtcHR5KCkgJiYgcmVnaW9uQ2xpcFJlY3QgIT0gTGF5
b3V0UmVjdDo6aW5maW5pdGVSZWN0KCkpCiAgICAgICAgIGNsaXBUb1JlY3QocGFpbnRpbmdJbmZv
LnJvb3RMYXllciwgY29udGV4dCwgcGFpbnRpbmdJbmZvLnBhaW50RGlydHlSZWN0LCByZWdpb25D
bGlwUmVjdCk7CiAKICAgICBmbG93VGhyZWFkTGF5ZXItPnBhaW50TmFtZWRGbG93VGhyZWFkSW5z
aWRlUmVnaW9uKGNvbnRleHQsIHJlZ2lvbiwgcGFpbnRpbmdJbmZvLnBhaW50RGlydHlSZWN0LCBw
YWludE9mZnNldCwgcGFpbnRpbmdJbmZvLnBhaW50QmVoYXZpb3IsIHBhaW50RmxhZ3MpOwogCi0g
ICAgaWYgKCFyZWdpb25DbGlwUmVjdC5pc0VtcHR5KCkgJiYgcmVnaW9uQ2xpcFJlY3QgIT0gSW50
UmVjdDo6aW5maW5pdGVSZWN0KCkpCisgICAgaWYgKCFyZWdpb25DbGlwUmVjdC5pc0VtcHR5KCkg
JiYgcmVnaW9uQ2xpcFJlY3QgIT0gTGF5b3V0UmVjdDo6aW5maW5pdGVSZWN0KCkpCiAgICAgICAg
IHJlc3RvcmVDbGlwKGNvbnRleHQsIHBhaW50aW5nSW5mby5wYWludERpcnR5UmVjdCwgcmVnaW9u
Q2xpcFJlY3QpOwogfQogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVu
ZGVyTGF5ZXJCYWNraW5nLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXll
ckJhY2tpbmcuY3BwCmluZGV4IDk0MGVlMTMuLjAxNzg5NzUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckJhY2tpbmcuY3BwCisrKyBiL1NvdXJjZS9XZWJD
b3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckJhY2tpbmcuY3BwCkBAIC02NDYsNyArNjQ2LDcgQEAg
Ym9vbCBSZW5kZXJMYXllckJhY2tpbmc6OnVwZGF0ZUdyYXBoaWNzTGF5ZXJDb25maWd1cmF0aW9u
KCkKIAogc3RhdGljIEludFJlY3QgY2xpcEJveChSZW5kZXJCb3gmIHJlbmRlcmVyKQogewotICAg
IExheW91dFJlY3QgcmVzdWx0ID0gSW50UmVjdDo6aW5maW5pdGVSZWN0KCk7CisgICAgTGF5b3V0
UmVjdCByZXN1bHQgPSBMYXlvdXRSZWN0OjppbmZpbml0ZVJlY3QoKTsKICAgICBpZiAocmVuZGVy
ZXIuaGFzT3ZlcmZsb3dDbGlwKCkpCiAgICAgICAgIHJlc3VsdCA9IHJlbmRlcmVyLm92ZXJmbG93
Q2xpcFJlY3QoTGF5b3V0UG9pbnQoKSwgMCk7IC8vIEZJWE1FOiBJbmNvcnJlY3QgZm9yIENTUyBy
ZWdpb25zLgogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5
ZXJDb21wb3NpdG9yLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckNv
bXBvc2l0b3IuY3BwCmluZGV4IGMyZjZiNmQuLjRmODBkMzggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckNvbXBvc2l0b3IuY3BwCisrKyBiL1NvdXJjZS9X
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckNvbXBvc2l0b3IuY3BwCkBAIC0yMTk1LDcgKzIx
OTUsNyBAQCBib29sIFJlbmRlckxheWVyQ29tcG9zaXRvcjo6Y2xpcHBlZEJ5QW5jZXN0b3IoUmVu
ZGVyTGF5ZXImIGxheWVyKSBjb25zdAogICAgIGlmICghY29tcHV0ZUNsaXBSb290IHx8IGNvbXB1
dGVDbGlwUm9vdCA9PSAmbGF5ZXIpCiAgICAgICAgIHJldHVybiBmYWxzZTsKIAotICAgIHJldHVy
biBsYXllci5iYWNrZ3JvdW5kQ2xpcFJlY3QoUmVuZGVyTGF5ZXI6OkNsaXBSZWN0c0NvbnRleHQo
Y29tcHV0ZUNsaXBSb290LCAwLCBUZW1wb3JhcnlDbGlwUmVjdHMpKS5yZWN0KCkgIT0gSW50UmVj
dDo6aW5maW5pdGVSZWN0KCk7IC8vIEZJWE1FOiBJbmNvcnJlY3QgZm9yIENTUyByZWdpb25zLgor
ICAgIHJldHVybiBsYXllci5iYWNrZ3JvdW5kQ2xpcFJlY3QoUmVuZGVyTGF5ZXI6OkNsaXBSZWN0
c0NvbnRleHQoY29tcHV0ZUNsaXBSb290LCAwLCBUZW1wb3JhcnlDbGlwUmVjdHMpKS5yZWN0KCkg
IT0gTGF5b3V0UmVjdDo6aW5maW5pdGVSZWN0KCk7IC8vIEZJWE1FOiBJbmNvcnJlY3QgZm9yIENT
UyByZWdpb25zLgogfQogCiAvLyBSZXR1cm4gdHJ1ZSBpZiB0aGUgZ2l2ZW4gbGF5ZXIgaXMgYSBz
dGFja2luZyBjb250ZXh0IGFuZCBoYXMgY29tcG9zaXRpbmcgY2hpbGQK
</data>
<flag name="review"
          id="244258"
          type_id="1"
          status="+"
          setter="thorton"
    />
          </attachment>
      

    </bug>

</bugzilla>