<?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>13793</bug_id>
          
          <creation_ts>2007-05-20 23:18:38 -0700</creation_ts>
          <short_desc>RULES=groups overridden by border, colgroup=span not honored by border collapsing</short_desc>
          <delta_ts>2007-05-28 06:25:12 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tables</component>
          <version>523.x (Safari 3)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://www.htmlcodetutorial.com/tables/_TABLE_RULES_GROUPS.html</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="Brady Eidson">beidson</reporter>
          <assigned_to name="Dave Hyatt">hyatt</assigned_to>
          <cc>ddkilzer</cc>
    
    <cc>mitz</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>9537</commentid>
    <comment_count>0</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2007-05-20 23:18:38 -0700</bug_when>
    <thetext>Look at the url http://www.htmlcodetutorial.com/tables/_TABLE_RULES_GROUPS.html in both ToT WebKit and Firefox, scroll to the bottom.

There is a discussion about colgroup with span.  Firefox supports it properly, but ToT WebKit puts a border between each column instead of the specified groups</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9534</commentid>
    <comment_count>1</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2007-05-20 23:20:27 -0700</bug_when>
    <thetext>Reduction taken from that page - 
&lt;TABLE BORDER=1 CELLPADDING=4 RULES=GROUPS  FRAME=BOX&gt;

&lt;COLGROUP&gt;&lt;/COLGROUP&gt;
&lt;COLGROUP SPAN=3&gt;&lt;/COLGROUP&gt;

&lt;TR&gt; &lt;TH&gt;Weekday&lt;/TH&gt; &lt;TH&gt;Date&lt;/TH&gt;    &lt;TH&gt;Manager&lt;/TH&gt; &lt;TH&gt;Qty&lt;/TH&gt; &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Monday&lt;/TD&gt; &lt;TD&gt;09/11&lt;/TD&gt;    &lt;TD&gt;Kelsey&lt;/TD&gt; &lt;TD&gt;639&lt;/TD&gt;  &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Tuesday&lt;/TD&gt; &lt;TD&gt;09/12&lt;/TD&gt;   &lt;TD&gt;Lindsey&lt;/TD&gt; &lt;TD&gt;596&lt;/TD&gt; &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Wednesday&lt;/TD&gt; &lt;TD&gt;09/13&lt;/TD&gt; &lt;TD&gt;Randy&lt;/TD&gt; &lt;TD&gt;1135&lt;/TD&gt;  &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Thursday&lt;/TD&gt; &lt;TD&gt;09/14&lt;/TD&gt;  &lt;TD&gt;Susan&lt;/TD&gt; &lt;TD&gt;1002&lt;/TD&gt;  &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Friday&lt;/TD&gt; &lt;TD&gt;09/15&lt;/TD&gt;    &lt;TD&gt;Randy&lt;/TD&gt; &lt;TD&gt;908&lt;/TD&gt;   &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Saturday&lt;/TD&gt; &lt;TD&gt;09/16&lt;/TD&gt;  &lt;TD&gt;Lindsey&lt;/TD&gt; &lt;TD&gt;371&lt;/TD&gt; &lt;/TR&gt;
&lt;TR&gt; &lt;TD&gt;Sunday&lt;/TD&gt; &lt;TD&gt;09/17&lt;/TD&gt;    &lt;TD&gt;Susan&lt;/TD&gt; &lt;TD&gt;272&lt;/TD&gt;   &lt;/TR&gt;
&lt;/TABLE&gt;
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9523</commentid>
    <comment_count>2</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-20 23:32:58 -0700</bug_when>
    <thetext>I assume this is incorrect but will need to verify.  The bug is that border= on table is causing every cell to get a border, without respect for the rules attribute.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9524</commentid>
    <comment_count>3</comment_count>
    <who name="">mitz</who>
    <bug_when>2007-05-20 23:35:05 -0700</bug_when>
    <thetext>Another thing that I notice here is that the default colgroup span in WebKit is 0, whereas in the spec and in Firefox it is 1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9525</commentid>
    <comment_count>4</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-20 23:44:28 -0700</bug_when>
    <thetext>There are two bugs here.  RULES=groups is improperly being ignored by border= on a table.  colgroup span is not honored when doing border collapsing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9526</commentid>
    <comment_count>5</comment_count>
      <attachid>14639</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-20 23:47:01 -0700</bug_when>
    <thetext>Created attachment 14639
