<?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>9314</bug_id>
          
          <creation_ts>2006-06-05 10:04:51 -0700</creation_ts>
          <short_desc>Relative positioned block size doesnt update root layer size</short_desc>
          <delta_ts>2009-10-02 02:37:26 -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>Layout and Rendering</component>
          <version>420+</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://soccernet.espn.go.com/origin?cameFrom=section%3Fid%3Dworldcup</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="Sriram">sriram.yadavalli</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ian</cc>
    
    <cc>koivisto</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>mitz</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>44779</commentid>
    <comment_count>0</comment_count>
    <who name="Sriram">sriram.yadavalli</who>
    <bug_when>2006-06-05 10:04:51 -0700</bug_when>
    <thetext>If a render block is relative positioned, the root layer doesnt get the correct size. As a result, if the browser window is too small, some of the content is clipped.


To run this test case, reduce the Safari browser window size and load the page. A scroll bar should appear allowing the user to see the entire content. 

The original webpage (espn.com)works because espn does browser sniffing and for Safari, it doesnt include the rel positioned div element.

The fix is most probably in RenderLayer::updateLayerPosition(). When the child layer position gets updated, the parent layer size needs to be updated.

The bug regarding content clipping exists in IE too, but works on Firefox.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>44780</commentid>
    <comment_count>1</comment_count>
      <attachid>8715</attachid>
    <who name="Sriram">sriram.yadavalli</who>
    <bug_when>2006-06-05 10:07:42 -0700</bug_when>
    <thetext>Created attachment 8715
Reduced test case for the espn rel postioning issue</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>44792</commentid>
    <comment_count>2</comment_count>
      <attachid>8718</attachid>
    <who name="Sriram">sriram.yadavalli</who>
    <bug_when>2006-06-05 12:50:06 -0700</bug_when>
    <thetext>Created attachment 8718
This fix resizes the parent layer after children have been positioned</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>44830</commentid>
    <comment_count>3</comment_count>
      <attachid>8718</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-06 02:10:20 -0700</bug_when>
    <thetext>Comment on attachment 8718
This fix resizes the parent layer after children have been positioned

This change would increase the sizes of layers even when the child is out of flow.   That is incorrect and will lead to layers being way larger than they need to be.

The correct place to patch would be the rightmost/lowest/leftmostPosition methods that are used to determine your scrollable area.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45295</commentid>
    <comment_count>4</comment_count>
      <attachid>8802</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2006-06-10 21:17:32 -0700</bug_when>
    <thetext>Created attachment 8802
refined test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45296</commentid>
    <comment_count>5</comment_count>
      <attachid>8803</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2006-06-10 21:24:46 -0700</bug_when>
    <thetext>Created attachment 8803
patch

- Take relative positioning into account in leftmost/rightmost/lowestPosition() 
- Ignore zero width/height elements when calculating extremes. This matches firefox behavior (and prevent this patch from breaking any layout tests).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45297</commentid>
    <comment_count>6</comment_count>
      <attachid>8804</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2006-06-10 22:22:12 -0700</bug_when>
    <thetext>Created attachment 8804
patch

updated patch, do relative for leftmostPosition() as well</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45299</commentid>
    <comment_count>7</comment_count>
      <attachid>8804</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 02:36:19 -0700</bug_when>
    <thetext>Comment on attachment 8804
patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45300</commentid>
    <comment_count>8</comment_count>
      <attachid>8804</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 02:38:42 -0700</bug_when>
    <thetext>Comment on attachment 8804
patch

Actually hold up.  I think there&apos;s a better way to do this.  There are more mistakes than just the scroll position being wrong.

Basically the overflowWidth/Height/Left/Top members of RenderBlock are all wrong.  When these values are computed, they don&apos;t account for relative positioning.  This means we incorrectly think relpositioned objects might overflow (when they don&apos;t) and vice versa.

Rather than patching leftmost/rightmost/lowestPosition, I think we should patch the computation of m_overflowLeft/Top/Width/Height to factor in relative position offsets when examining children.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45301</commentid>
    <comment_count>9</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 02:42:54 -0700</bug_when>
    <thetext>This overflow-patching approach would also deal correctly with relative-positioned elements other than RenderFlows (like images).
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45302</commentid>
    <comment_count>10</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 02:46:42 -0700</bug_when>
    <thetext>cc&apos;ing mitz.  We basically need test cases for relative positioned elements that cause overflow (incorrectly) in the untranslated position but not in their translated position and vice versa.  We should have tests where the relative positioned objects are both inline and block and flows vs. replaced content.

We&apos;ll need to make sure that if we change the definition of overflow to account for relative positioning that the incremental repaint code still works properly when repainting the relpositioned child layers as a result of ancestor movement.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45303</commentid>
    <comment_count>11</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 02:59:06 -0700</bug_when>
    <thetext>Even if we don&apos;t patch overflowLeft/Top/Width/Height, the leftmost/Rightmost/LowestPosition calls have to be patched in basically all the places they grab kids and look at their x/y positions, and I think it&apos;s more important to fix this case where we don&apos;t show scrollbars when we should then to get all the edge cases right.  (The case where we show scrollbars when we shouldn&apos;t seems much more unlikely to occur in practice.)

So you could adjust your patch to apply the relativePositionOffset at the floating objects walk (have to account for floating objects like images being relatively positioned) in RenderBlock.cpp and the child walks in RenderFlow.cpp.

I would r+ a patch that is very similar to what you posted as long as it accounts for non-RenderFlow relative-positioned objects.  (Patch the places in leftmost/rightmost/lowestPosition that fetch children, rather than the RenderFlow return chokepoint.)
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45304</commentid>
    <comment_count>12</comment_count>
      <attachid>8805</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 03:06:40 -0700</bug_when>
    <thetext>Created attachment 8805
Test case that needs to be addressed via leftmost/Rightmost/lowestPosition changes.

Test case that shows the same problem (not showing scrollbars when we need to) when an image is relative positioned.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45305</commentid>
    <comment_count>13</comment_count>
      <attachid>8806</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 03:07:58 -0700</bug_when>
    <thetext>Created attachment 8806
Second test case that uses a floating image instead of an in-flow image.

Here&apos;s the same example, but with a floating image.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45306</commentid>
    <comment_count>14</comment_count>
      <attachid>8807</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-11 03:13:44 -0700</bug_when>
    <thetext>Created attachment 8807
Test Case that shows the opposite problem (does not have to be fixed to land).

Here&apos;s the reason that overflowWidth/Height/Left/Top have to be patched.  In this case we show scrollbars when we didn&apos;t need to.  Even if we patch left/right/lowest position, as long as overflowWidth/Height/LEft/Top are wrong, this bug will persist.

Note, though, that this bug is much less severe and in fact Firefox gets it wrong too.  It&apos;s unclear to me whether it even *is* wrong, since maybe you are supposed to go ahead and leave &quot;space&quot; for the spot in which the element participated untranslated in the layout of the object.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45455</commentid>
    <comment_count>15</comment_count>
      <attachid>8831</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2006-06-12 15:36:11 -0700</bug_when>
    <thetext>Created attachment 8831