Patch to fix first issue.

Two of the Mozilla tables tests in the expected failures directory now pass with this bug fix.  No other tests are affected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9527</commentid>
    <comment_count>6</comment_count>
      <attachid>14639</attachid>
    <who name="">mitz</who>
    <bug_when>2007-05-20 23:50:18 -0700</bug_when>
    <thetext>Comment on attachment 14639
Patch to fix first issue.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9529</commentid>
    <comment_count>7</comment_count>
      <attachid>14640</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-21 00:37:50 -0700</bug_when>
    <thetext>Created attachment 14640
Patch to fix second issue</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9531</commentid>
    <comment_count>8</comment_count>
      <attachid>14640</attachid>
    <who name="">mitz</who>
    <bug_when>2007-05-21 00:42:48 -0700</bug_when>
    <thetext>Comment on attachment 14640
Patch to fix second issue

r=me. you can remove the {start,end}ColEdge = false; lines</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9532</commentid>
    <comment_count>9</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-21 01:09:23 -0700</bug_when>
    <thetext>Fixed.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9515</commentid>
    <comment_count>10</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2007-05-21 07:21:54 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; Created an attachment (id=14639) [edit]
&gt; Patch to fix first issue.
&gt; 
&gt; Two of the Mozilla tables tests in the expected failures directory now pass
&gt; with this bug fix.  No other tests are affected.

Shouldn&apos;t these tests be svn-moved to the non-failing directory if they now pass?

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9027</commentid>
    <comment_count>11</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2007-05-27 09:59:39 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; Fixed.