updated patch

Moved the relative position correction to renderbox so it gets applied to all elements. I also split relativePositionOffset() to x and y coordinats since thats how it is mostly used.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45460</commentid>
    <comment_count>16</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-12 19:19:47 -0700</bug_when>
    <thetext>For this...

-    int bottom = RenderContainer::lowestPosition(includeOverflowInterior, includeSelf);
+    int bottom = includeSelf &amp;&amp; m_width &gt; 0 ? m_height : 0;

Won&apos;t calling the base class work?
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45461</commentid>
    <comment_count>17</comment_count>
      <attachid>8831</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-12 19:21:28 -0700</bug_when>
    <thetext>Comment on attachment 8831
updated patch

Ok, this is ready.  Just need test cases and results included and then I can +.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45462</commentid>
    <comment_count>18</comment_count>
      <attachid>8832</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-12 19:24:32 -0700</bug_when>
    <thetext>Created attachment 8832
Another test case that is broken.

Here&apos;s another test case that would be fixed if overflowWidth/Height were patched.  No need to fix it before landing, but getting it attached for future work on this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45482</commentid>
    <comment_count>19</comment_count>
      <attachid>8831</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-06-13 02:17:34 -0700</bug_when>
    <thetext>Comment on attachment 8831
updated patch

Antti will commit tests to go with this, so marking r+.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45519</commentid>
    <comment_count>20</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2006-06-13 16:21:03 -0700</bug_when>
    <thetext>Commited. Overflow fix is still needed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45716</commentid>
    <comment_count>21</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-14 19:45:34 -0700</bug_when>
    <thetext>(In reply to comment #20)
&gt; Commited. Overflow fix is still needed.

Committed as r14847.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>46616</commentid>
    <comment_count>22</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2006-06-21 20:39:21 -0700</bug_when>
    <thetext>Should this be open still for the other issues?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>47082</commentid>
    <comment_count>23</comment_count>
      <attachid>8831</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-06-24 20:23:55 -0700</bug_when>
    <thetext>Comment on attachment 8831
updated patch

Clearing the review flag so this won&apos;t show up as a patch that needs to be landed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>151804</commentid>
    <comment_count>24</comment_count>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2009-10-02 02:37:26 -0700</bug_when>
    <thetext>Changing the status to FIXED as I belie the fix has been landed a while ago.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8715</attachid>
            <date>2006-06-05 10:07:42 -0700</date>
            <delta_ts>2006-06-05 10:07:42 -0700</delta_ts>
            <desc>Reduced test case for the espn rel postioning issue</desc>
            <filename>test.html</filename>
            <type>text/html</type>
            <size>139</size>
            <attacher name="Sriram">sriram.yadavalli</attacher>
            
              <data encoding="base64">PGh0bWw+DQo8Ym9keT4NCjxkaXYgc3R5bGU9InBvc2l0aW9uOiByZWxhdGl2ZTsgdG9wOiA0MDBw
eCI+DQo8cD5UaGlzIGlzIGEgdGVzdCBjYXNlIGZvciByZWxhdGl2ZSBwb3NpdGlvbmluZzwvcD4N
CjwvZGl2Pg0KPC9ib2R5Pg0KPC9odG1sPg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8718</attachid>
            <date>2006-06-05 12:50:06 -0700</date>
            <delta_ts>2006-06-06 02:10:20 -0700</delta_ts>
            <desc>This fix resizes the parent layer after children have been positioned</desc>
            <filename>FixForRelPos.txt</filename>
            <type>text/plain</type>
            <size>1557</size>
            <attacher name="Sriram">sriram.yadavalli</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxheWVyLmNwcAo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBX
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllci5jcHAJKHJldmlzaW9uIDE0NzMzKQorKysgV2Vi
Q29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yMjAsNiAr
MjIwLDExIEBAIHZvaWQgUmVuZGVyTGF5ZXI6OnVwZGF0ZUxheWVyUG9zaXRpb25zKGIKICAgICBm
b3IgKFJlbmRlckxheWVyKiBjaGlsZCA9IGZpcnN0Q2hpbGQoKTsgY2hpbGQ7IGNoaWxkID0gY2hp
bGQtPm5leHRTaWJsaW5nKCkpCiAgICAgICAgIGNoaWxkLT51cGRhdGVMYXllclBvc2l0aW9ucyhk
b0Z1bGxSZXBhaW50LCBjaGVja0ZvclJlcGFpbnQpOwogICAgICAgICAKKyAgICAgaWYobV9wYXJl
bnQpIHsKKyAgICAgICBtX3BhcmVudC0+c2V0V2lkdGgobWF4KG1fcGFyZW50LT53aWR0aCgpLCBt
X3ggKyBtX3dpZHRoKSk7CisgICAgICAgbV9wYXJlbnQtPnNldEhlaWdodChtYXgobV9wYXJlbnQt
PmhlaWdodCgpLCBtX3kgKyBtX2hlaWdodCkpOworICAgIH0KKyAgICAgICAgCiAgICAgLy8gV2l0
aCBhbGwgb3VyIGNoaWxkcmVuIHBvc2l0aW9uZWQsIG5vdyB1cGRhdGUgb3VyIG1hcnF1ZWUgaWYg
d2UgbmVlZCB0by4KICAgICBpZiAobV9tYXJxdWVlKQogICAgICAgICBtX21hcnF1ZWUtPnVwZGF0
ZU1hcnF1ZWVQb3NpdGlvbigpOwpJbmRleDogV2ViQ29yZS9wYWdlL0ZyYW1lVmlldy5jcHAKPT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PQotLS0gV2ViQ29yZS9wYWdlL0ZyYW1lVmlldy5jcHAJKHJldmlzaW9uIDE0NzMzKQor
KysgV2ViQ29yZS9wYWdlL0ZyYW1lVmlldy5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTQzNCwxMiAr
NDM0LDEyIEBAIHZvaWQgRnJhbWVWaWV3OjpsYXlvdXQoYm9vbCBhbGxvd1N1YnRyZWUKICAgIAog
ICAgIGQtPmxheW91dFNjaGVkdWxpbmdFbmFibGVkPXRydWU7CiAgICAgZC0+bGF5b3V0U3VwcHJl
c3NlZCA9IGZhbHNlOwotCi0gICAgaWYgKCFzdWJ0cmVlICYmICFzdGF0aWNfY2FzdDxSZW5kZXJW
aWV3Kj4ocm9vdCktPnByaW50aW5nTW9kZSgpKQotICAgICAgICByZXNpemVDb250ZW50cyhsYXll
ci0+d2lkdGgoKSwgbGF5ZXItPmhlaWdodCgpKTsKLQorICAgIAogICAgIC8vIE5vdyB1cGRhdGUg
dGhlIHBvc2l0aW9ucyBvZiBhbGwgbGF5ZXJzLgogICAgIGxheWVyLT51cGRhdGVMYXllclBvc2l0
aW9ucyhkLT5kb0Z1bGxSZXBhaW50KTsKKyAgICAKKyAgICBpZiAoIXN1YnRyZWUgJiYgIXN0YXRp
Y19jYXN0PFJlbmRlclZpZXcqPihyb290KS0+cHJpbnRpbmdNb2RlKCkpCisgICAgICAgIHJlc2l6
ZUNvbnRlbnRzKGxheWVyLT53aWR0aCgpLCBsYXllci0+aGVpZ2h0KCkpOwogCiAgICAgLy8gV2Ug
dXBkYXRlIG91ciB3aWRnZXQgcG9zaXRpb25zIHJpZ2h0IGFmdGVyIGRvaW5nIGEgbGF5b3V0Lgog
ICAgIGlmICghc3VidHJlZSkK
</data>
<flag name="review"
          id="2524"
          type_id="1"
          status="-"
          setter="hyatt"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8802</attachid>
            <date>2006-06-10 21:17:32 -0700</date>
            <delta_ts>2006-06-10 21:17:32 -0700</delta_ts>
            <desc>refined test case</desc>
            <filename>rel-canvas-sizing.html</filename>
            <type>text/html</type>
            <size>371</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">PGh0bWw+Cjxib2R5PgpEb2N1bWVudCBjYW52YXMgc2hvdWxkIGJlIGJpZyBlbm91Z2ggdG8gZml0
IGJvdGggYmx1ZSBhbmQgcmVkIChyZWxhdGl2ZSBwb3NpdGlvbmVkKSByZWN0YW5nbGVzIChpZS4g
aGF2ZSBzY3JvbGxiYXJzIGlmIHRoZXkgZG9uJ3QgZml0IHRvIHRoZSB2aWV3KQo8ZGl2IHN0eWxl
PSJ3aWR0aDogNTAwcHg7IGhlaWdodDogNTAwcHg7IHBvc2l0aW9uOiBhYnNvbHV0ZTsgYm9yZGVy
OiAycHggc29saWQgYmx1ZTsiPgo8ZGl2IHN0eWxlPSJ3aWR0aDogNTAwcHg7IGhlaWdodDogNTAw
cHg7IGxlZnQ6IDMwMHB4OyB0b3A6MzAwcHg7IHBvc2l0aW9uOiByZWxhdGl2ZTsgYm9yZGVyOiAy
cHggc29saWQgcmVkOyI+CjwvZGl2Pgo8L2Rpdj4=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8803</attachid>
            <date>2006-06-10 21:24:46 -0700</date>
            <delta_ts>2006-06-10 21:35:08 -0700</delta_ts>
            <desc>patch</desc>
            <filename>rel-canvas-sizing.diff</filename>
            <type>text/plain</type>
            <size>2920</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJsb2NrLmNwcAo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBX
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCbG9jay5jcHAJKHJldmlzaW9uIDE0Nzg0KQorKysgV2Vi
Q29yZS9yZW5kZXJpbmcvUmVuZGVyQmxvY2suY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yMTU0LDcg
KzIxNTQsNyBAQCBpbnQgUmVuZGVyQmxvY2s6OnJpZ2h0bW9zdFBvc2l0aW9uKGJvb2wgCiAgICAg
aW50IHJpZ2h0ID0gUmVuZGVyRmxvdzo6cmlnaHRtb3N0UG9zaXRpb24oaW5jbHVkZU92ZXJmbG93
SW50ZXJpb3IsIGluY2x1ZGVTZWxmKTsKICAgICBpZiAoIWluY2x1ZGVPdmVyZmxvd0ludGVyaW9y
ICYmIGhhc092ZXJmbG93Q2xpcCgpKQogICAgICAgICByZXR1cm4gcmlnaHQ7Ci0gICAgaWYgKGlu
Y2x1ZGVTZWxmICYmIG1fb3ZlcmZsb3dXaWR0aCA+IHJpZ2h0KQorICAgIGlmIChpbmNsdWRlU2Vs
ZiAmJiBtX292ZXJmbG93V2lkdGggPiByaWdodCAmJiBtX2hlaWdodCA+IDApCiAgICAgICAgIHJp
Z2h0ID0gbV9vdmVyZmxvd1dpZHRoOwogICAgIAogICAgIGlmIChtX2Zsb2F0aW5nT2JqZWN0cykg
ewpJbmRleDogV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyQm94LmNwcAo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBX
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCb3guY3BwCShyZXZpc2lvbiAxNDc4NCkKKysrIFdlYkNv
cmUvcmVuZGVyaW5nL1JlbmRlckJveC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTkzMyw3ICs5MzMs
NyBAQCB2b2lkIFJlbmRlckJveDo6cmVwYWludER1cmluZ0xheW91dElmTW92CiAgICAgfQogfQog
Ci12b2lkIFJlbmRlckJveDo6cmVsYXRpdmVQb3NpdGlvbk9mZnNldChpbnQgJnR4LCBpbnQgJnR5
KQordm9pZCBSZW5kZXJCb3g6OnJlbGF0aXZlUG9zaXRpb25PZmZzZXQoaW50ICZ0eCwgaW50ICZ0
eSkgY29uc3QKIHsKICAgICBpZighc3R5bGUoKS0+bGVmdCgpLmlzQXV0bygpKQogICAgICAgICB0
eCArPSBzdHlsZSgpLT5sZWZ0KCkuY2FsY1ZhbHVlKGNvbnRhaW5pbmdCbG9ja1dpZHRoKCkpOwpA
QCAtMjM2NSwxNyArMjM2NSwxNyBAQCBJbnRSZWN0IFJlbmRlckJveDo6Y2FyZXRSZWN0KGludCBv
ZmZzZXQsCiAKIGludCBSZW5kZXJCb3g6Omxvd2VzdFBvc2l0aW9uKGJvb2wgaW5jbHVkZU92ZXJm
bG93SW50ZXJpb3IsIGJvb2wgaW5jbHVkZVNlbGYpIGNvbnN0CiB7Ci0gICAgcmV0dXJuIGluY2x1
ZGVTZWxmID8gbV9oZWlnaHQgOiAwOworICAgIHJldHVybiBpbmNsdWRlU2VsZiAmJiBtX3dpZHRo
ID4gMCA/IG1faGVpZ2h0IDogMDsKIH0KIAogaW50IFJlbmRlckJveDo6cmlnaHRtb3N0UG9zaXRp
b24oYm9vbCBpbmNsdWRlT3ZlcmZsb3dJbnRlcmlvciwgYm9vbCBpbmNsdWRlU2VsZikgY29uc3QK
IHsKLSAgICByZXR1cm4gaW5jbHVkZVNlbGYgPyBtX3dpZHRoIDogMDsKKyAgICByZXR1cm4gaW5j
bHVkZVNlbGYgJiYgbV9oZWlnaHQgPiAwID8gbV93aWR0aCA6IDA7CiB9CiAKIGludCBSZW5kZXJC
b3g6OmxlZnRtb3N0UG9zaXRpb24oYm9vbCBpbmNsdWRlT3ZlcmZsb3dJbnRlcmlvciwgYm9vbCBp
bmNsdWRlU2VsZikgY29uc3QKIHsKLSAgICByZXR1cm4gaW5jbHVkZVNlbGYgPyAwIDogbV93aWR0
aDsKKyAgICByZXR1cm4gaW5jbHVkZVNlbGYgJiYgbV9oZWlnaHQgPiAwID8gMCA6IG1fd2lkdGg7
CiB9CiAKIH0KSW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5oCj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5oCShyZXZpc2lvbiAxNDc4NCkKKysrIFdl
YkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5oCSh3b3JraW5nIGNvcHkpCkBAIC0xMzIsNyArMTMy
LDcgQEAgcHVibGljOgogICAgIAogICAgIHZvaWQgY2FsY1ZlcnRpY2FsTWFyZ2lucygpOwogCi0g
ICAgdm9pZCByZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0KGludCAmdHgsIGludCAmdHkpOworICAgIHZv
aWQgcmVsYXRpdmVQb3NpdGlvbk9mZnNldChpbnQgJnR4LCBpbnQgJnR5KSBjb25zdDsKIAogICAg
IHZpcnR1YWwgUmVuZGVyTGF5ZXIqIGxheWVyKCkgY29uc3QgeyByZXR1cm4gbV9sYXllcjsgfQog
ICAgIApJbmRleDogV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyRmxvdy5jcHAKPT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot
LS0gV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyRmxvdy5jcHAJKHJldmlzaW9uIDE0Nzg0KQorKysg
V2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyRmxvdy5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTUzMCw2
ICs1MzAsMTEgQEAgUmVuZGVyRmxvdzo6bG93ZXN0UG9zaXRpb24oYm9vbCBpbmNsdWRlTwogICAg
ICAgICAgICAgYm90dG9tID0gbWF4KGJvdHRvbSwgbHApOwogICAgICAgICB9CiAgICAgfQorIAor
ICAgIGlmIChpc1JlbFBvc2l0aW9uZWQoKSkgeworICAgICAgICBpbnQgdHg7CisgICAgICAgIHJl
bGF0aXZlUG9zaXRpb25PZmZzZXQodHgsIGJvdHRvbSk7CisgICAgfSAgICAgICAgICAKICAgICAK
ICAgICByZXR1cm4gYm90dG9tOwogfQpAQCAtNTUyLDYgKzU1NywxMSBAQCBpbnQgUmVuZGVyRmxv
dzo6cmlnaHRtb3N0UG9zaXRpb24oYm9vbCBpCiAgICAgICAgIH0KICAgICB9CiAgICAgCisgICAg
aWYgKGlzUmVsUG9zaXRpb25lZCgpKSB7CisgICAgICAgIGludCB0eTsKKyAgICAgICAgcmVsYXRp
dmVQb3NpdGlvbk9mZnNldChyaWdodCwgdHkpOworICAgIH0gICAgCisgICAgCiAgICAgcmV0dXJu
IHJpZ2h0OwogfQogCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8804</attachid>
            <date>2006-06-10 22:22:12 -0700</date>
            <delta_ts>2006-06-12 15:36:11 -0700</delta_ts>
            <desc>patch</desc>
            <filename>rel-canvas-sizing2.diff</filename>
            <type>text/plain</type>
            <size>2510</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5jcHAKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2Vi
Q29yZS9yZW5kZXJpbmcvUmVuZGVyQm94LmNwcAkocmV2aXNpb24gMTQ3ODQpCisrKyBXZWJDb3Jl
L3JlbmRlcmluZy9SZW5kZXJCb3guY3BwCSh3b3JraW5nIGNvcHkpCkBAIC05MzMsNyArOTMzLDcg
QEAgdm9pZCBSZW5kZXJCb3g6OnJlcGFpbnREdXJpbmdMYXlvdXRJZk1vdgogICAgIH0KIH0KIAot
dm9pZCBSZW5kZXJCb3g6OnJlbGF0aXZlUG9zaXRpb25PZmZzZXQoaW50ICZ0eCwgaW50ICZ0eSkK
K3ZvaWQgUmVuZGVyQm94OjpyZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0KGludCAmdHgsIGludCAmdHkp
IGNvbnN0CiB7CiAgICAgaWYoIXN0eWxlKCktPmxlZnQoKS5pc0F1dG8oKSkKICAgICAgICAgdHgg
Kz0gc3R5bGUoKS0+bGVmdCgpLmNhbGNWYWx1ZShjb250YWluaW5nQmxvY2tXaWR0aCgpKTsKQEAg
LTIzNjUsMTcgKzIzNjUsMTcgQEAgSW50UmVjdCBSZW5kZXJCb3g6OmNhcmV0UmVjdChpbnQgb2Zm
c2V0LAogCiBpbnQgUmVuZGVyQm94Ojpsb3dlc3RQb3NpdGlvbihib29sIGluY2x1ZGVPdmVyZmxv
d0ludGVyaW9yLCBib29sIGluY2x1ZGVTZWxmKSBjb25zdAogewotICAgIHJldHVybiBpbmNsdWRl
U2VsZiA/IG1faGVpZ2h0IDogMDsKKyAgICByZXR1cm4gaW5jbHVkZVNlbGYgJiYgbV93aWR0aCA+
IDAgPyBtX2hlaWdodCA6IDA7CiB9CiAKIGludCBSZW5kZXJCb3g6OnJpZ2h0bW9zdFBvc2l0aW9u
KGJvb2wgaW5jbHVkZU92ZXJmbG93SW50ZXJpb3IsIGJvb2wgaW5jbHVkZVNlbGYpIGNvbnN0CiB7
Ci0gICAgcmV0dXJuIGluY2x1ZGVTZWxmID8gbV93aWR0aCA6IDA7CisgICAgcmV0dXJuIGluY2x1
ZGVTZWxmICYmIG1faGVpZ2h0ID4gMCA/IG1fd2lkdGggOiAwOwogfQogCiBpbnQgUmVuZGVyQm94
OjpsZWZ0bW9zdFBvc2l0aW9uKGJvb2wgaW5jbHVkZU92ZXJmbG93SW50ZXJpb3IsIGJvb2wgaW5j
bHVkZVNlbGYpIGNvbnN0CiB7Ci0gICAgcmV0dXJuIGluY2x1ZGVTZWxmID8gMCA6IG1fd2lkdGg7
CisgICAgcmV0dXJuIGluY2x1ZGVTZWxmICYmIG1faGVpZ2h0ID4gMCA/IDAgOiBtX3dpZHRoOwog
fQogCiB9CkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCb3guaAo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCb3guaAkocmV2aXNpb24gMTQ3ODQpCisrKyBXZWJD
b3JlL3JlbmRlcmluZy9SZW5kZXJCb3guaAkod29ya2luZyBjb3B5KQpAQCAtMTMyLDcgKzEzMiw3
IEBAIHB1YmxpYzoKICAgICAKICAgICB2b2lkIGNhbGNWZXJ0aWNhbE1hcmdpbnMoKTsKIAotICAg
IHZvaWQgcmVsYXRpdmVQb3NpdGlvbk9mZnNldChpbnQgJnR4LCBpbnQgJnR5KTsKKyAgICB2b2lk
IHJlbGF0aXZlUG9zaXRpb25PZmZzZXQoaW50ICZ0eCwgaW50ICZ0eSkgY29uc3Q7CiAKICAgICB2
aXJ0dWFsIFJlbmRlckxheWVyKiBsYXllcigpIGNvbnN0IHsgcmV0dXJuIG1fbGF5ZXI7IH0KICAg
ICAKSW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckZsb3cuY3BwCj09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t
IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckZsb3cuY3BwCShyZXZpc2lvbiAxNDc4NCkKKysrIFdl
YkNvcmUvcmVuZGVyaW5nL1JlbmRlckZsb3cuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC01MzAsNiAr
NTMwLDExIEBAIFJlbmRlckZsb3c6Omxvd2VzdFBvc2l0aW9uKGJvb2wgaW5jbHVkZU8KICAgICAg
ICAgICAgIGJvdHRvbSA9IG1heChib3R0b20sIGxwKTsKICAgICAgICAgfQogICAgIH0KKyAKKyAg
ICBpZiAoaXNSZWxQb3NpdGlvbmVkKCkpIHsKKyAgICAgICAgaW50IHR4OworICAgICAgICByZWxh
dGl2ZVBvc2l0aW9uT2Zmc2V0KHR4LCBib3R0b20pOworICAgIH0gICAgICAgICAgCiAgICAgCiAg
ICAgcmV0dXJuIGJvdHRvbTsKIH0KQEAgLTU1Miw2ICs1NTcsMTEgQEAgaW50IFJlbmRlckZsb3c6
OnJpZ2h0bW9zdFBvc2l0aW9uKGJvb2wgaQogICAgICAgICB9CiAgICAgfQogICAgIAorICAgIGlm
IChpc1JlbFBvc2l0aW9uZWQoKSkgeworICAgICAgICBpbnQgdHk7CisgICAgICAgIHJlbGF0aXZl
UG9zaXRpb25PZmZzZXQocmlnaHQsIHR5KTsKKyAgICB9ICAgIAorICAgIAogICAgIHJldHVybiBy
aWdodDsKIH0KIApAQCAtNTczLDYgKzU4MywxMSBAQCBpbnQgUmVuZGVyRmxvdzo6bGVmdG1vc3RQ
b3NpdGlvbihib29sIGluCiAgICAgICAgIH0KICAgICB9CiAgICAgCisgICAgaWYgKGlzUmVsUG9z
aXRpb25lZCgpKSB7CisgICAgICAgIGludCB0eTsKKyAgICAgICAgcmVsYXRpdmVQb3NpdGlvbk9m
ZnNldChsZWZ0LCB0eSk7CisgICAgfSAgIAorICAgICAgICAKICAgICByZXR1cm4gbGVmdDsKIH0K
IAo=
</data>
<flag name="review"
          id="2555"
          type_id="1"
          status="-"
          setter="hyatt"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8805</attachid>
            <date>2006-06-11 03:06:40 -0700</date>
            <delta_ts>2006-06-11 03:06:40 -0700</delta_ts>
            <desc>Test case that needs to be addressed via leftmost/Rightmost/lowestPosition changes.</desc>
            <filename>imageflaw.html</filename>
            <type>text/html</type>
            <size>246</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">PGh0bWw+Cjxib2R5Pgo8aW1nIHN0eWxlPSJwb3NpdGlvbjpyZWxhdGl2ZTtsZWZ0OjEwMDBweDt0
b3A6MTAwMHB4O2hlaWdodDoxMDBweDt3aWR0aDoxMDBweDtiYWNrZ3JvdW5kLWNvbG9yOmxpbWUi
PgpUaGUgaW1hZ2Ugc2hvdWxkIGJlIDEwMDBweCBvdmVyIHRvIHRoZSByaWdodCBhbmQgYm90dG9t
LCB3aGljaCBzaG91bGQgY2F1c2UgaG9yaXpvbnRhbCBhbmQgdmVydGljYWwgc2Nyb2xsYmFycyB0
byBhcHBlYXIuCjwvYm9keT4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8806</attachid>
            <date>2006-06-11 03:07:58 -0700</date>
            <delta_ts>2006-06-11 03:07:58 -0700</delta_ts>
            <desc>Second test case that uses a floating image instead of an in-flow image.</desc>
            <filename>imageflaw.html</filename>
            <type>text/html</type>
            <size>257</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">PGh0bWw+Cjxib2R5Pgo8aW1nIHN0eWxlPSJmbG9hdDpsZWZ0O3Bvc2l0aW9uOnJlbGF0aXZlO2xl
ZnQ6MTAwMHB4O3RvcDoxMDAwcHg7aGVpZ2h0OjEwMHB4O3dpZHRoOjEwMHB4O2JhY2tncm91bmQt
Y29sb3I6bGltZSI+ClRoZSBpbWFnZSBzaG91bGQgYmUgMTAwMHB4IG92ZXIgdG8gdGhlIHJpZ2h0
IGFuZCBib3R0b20sIHdoaWNoIHNob3VsZCBjYXVzZSBob3Jpem9udGFsIGFuZCB2ZXJ0aWNhbCBz
Y3JvbGxiYXJzIHRvIGFwcGVhci4KPC9ib2R5Pgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8807</attachid>
            <date>2006-06-11 03:13:44 -0700</date>
            <delta_ts>2006-06-11 03:13:44 -0700</delta_ts>
            <desc>Test Case that shows the opposite problem (does not have to be fixed to land).</desc>
            <filename>imageflaw.html</filename>
            <type>text/html</type>
            <size>389</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">PGh0bWw+Cjxib2R5PgpJbiB0aGUgZXhhbXBsZSBiZWxvdywgdGhlcmUgc2hvdWxkIGJlIG5vIHNj
cm9sbGJhcnMgc2hvd24sIGJlY2F1c2UgdGhlIHJlbGF0aXZlIHBvc2l0aW9uZWQgb2JqZWN0IGlz
IHRyYW5zbGF0ZWQgCnN1Y2ggdGhhdCBubyBvdmVyZmxvdyBpcyBuZWNlc3NhcnkuCgo8ZGl2IHN0
eWxlPSJ3aWR0aDo0MDBweDtoZWlnaHQ6NDAwcHg7Ym9yZGVyOjJweCBzb2xpZCBibGFjazt3aGl0
ZS1zcGFjZTpub3dyYXA7b3ZlcmZsb3c6YXV0byI+CjxpbWcgc3R5bGU9IndpZHRoOjQwMHB4OyI+
PGltZyBzdHlsZT0icG9zaXRpb246cmVsYXRpdmU7bGVmdDotNDAwcHg7dG9wOjA7d2lkdGg6NDAw
cHg7aGVpZ2h0OjQwMHB4O2JhY2tncm91bmQtY29sb3I6bGltZSI+CjwvZGl2Pgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8831</attachid>
            <date>2006-06-12 15:36:11 -0700</date>
            <delta_ts>2006-06-24 20:23:55 -0700</delta_ts>
            <desc>updated patch</desc>
            <filename>rel-canvas-sizing3.diff</filename>
            <type>text/plain</type>
            <size>7075</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5jcHAKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2Vi
Q29yZS9yZW5kZXJpbmcvUmVuZGVyQm94LmNwcAkocmV2aXNpb24gMTQ3ODQpCisrKyBXZWJDb3Jl
L3JlbmRlcmluZy9SZW5kZXJCb3guY3BwCSh3b3JraW5nIGNvcHkpCkBAIC03NjIsOCArNzYyLDEw
IEBAIGJvb2wgUmVuZGVyQm94OjphYnNvbHV0ZVBvc2l0aW9uKGludCAmeFAKICAgICAgICAgICAg
IHlQb3MgKz0gbV95OwogICAgICAgICB9CiAKLSAgICAgICAgaWYgKGlzUmVsUG9zaXRpb25lZCgp
KQotICAgICAgICAgICAgcmVsYXRpdmVQb3NpdGlvbk9mZnNldCh4UG9zLCB5UG9zKTsKKyAgICAg
ICAgaWYgKGlzUmVsUG9zaXRpb25lZCgpKSB7CisgICAgICAgICAgICB4UG9zICs9IHJlbGF0aXZl
UG9zaXRpb25PZmZzZXRYKCk7CisgICAgICAgICAgICB5UG9zICs9IHJlbGF0aXZlUG9zaXRpb25P
ZmZzZXRZKCk7ICAgICAgICAgICAgCisgICAgICAgIH0KIAogICAgICAgICByZXR1cm4gdHJ1ZTsK
ICAgICB9CkBAIC05MzMsMjQgKzkzNSwzMiBAQCB2b2lkIFJlbmRlckJveDo6cmVwYWludER1cmlu
Z0xheW91dElmTW92CiAgICAgfQogfQogCi12b2lkIFJlbmRlckJveDo6cmVsYXRpdmVQb3NpdGlv
bk9mZnNldChpbnQgJnR4LCBpbnQgJnR5KQoraW50IFJlbmRlckJveDo6cmVsYXRpdmVQb3NpdGlv
bk9mZnNldFgoKSBjb25zdAogeworICAgIGludCB0eCA9IDA7CiAgICAgaWYoIXN0eWxlKCktPmxl
ZnQoKS5pc0F1dG8oKSkKLSAgICAgICAgdHggKz0gc3R5bGUoKS0+bGVmdCgpLmNhbGNWYWx1ZShj
b250YWluaW5nQmxvY2tXaWR0aCgpKTsKKyAgICAgICAgdHggPSBzdHlsZSgpLT5sZWZ0KCkuY2Fs
Y1ZhbHVlKGNvbnRhaW5pbmdCbG9ja1dpZHRoKCkpOwogICAgIGVsc2UgaWYoIXN0eWxlKCktPnJp
Z2h0KCkuaXNBdXRvKCkpCi0gICAgICAgIHR4IC09IHN0eWxlKCktPnJpZ2h0KCkuY2FsY1ZhbHVl
KGNvbnRhaW5pbmdCbG9ja1dpZHRoKCkpOworICAgICAgICB0eCA9IC1zdHlsZSgpLT5yaWdodCgp
LmNhbGNWYWx1ZShjb250YWluaW5nQmxvY2tXaWR0aCgpKTsKKyAgICByZXR1cm4gdHg7Cit9CisK
K2ludCBSZW5kZXJCb3g6OnJlbGF0aXZlUG9zaXRpb25PZmZzZXRZKCkgY29uc3QKK3sKKyAgICBp
bnQgdHkgPSAwOwogICAgIGlmKCFzdHlsZSgpLT50b3AoKS5pc0F1dG8oKSkKICAgICB7CiAgICAg
ICAgIGlmICghc3R5bGUoKS0+dG9wKCkuaXNQZXJjZW50KCkKICAgICAgICAgICAgICAgICB8fCBj
b250YWluaW5nQmxvY2soKS0+c3R5bGUoKS0+aGVpZ2h0KCkuaXNGaXhlZCgpKQotICAgICAgICAg
ICAgdHkgKz0gc3R5bGUoKS0+dG9wKCkuY2FsY1ZhbHVlKGNvbnRhaW5pbmdCbG9ja0hlaWdodCgp
KTsKKyAgICAgICAgICAgIHR5ID0gc3R5bGUoKS0+dG9wKCkuY2FsY1ZhbHVlKGNvbnRhaW5pbmdC
bG9ja0hlaWdodCgpKTsKICAgICB9CiAgICAgZWxzZSBpZighc3R5bGUoKS0+Ym90dG9tKCkuaXNB
dXRvKCkpCiAgICAgewogICAgICAgICBpZiAoIXN0eWxlKCktPmJvdHRvbSgpLmlzUGVyY2VudCgp
CiAgICAgICAgICAgICAgICAgfHwgY29udGFpbmluZ0Jsb2NrKCktPnN0eWxlKCktPmhlaWdodCgp
LmlzRml4ZWQoKSkKLSAgICAgICAgICAgIHR5IC09IHN0eWxlKCktPmJvdHRvbSgpLmNhbGNWYWx1
ZShjb250YWluaW5nQmxvY2tIZWlnaHQoKSk7CisgICAgICAgICAgICB0eSA9IC1zdHlsZSgpLT5i
b3R0b20oKS5jYWxjVmFsdWUoY29udGFpbmluZ0Jsb2NrSGVpZ2h0KCkpOwogICAgIH0KKyAgICBy
ZXR1cm4gdHk7CiB9CiAKIHZvaWQgUmVuZGVyQm94OjpjYWxjV2lkdGgoKQpAQCAtMjM2NSwxNyAr
MjM3NSwzMiBAQCBJbnRSZWN0IFJlbmRlckJveDo6Y2FyZXRSZWN0KGludCBvZmZzZXQsCiAKIGlu
dCBSZW5kZXJCb3g6Omxvd2VzdFBvc2l0aW9uKGJvb2wgaW5jbHVkZU92ZXJmbG93SW50ZXJpb3Is
IGJvb2wgaW5jbHVkZVNlbGYpIGNvbnN0CiB7Ci0gICAgcmV0dXJuIGluY2x1ZGVTZWxmID8gbV9o
ZWlnaHQgOiAwOworICAgIGlmICghaW5jbHVkZVNlbGYgfHwgIW1fd2lkdGgpCisgICAgICAgIHJl
dHVybiAwOworICAgIGludCBib3R0b20gPSBtX2hlaWdodDsKKyAgICBpZiAoaXNSZWxQb3NpdGlv
bmVkKCkpCisgICAgICAgIGJvdHRvbSArPSByZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0WSgpOworICAg
IHJldHVybiBib3R0b207CiB9CiAKIGludCBSZW5kZXJCb3g6OnJpZ2h0bW9zdFBvc2l0aW9uKGJv
b2wgaW5jbHVkZU92ZXJmbG93SW50ZXJpb3IsIGJvb2wgaW5jbHVkZVNlbGYpIGNvbnN0CiB7Ci0g
ICAgcmV0dXJuIGluY2x1ZGVTZWxmID8gbV93aWR0aCA6IDA7CisgICAgaWYgKCFpbmNsdWRlU2Vs
ZiB8fCAhbV9oZWlnaHQpCisgICAgICAgIHJldHVybiAwOworICAgIGludCByaWdodCA9IG1fd2lk
dGg7CisgICAgaWYgKGlzUmVsUG9zaXRpb25lZCgpKQorICAgICAgICByaWdodCArPSByZWxhdGl2
ZVBvc2l0aW9uT2Zmc2V0WCgpOworICAgIHJldHVybiByaWdodDsKIH0KIAogaW50IFJlbmRlckJv
eDo6bGVmdG1vc3RQb3NpdGlvbihib29sIGluY2x1ZGVPdmVyZmxvd0ludGVyaW9yLCBib29sIGlu
Y2x1ZGVTZWxmKSBjb25zdAogewotICAgIHJldHVybiBpbmNsdWRlU2VsZiA/IDAgOiBtX3dpZHRo
OworICAgIGlmICghaW5jbHVkZVNlbGYgfHwgIW1faGVpZ2h0KQorICAgICAgICByZXR1cm4gbV93
aWR0aDsKKyAgICBpbnQgbGVmdCA9IDA7CisgICAgaWYgKGlzUmVsUG9zaXRpb25lZCgpKQorICAg
ICAgICBsZWZ0ICs9IHJlbGF0aXZlUG9zaXRpb25PZmZzZXRYKCk7CisgICAgcmV0dXJuIGxlZnQ7
CiB9CiAKIH0KSW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5oCj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5oCShyZXZpc2lvbiAxNDc4NCkKKysrIFdl
YkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5oCSh3b3JraW5nIGNvcHkpCkBAIC0xMzIsNyArMTMy
LDggQEAgcHVibGljOgogICAgIAogICAgIHZvaWQgY2FsY1ZlcnRpY2FsTWFyZ2lucygpOwogCi0g
ICAgdm9pZCByZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0KGludCAmdHgsIGludCAmdHkpOworICAgIGlu
dCByZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0WCgpIGNvbnN0OworICAgIGludCByZWxhdGl2ZVBvc2l0
aW9uT2Zmc2V0WSgpIGNvbnN0OwogCiAgICAgdmlydHVhbCBSZW5kZXJMYXllciogbGF5ZXIoKSBj
b25zdCB7IHJldHVybiBtX2xheWVyOyB9CiAgICAgCkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9S
ZW5kZXJGbG93LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93
LmNwcAkocmV2aXNpb24gMTQ3ODQpCisrKyBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93LmNw
cAkod29ya2luZyBjb3B5KQpAQCAtNTE2LDcgKzUxNiw3IEBAIGludAogUmVuZGVyRmxvdzo6bG93
ZXN0UG9zaXRpb24oYm9vbCBpbmNsdWRlT3ZlcmZsb3dJbnRlcmlvciwgYm9vbCBpbmNsdWRlU2Vs
ZikgY29uc3QKIHsKICAgICBhc3NlcnQoIWlzSW5saW5lRmxvdygpKTsKLSAgICBpbnQgYm90dG9t
ID0gUmVuZGVyQ29udGFpbmVyOjpsb3dlc3RQb3NpdGlvbihpbmNsdWRlT3ZlcmZsb3dJbnRlcmlv
ciwgaW5jbHVkZVNlbGYpOworICAgIGludCBib3R0b20gPSBpbmNsdWRlU2VsZiAmJiBtX3dpZHRo
ID4gMCA/IG1faGVpZ2h0IDogMDsKICAgICBpZiAoIWluY2x1ZGVPdmVyZmxvd0ludGVyaW9yICYm
IGhhc092ZXJmbG93Q2xpcCgpKQogICAgICAgICByZXR1cm4gYm90dG9tOwogCkBAIC01MzAsNiAr
NTMwLDkgQEAgUmVuZGVyRmxvdzo6bG93ZXN0UG9zaXRpb24oYm9vbCBpbmNsdWRlTwogICAgICAg
ICAgICAgYm90dG9tID0gbWF4KGJvdHRvbSwgbHApOwogICAgICAgICB9CiAgICAgfQorIAorICAg
IGlmIChpc1JlbFBvc2l0aW9uZWQoKSkKKyAgICAgICAgYm90dG9tICs9IHJlbGF0aXZlUG9zaXRp
b25PZmZzZXRZKCk7ICAgICAgICAgCiAgICAgCiAgICAgcmV0dXJuIGJvdHRvbTsKIH0KQEAgLTUz
Nyw3ICs1NDAsNyBAQCBSZW5kZXJGbG93Ojpsb3dlc3RQb3NpdGlvbihib29sIGluY2x1ZGVPCiBp
bnQgUmVuZGVyRmxvdzo6cmlnaHRtb3N0UG9zaXRpb24oYm9vbCBpbmNsdWRlT3ZlcmZsb3dJbnRl
cmlvciwgYm9vbCBpbmNsdWRlU2VsZikgY29uc3QKIHsKICAgICBhc3NlcnQoIWlzSW5saW5lRmxv
dygpKTsKLSAgICBpbnQgcmlnaHQgPSBSZW5kZXJDb250YWluZXI6OnJpZ2h0bW9zdFBvc2l0aW9u
KGluY2x1ZGVPdmVyZmxvd0ludGVyaW9yLCBpbmNsdWRlU2VsZik7CisgICAgaW50IHJpZ2h0ID0g
aW5jbHVkZVNlbGYgJiYgbV9oZWlnaHQgPiAwID8gbV93aWR0aCA6IDA7CiAgICAgaWYgKCFpbmNs
dWRlT3ZlcmZsb3dJbnRlcmlvciAmJiBoYXNPdmVyZmxvd0NsaXAoKSkKICAgICAgICAgcmV0dXJu
IHJpZ2h0OwogCkBAIC01NTIsMTMgKzU1NSwxNiBAQCBpbnQgUmVuZGVyRmxvdzo6cmlnaHRtb3N0
UG9zaXRpb24oYm9vbCBpCiAgICAgICAgIH0KICAgICB9CiAgICAgCisgICAgaWYgKGlzUmVsUG9z
aXRpb25lZCgpKQorICAgICAgICByaWdodCArPSByZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0WCgpOwor
ICAgIAogICAgIHJldHVybiByaWdodDsKIH0KIAogaW50IFJlbmRlckZsb3c6OmxlZnRtb3N0UG9z
aXRpb24oYm9vbCBpbmNsdWRlT3ZlcmZsb3dJbnRlcmlvciwgYm9vbCBpbmNsdWRlU2VsZikgY29u
c3QKIHsKICAgICBhc3NlcnQoIWlzSW5saW5lRmxvdygpKTsKLSAgICBpbnQgbGVmdCA9IFJlbmRl
ckNvbnRhaW5lcjo6bGVmdG1vc3RQb3NpdGlvbihpbmNsdWRlT3ZlcmZsb3dJbnRlcmlvciwgaW5j
bHVkZVNlbGYpOworICAgIGludCBsZWZ0ID0gaW5jbHVkZVNlbGYgJiYgbV9oZWlnaHQgPiAwID8g
MCA6IG1fd2lkdGg7CiAgICAgaWYgKCFpbmNsdWRlT3ZlcmZsb3dJbnRlcmlvciAmJiBoYXNPdmVy
Zmxvd0NsaXAoKSkKICAgICAgICAgcmV0dXJuIGxlZnQ7CiAgICAgCkBAIC01NzMsNiArNTc5LDkg
QEAgaW50IFJlbmRlckZsb3c6OmxlZnRtb3N0UG9zaXRpb24oYm9vbCBpbgogICAgICAgICB9CiAg
ICAgfQogICAgIAorICAgIGlmIChpc1JlbFBvc2l0aW9uZWQoKSkKKyAgICAgICAgbGVmdCArPSBy
ZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0WCgpOyAKKyAgICAgICAgCiAgICAgcmV0dXJuIGxlZnQ7CiB9
CiAKSW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxheWVyLmNwcAo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllci5jcHAJKHJldmlzaW9uIDE0Nzg0KQorKysg
V2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yNjMs
NyArMjYzLDggQEAgdm9pZCBSZW5kZXJMYXllcjo6dXBkYXRlTGF5ZXJQb3NpdGlvbigpCiAKICAg
ICBtX3JlbFggPSBtX3JlbFkgPSAwOwogICAgIGlmIChtX29iamVjdC0+aXNSZWxQb3NpdGlvbmVk
KCkpIHsKLSAgICAgICAgc3RhdGljX2Nhc3Q8UmVuZGVyQm94Kj4obV9vYmplY3QpLT5yZWxhdGl2
ZVBvc2l0aW9uT2Zmc2V0KG1fcmVsWCwgbV9yZWxZKTsKKyAgICAgICAgbV9yZWxYID0gc3RhdGlj
X2Nhc3Q8UmVuZGVyQm94Kj4obV9vYmplY3QpLT5yZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0WCgpOwor
ICAgICAgICBtX3JlbFkgPSBzdGF0aWNfY2FzdDxSZW5kZXJCb3gqPihtX29iamVjdCktPnJlbGF0
aXZlUG9zaXRpb25PZmZzZXRZKCk7CiAgICAgICAgIHggKz0gbV9yZWxYOyB5ICs9IG1fcmVsWTsK
ICAgICB9CiAgICAgCkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJPYmplY3QuY3BwCj09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT0KLS0tIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlck9iamVjdC5jcHAJKHJldmlzaW9u
IDE0Nzg0KQorKysgV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyT2JqZWN0LmNwcAkod29ya2luZyBj
b3B5KQpAQCAtNTQ5LDEwICs1NDksOCBAQCBpbnQgUmVuZGVyT2JqZWN0OjpvZmZzZXRMZWZ0KCkg
Y29uc3QKIHsKICAgICBpbnQgeCA9IHhQb3MoKTsKICAgICBpZiAoIWlzUG9zaXRpb25lZCgpKSB7
Ci0gICAgICAgIGlmIChpc1JlbFBvc2l0aW9uZWQoKSkgewotICAgICAgICAgICAgaW50IHkgPSAw
OwotICAgICAgICAgICAgKChSZW5kZXJCb3gqKXRoaXMpLT5yZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0
KHgsIHkpOwotICAgICAgICB9CisgICAgICAgIGlmIChpc1JlbFBvc2l0aW9uZWQoKSkKKyAgICAg
ICAgICAgIHggKz0gKChSZW5kZXJCb3gqKXRoaXMpLT5yZWxhdGl2ZVBvc2l0aW9uT2Zmc2V0WCgp
OwogICAgICAgICAKICAgICAgICAgUmVuZGVyT2JqZWN0KiBvZmZzZXRQYXIgPSBvZmZzZXRQYXJl
bnQoKTsKICAgICAgICAgUmVuZGVyT2JqZWN0KiBjdXJyID0gcGFyZW50KCk7CkBAIC01NjgsMTAg
KzU2Niw4IEBAIGludCBSZW5kZXJPYmplY3Q6Om9mZnNldFRvcCgpIGNvbnN0CiB7CiAgICAgaW50
IHkgPSB5UG9zKCk7CiAgICAgaWYgKCFpc1Bvc2l0aW9uZWQoKSkgewotICAgICAgICBpZiAoaXNS
ZWxQb3NpdGlvbmVkKCkpIHsKLSAgICAgICAgICAgIGludCB4ID0gMDsKLSAgICAgICAgICAgICgo
UmVuZGVyQm94Kil0aGlzKS0+cmVsYXRpdmVQb3NpdGlvbk9mZnNldCh4LCB5KTsKLSAgICAgICAg
fQorICAgICAgICBpZiAoaXNSZWxQb3NpdGlvbmVkKCkpCisgICAgICAgICAgICB5ICs9ICgoUmVu
ZGVyQm94Kil0aGlzKS0+cmVsYXRpdmVQb3NpdGlvbk9mZnNldFkoKTsKICAgICAgICAgUmVuZGVy
T2JqZWN0KiBvZmZzZXRQYXIgPSBvZmZzZXRQYXJlbnQoKTsKICAgICAgICAgUmVuZGVyT2JqZWN0
KiBjdXJyID0gcGFyZW50KCk7CiAgICAgICAgIHdoaWxlIChjdXJyICYmIGN1cnIgIT0gb2Zmc2V0
UGFyKSB7Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8832</attachid>
            <date>2006-06-12 19:24:32 -0700</date>
            <delta_ts>2006-06-12 22:01:45 -0700</delta_ts>
            <desc>Another test case that is broken.</desc>
            <filename>imageflaw.html</filename>
            <type>text/html</type>
            <size>259</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">PGh0bWw+Cjxib2R5Pgo8c3Bhbj48aW1nIHN0eWxlPSJwb3NpdGlvbjpyZWxhdGl2ZTtsZWZ0OjEw
MDBweDt0b3A6MTAwMHB4O2hlaWdodDoxMDBweDt3aWR0aDoxMDBweDtiYWNrZ3JvdW5kLWNvbG9y
OmxpbWUiPjwvc3Bhbj4KVGhlIGltYWdlIHNob3VsZCBiZSAxMDAwcHggb3ZlciB0byB0aGUgcmln
aHQgYW5kIGJvdHRvbSwgd2hpY2ggc2hvdWxkIGNhdXNlIGhvcml6b250YWwgYW5kIHZlcnRpY2Fs
IHNjcm9sbGJhcnMgdG8gYXBwZWFyLgo8L2JvZHk+Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>