Code: r21614, r21615
Tests: r21616, r21617

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9032</commentid>
    <comment_count>12</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2007-05-27 12:06:01 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; (In reply to comment #5)
&gt; &gt; Two of the Mozilla tables tests in the expected failures directory now pass
&gt; &gt; with this bug fix.  No other tests are affected.
&gt; Shouldn&apos;t these tests be svn-moved to the non-failing directory if they now
&gt; pass?

Actually three tests now pass when compared to Firefox 2.0.0.3 and 3.0a5pre (from 27-May-2007 5:24 AM):

LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct.html
LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px.html
LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct.xml

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>8960</commentid>
    <comment_count>13</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2007-05-28 06:25:12 -0700</bug_when>
    <thetext>$ svn commit LayoutTestResults LayoutTests/ChangeLog LayoutTests/tables
Sending        LayoutTestResults/ChangeLog
Adding         LayoutTestResults/qt/tables/mozilla/marvin/colgroup_width_pct-expected.txt
Adding         LayoutTestResults/qt/tables/mozilla/marvin/colgroup_width_px-expected.txt
Adding         LayoutTestResults/qt/tables/mozilla/marvin/x_colgroup_width_pct-expected.txt
Deleting       LayoutTestResults/qt/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.txt
Deleting       LayoutTestResults/qt/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.txt
Deleting       LayoutTestResults/qt/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.txt
Sending        LayoutTests/ChangeLog
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_pct-expected.checksum
Adding  (bin)  LayoutTests/tables/mozilla/marvin/colgroup_width_pct-expected.png
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_pct-expected.txt
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_pct.html
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_px-expected.checksum
Adding  (bin)  LayoutTests/tables/mozilla/marvin/colgroup_width_px-expected.png
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_px-expected.txt
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_px.html
Adding         LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct-expected.checksum
Adding  (bin)  LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct-expected.png
Adding         LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct-expected.txt
Adding         LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct.xml
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.checksum
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.png
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.txt
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct.html
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.checksum
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.png
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.txt
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px.html
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.checksum
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.png
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.txt
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct.xml
Transmitting file data ..
Committed revision 21835.

</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>14639</attachid>
            <date>2007-05-20 23:47:01 -0700</date>
            <delta_ts>2007-05-20 23:50:18 -0700</delta_ts>
            <desc>Patch to fix first issue.</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>1572</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">SW5kZXg6IGh0bWwvSFRNTFRhYmxlRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gaHRtbC9IVE1M
VGFibGVFbGVtZW50LmNwcAkocmV2aXNpb24gMjE1OTcpCisrKyBodG1sL0hUTUxUYWJsZUVsZW1l
bnQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC01MjYsNyArNTI2LDcgQEAgQ1NTTXV0YWJsZVN0eWxl
RGVjbGFyYXRpb24qIEhUTUxUYWJsZUVsZQogewogICAgIE1hcHBlZEF0dHJpYnV0ZSBhdHRyKGNl
bGxib3JkZXJBdHRyLCBtX3J1bGVzQXR0ciA9PSBBbGxSdWxlcyA/ICJzb2xpZC1hbGwiIDogCiAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIChtX3J1bGVzQXR0ciA9PSBD
b2xzUnVsZXMgPyAic29saWQtY29scyIgOiAKLSAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgKG1fcnVsZXNBdHRyID09IFJvd3NSdWxlcyA/ICJzb2xpZC1yb3dzIiA6ICgh
bV9ib3JkZXJBdHRyID8gIm5vbmUiIDogKG1fYm9yZGVyQ29sb3JBdHRyID8gInNvbGlkIiA6ICJp
bnNldCIpKSkpKTsKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgKG1f
cnVsZXNBdHRyID09IFJvd3NSdWxlcyA/ICJzb2xpZC1yb3dzIiA6ICghbV9ib3JkZXJBdHRyIHx8
IG1fcnVsZXNBdHRyID09IEdyb3Vwc1J1bGVzID8gIm5vbmUiIDogKG1fYm9yZGVyQ29sb3JBdHRy
ID8gInNvbGlkIiA6ICJpbnNldCIpKSkpKTsKIAogICAgIENTU01hcHBlZEF0dHJpYnV0ZURlY2xh
cmF0aW9uKiBkZWNsID0gZ2V0TWFwcGVkQXR0cmlidXRlRGVjbChlUGVyc2lzdGVudCwgJmF0dHIp
OwogICAgIGlmICghZGVjbCkgewpAQCAtNTQ5LDcgKzU0OSw3IEBAIENTU011dGFibGVTdHlsZURl
Y2xhcmF0aW9uKiBIVE1MVGFibGVFbGUKICAgICAgICAgICAgIGRlY2wtPnNldFByb3BlcnR5KENT
U19QUk9QX0JPUkRFUl9UT1BfU1RZTEUsIENTU19WQUxfU09MSUQsIGZhbHNlKTsKICAgICAgICAg
ICAgIGRlY2wtPnNldFByb3BlcnR5KENTU19QUk9QX0JPUkRFUl9CT1RUT01fU1RZTEUsIENTU19W
QUxfU09MSUQsIGZhbHNlKTsKICAgICAgICAgICAgIGRlY2wtPnNldFByb3BlcnR5KENTU19QUk9Q
X0JPUkRFUl9DT0xPUiwgImluaGVyaXQiLCBmYWxzZSk7Ci0gICAgICAgIH0gZWxzZSBpZiAobV9i
b3JkZXJBdHRyIHx8IG1fcnVsZXNBdHRyID09IEFsbFJ1bGVzKSB7CisgICAgICAgIH0gZWxzZSBp
ZiAobV9ydWxlc0F0dHIgIT0gR3JvdXBzUnVsZXMgJiYgKG1fYm9yZGVyQXR0ciB8fCBtX3J1bGVz
QXR0ciA9PSBBbGxSdWxlcykpIHsKICAgICAgICAgICAgIGRlY2wtPnNldFByb3BlcnR5KENTU19Q
Uk9QX0JPUkRFUl9XSURUSCwgIjFweCIsIGZhbHNlKTsKICAgICAgICAgICAgICBpbnQgdiA9ICht
X2JvcmRlckNvbG9yQXR0ciB8fCBtX3J1bGVzQXR0ciA9PSBBbGxSdWxlcykgPyBDU1NfVkFMX1NP
TElEIDogQ1NTX1ZBTF9JTlNFVDsKICAgICAgICAgICAgIGRlY2wtPnNldFByb3BlcnR5KENTU19Q
Uk9QX0JPUkRFUl9UT1BfU1RZTEUsIHYsIGZhbHNlKTsK
</data>
<flag name="review"
          id="6008"
          type_id="1"
          status="+"
          setter="mitz"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>14640</attachid>
            <date>2007-05-21 00:37:50 -0700</date>
            <delta_ts>2007-05-21 00:42:48 -0700</delta_ts>
            <desc>Patch to fix second issue</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>5055</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">SW5kZXg6IGh0bWwvSFRNTFRhYmxlQ29sRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gaHRtbC9I
VE1MVGFibGVDb2xFbGVtZW50LmNwcAkocmV2aXNpb24gMjE1OTcpCisrKyBodG1sL0hUTUxUYWJs
ZUNvbEVsZW1lbnQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zOCw3ICszOCw3IEBAIHVzaW5nIG5h
bWVzcGFjZSBIVE1MTmFtZXM7CiBIVE1MVGFibGVDb2xFbGVtZW50OjpIVE1MVGFibGVDb2xFbGVt
ZW50KGNvbnN0IFF1YWxpZmllZE5hbWUmIHRhZ05hbWUsIERvY3VtZW50ICpkb2MpCiAgICAgOiBI
VE1MVGFibGVQYXJ0RWxlbWVudCh0YWdOYW1lLCBkb2MpCiB7Ci0gICAgX3NwYW4gPSAodGFnTmFt
ZS5tYXRjaGVzKGNvbGdyb3VwVGFnKSA/IDAgOiAxKTsKKyAgICBfc3BhbiA9IDE7CiB9CiAKIEhU
TUxUYWdTdGF0dXMgSFRNTFRhYmxlQ29sRWxlbWVudDo6ZW5kVGFnUmVxdWlyZW1lbnQoKSBjb25z
dApJbmRleDogcmVuZGVyaW5nL1JlbmRlclRhYmxlLmNwcAo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJp
bmcvUmVuZGVyVGFibGUuY3BwCShyZXZpc2lvbiAyMTU5NykKKysrIHJlbmRlcmluZy9SZW5kZXJU
YWJsZS5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTU3MCw3ICs1NzAsNyBAQCB2b2lkIFJlbmRlclRh
YmxlOjphcHBlbmRDb2x1bW4oaW50IHNwYW4pCiAgICAgc2V0TmVlZHNMYXlvdXRBbmRQcmVmV2lk
dGhzUmVjYWxjKCk7CiB9CiAKLVJlbmRlclRhYmxlQ29sKiBSZW5kZXJUYWJsZTo6Y29sRWxlbWVu
dChpbnQgY29sKSBjb25zdAorUmVuZGVyVGFibGVDb2wqIFJlbmRlclRhYmxlOjpjb2xFbGVtZW50
KGludCBjb2wsIGJvb2wqIHN0YXJ0RWRnZSwgYm9vbCogZW5kRWRnZSkgY29uc3QKIHsKICAgICBp
ZiAoIW1faGFzQ29sRWxlbWVudHMpCiAgICAgICAgIHJldHVybiAwOwpAQCAtNTgyLDkgKzU4Miwx
NiBAQCBSZW5kZXJUYWJsZUNvbCogUmVuZGVyVGFibGU6OmNvbEVsZW1lbnQoCiAgICAgICAgICAg
ICBSZW5kZXJUYWJsZUNvbCogY29sRWxlbSA9IHN0YXRpY19jYXN0PFJlbmRlclRhYmxlQ29sKj4o
Y2hpbGQpOwogICAgICAgICAgICAgaW50IHNwYW4gPSBjb2xFbGVtLT5zcGFuKCk7CiAgICAgICAg
ICAgICBpZiAoIWNvbEVsZW0tPmZpcnN0Q2hpbGQoKSkgeworICAgICAgICAgICAgICAgIGludCBz
dGFydENvbCA9IGNDb2w7CisgICAgICAgICAgICAgICAgaW50IGVuZENvbCA9IGNDb2wgKyBzcGFu
IC0gMTsKICAgICAgICAgICAgICAgICBjQ29sICs9IHNwYW47Ci0gICAgICAgICAgICAgICAgaWYg
KGNDb2wgPiBjb2wpCisgICAgICAgICAgICAgICAgaWYgKGNDb2wgPiBjb2wpIHsKKyAgICAgICAg
ICAgICAgICAgICAgaWYgKHN0YXJ0RWRnZSkKKyAgICAgICAgICAgICAgICAgICAgICAgICpzdGFy
dEVkZ2UgPSBzdGFydENvbCA9PSBjb2w7CisgICAgICAgICAgICAgICAgICAgIGlmIChlbmRFZGdl
KQorICAgICAgICAgICAgICAgICAgICAgICAgKmVuZEVkZ2UgPSBlbmRDb2wgPT0gY29sOwogICAg
ICAgICAgICAgICAgICAgICByZXR1cm4gY29sRWxlbTsKKyAgICAgICAgICAgICAgICB9CiAgICAg
ICAgICAgICB9CiAKICAgICAgICAgICAgIFJlbmRlck9iamVjdCogbmV4dCA9IGNoaWxkLT5maXJz
dENoaWxkKCk7CkluZGV4OiByZW5kZXJpbmcvUmVuZGVyVGFibGUuaAo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBy
ZW5kZXJpbmcvUmVuZGVyVGFibGUuaAkocmV2aXNpb24gMjE1OTcpCisrKyByZW5kZXJpbmcvUmVu
ZGVyVGFibGUuaAkod29ya2luZyBjb3B5KQpAQCAtMTYwLDcgKzE2MCw3IEBAIHB1YmxpYzoKICAg
ICAgICAgICAgICAgIChjb2xsYXBzZUJvcmRlcnMoKSA/IDAgOiAocGFkZGluZ0xlZnQoKSArIHBh
ZGRpbmdSaWdodCgpICsgKG51bUVmZkNvbHMoKSArIDEpICogaEJvcmRlclNwYWNpbmcoKSkpOwog
ICAgIH0KIAotICAgIFJlbmRlclRhYmxlQ29sKiBjb2xFbGVtZW50KGludCBjb2wpIGNvbnN0Owor
ICAgIFJlbmRlclRhYmxlQ29sKiBjb2xFbGVtZW50KGludCBjb2wsIGJvb2wqIGxlZnRFZGdlID0g
MCwgYm9vbCogcmlnaHRFZGdlID0gMCkgY29uc3Q7CiAKICAgICBib29sIG5lZWRzU2VjdGlvblJl
Y2FsYygpIGNvbnN0IHsgcmV0dXJuIG1fbmVlZHNTZWN0aW9uUmVjYWxjOyB9CiAgICAgdm9pZCBz
ZXROZWVkc1NlY3Rpb25SZWNhbGMoKSB7IG1fbmVlZHNTZWN0aW9uUmVjYWxjID0gdHJ1ZTsgfQpJ
bmRleDogcmVuZGVyaW5nL1JlbmRlclRhYmxlQ2VsbC5jcHAKPT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gcmVuZGVy
aW5nL1JlbmRlclRhYmxlQ2VsbC5jcHAJKHJldmlzaW9uIDIxNTk3KQorKysgcmVuZGVyaW5nL1Jl
bmRlclRhYmxlQ2VsbC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTMyOSw4ICszMjksMTAgQEAgQ29s
bGFwc2VkQm9yZGVyVmFsdWUgUmVuZGVyVGFibGVDZWxsOjpjbwogICAgIH0KICAgICAKICAgICAv
LyAoNSkgT3VyIGNvbHVtbidzIGxlZnQgYm9yZGVyLgotICAgIFJlbmRlclRhYmxlQ29sKiBjb2xF
bHQgPSB0YWJsZUVsdC0+Y29sRWxlbWVudChjb2woKSArIChydGwgPyBjb2xTcGFuKCkgLSAxIDog
MCkpOwotICAgIGlmIChjb2xFbHQpIHsKKyAgICBib29sIHN0YXJ0Q29sRWRnZSA9IGZhbHNlOwor
ICAgIGJvb2wgZW5kQ29sRWRnZSA9IGZhbHNlOworICAgIFJlbmRlclRhYmxlQ29sKiBjb2xFbHQg
PSB0YWJsZUVsdC0+Y29sRWxlbWVudChjb2woKSArIChydGwgPyBjb2xTcGFuKCkgLSAxIDogMCks
ICZzdGFydENvbEVkZ2UsICZlbmRDb2xFZGdlKTsKKyAgICBpZiAoY29sRWx0ICYmICghcnRsID8g
c3RhcnRDb2xFZGdlIDogZW5kQ29sRWRnZSkpIHsKICAgICAgICAgcmVzdWx0ID0gY29tcGFyZUJv
cmRlcnMocmVzdWx0LCBDb2xsYXBzZWRCb3JkZXJWYWx1ZSgmY29sRWx0LT5zdHlsZSgpLT5ib3Jk
ZXJMZWZ0KCksIEJDT0wpKTsKICAgICAgICAgaWYgKCFyZXN1bHQuZXhpc3RzKCkpCiAgICAgICAg
ICAgICByZXR1cm4gcmVzdWx0OwpAQCAtMzM4LDggKzM0MCwxMCBAQCBDb2xsYXBzZWRCb3JkZXJW
YWx1ZSBSZW5kZXJUYWJsZUNlbGw6OmNvCiAgICAgCiAgICAgLy8gKDYpIFRoZSByaWdodCBib3Jk
ZXIgb2YgdGhlIGNvbHVtbiB0byB0aGUgbGVmdC4KICAgICBpZiAoIWxlZnRtb3N0Q29sdW1uKSB7
Ci0gICAgICAgIGNvbEVsdCA9IHRhYmxlRWx0LT5jb2xFbGVtZW50KGNvbCgpICsgKHJ0bCA/IGNv
bFNwYW4oKSA6IC0xKSk7Ci0gICAgICAgIGlmIChjb2xFbHQpIHsKKyAgICAgICAgc3RhcnRDb2xF
ZGdlID0gZmFsc2U7CisgICAgICAgIGVuZENvbEVkZ2UgPSBmYWxzZTsKKyAgICAgICAgY29sRWx0
ID0gdGFibGVFbHQtPmNvbEVsZW1lbnQoY29sKCkgKyAocnRsID8gY29sU3BhbigpIDogLTEpLCAm
c3RhcnRDb2xFZGdlLCAmZW5kQ29sRWRnZSk7CisgICAgICAgIGlmIChjb2xFbHQgJiYgKCFydGwg
PyBlbmRDb2xFZGdlIDogc3RhcnRDb2xFZGdlKSkgewogICAgICAgICAgICAgcmVzdWx0ID0gY29t
cGFyZUJvcmRlcnMocmVzdWx0LCBDb2xsYXBzZWRCb3JkZXJWYWx1ZSgmY29sRWx0LT5zdHlsZSgp
LT5ib3JkZXJSaWdodCgpLCBCQ09MKSk7CiAgICAgICAgICAgICBpZiAoIXJlc3VsdC5leGlzdHMo
KSkKICAgICAgICAgICAgICAgICByZXR1cm4gcmVzdWx0OwpAQCAtMzkwLDggKzM5NCwxMCBAQCBD
b2xsYXBzZWRCb3JkZXJWYWx1ZSBSZW5kZXJUYWJsZUNlbGw6OmNvCiAgICAgfQogICAgIAogICAg
IC8vICg1KSBPdXIgY29sdW1uJ3MgcmlnaHQgYm9yZGVyLgotICAgIFJlbmRlclRhYmxlQ29sKiBj
b2xFbHQgPSB0YWJsZUVsdC0+Y29sRWxlbWVudChjb2woKSArIChydGwgPyAwIDogY29sU3Bhbigp
IC0gMSkpOwotICAgIGlmIChjb2xFbHQpIHsKKyAgICBib29sIHN0YXJ0Q29sRWRnZSA9IGZhbHNl
OworICAgIGJvb2wgZW5kQ29sRWRnZSA9IGZhbHNlOworICAgIFJlbmRlclRhYmxlQ29sKiBjb2xF
bHQgPSB0YWJsZUVsdC0+Y29sRWxlbWVudChjb2woKSArIChydGwgPyAwIDogY29sU3BhbigpIC0g
MSksICZzdGFydENvbEVkZ2UsICZlbmRDb2xFZGdlKTsKKyAgICBpZiAoY29sRWx0ICYmICghcnRs
ID8gZW5kQ29sRWRnZSA6IHN0YXJ0Q29sRWRnZSkpIHsKICAgICAgICAgcmVzdWx0ID0gY29tcGFy
ZUJvcmRlcnMocmVzdWx0LCBDb2xsYXBzZWRCb3JkZXJWYWx1ZSgmY29sRWx0LT5zdHlsZSgpLT5i
b3JkZXJSaWdodCgpLCBCQ09MKSk7CiAgICAgICAgIGlmICghcmVzdWx0LmV4aXN0cygpKQogICAg
ICAgICAgICAgcmV0dXJuIHJlc3VsdDsKQEAgLTM5OSw4ICs0MDUsMTAgQEAgQ29sbGFwc2VkQm9y
ZGVyVmFsdWUgUmVuZGVyVGFibGVDZWxsOjpjbwogICAgIAogICAgIC8vICg2KSBUaGUgbGVmdCBi
b3JkZXIgb2YgdGhlIGNvbHVtbiB0byB0aGUgcmlnaHQuCiAgICAgaWYgKCFyaWdodG1vc3RDb2x1
bW4pIHsKLSAgICAgICAgY29sRWx0ID0gdGFibGVFbHQtPmNvbEVsZW1lbnQoY29sKCkgKyAocnRs
ID8gLTEgOiBjb2xTcGFuKCkpKTsKLSAgICAgICAgaWYgKGNvbEVsdCkgeworICAgICAgICBzdGFy
dENvbEVkZ2UgPSBmYWxzZTsKKyAgICAgICAgZW5kQ29sRWRnZSA9IGZhbHNlOworICAgICAgICBj
b2xFbHQgPSB0YWJsZUVsdC0+Y29sRWxlbWVudChjb2woKSArIChydGwgPyAtMSA6IGNvbFNwYW4o
KSksICZzdGFydENvbEVkZ2UsICZlbmRDb2xFZGdlKTsKKyAgICAgICAgaWYgKGNvbEVsdCAmJiAo
IXJ0bCA/IHN0YXJ0Q29sRWRnZSA6IGVuZENvbEVkZ2UpKSB7CiAgICAgICAgICAgICByZXN1bHQg
PSBjb21wYXJlQm9yZGVycyhyZXN1bHQsIENvbGxhcHNlZEJvcmRlclZhbHVlKCZjb2xFbHQtPnN0
eWxlKCktPmJvcmRlckxlZnQoKSwgQkNPTCkpOwogICAgICAgICAgICAgaWYgKCFyZXN1bHQuZXhp
c3RzKCkpCiAgICAgICAgICAgICAgICAgcmV0dXJuIHJlc3VsdDsK
</data>
<flag name="review"
          id="6009"
          type_id="1"
          status="+"
          setter="mitz"
    />
          </attachment>
      

    </bug>

</bugzilla>