<?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>64140</bug_id>
          
          <creation_ts>2011-07-07 18:01:43 -0700</creation_ts>
          <short_desc>REGRESSION: Pressing return in a particular document sends the cursor to the end of the document</short_desc>
          <delta_ts>2011-07-18 01:12:49 -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>HTML Editing</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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Justin Garcia">justin.garcia</reporter>
          <assigned_to name="Enrica Casucci">enrica</assigned_to>
          <cc>enrica</cc>
    
    <cc>leviw</cc>
    
    <cc>rniwa</cc>
    
    <cc>sullivan</cc>
    
    <cc>wcarss</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>433829</commentid>
    <comment_count>0</comment_count>
    <who name="Justin Garcia">justin.garcia</who>
    <bug_when>2011-07-07 18:01:43 -0700</bug_when>
    <thetext>Open the following test case in Safari:

&lt;body contentEditable=&quot;true&quot;&gt;&lt;b&gt;place the cursor between these two lines&lt;br&gt;&lt;br&gt;and press return&lt;/b&gt;&lt;br&gt;&lt;/body&gt;

and place the cursor between the two lines and press return. The cursor moves to the end of the document.

&lt;rdar://problem/9737491&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>433831</commentid>
    <comment_count>1</comment_count>
    <who name="Justin Garcia">justin.garcia</who>
    <bug_when>2011-07-07 18:04:34 -0700</bug_when>
    <thetext>Using a WebKit nightly build from r90560 on Mac OS 10.6.7. Does not reproduce with Safari 5.0.5 on the same OS.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>433834</commentid>
    <comment_count>2</comment_count>
    <who name="Annie Sullivan">sullivan</who>
    <bug_when>2011-07-07 18:09:56 -0700</bug_when>
    <thetext>This sounds really similar to bug 61594, but I can reproduce this test case and not the one in that bug. It is probably in the same codepath (InsertParagraphSeparatorCommand).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438197</commentid>
    <comment_count>3</comment_count>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 16:11:34 -0700</bug_when>
    <thetext>I&apos;m not sure exactly where this regressed. I suspect http://trac.webkit.org/changeset/83247 where the cloning of the hierarchy was removed.
The bug seems to reproduce always when the insertion point is over a BR element inside an inline.
I&apos;ve implemented a patch that detects when the insertion point is a BR element and simply adds another BR element, without trying to create another block and move stuff around.
I&apos;ll run all the regression tests to see what breaks.
The approach is promising, because it produces a much simpler markup.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438201</commentid>
    <comment_count>4</comment_count>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 16:20:38 -0700</bug_when>
    <thetext>Only one test is currently failing, but the visual result is correct. The failure is due to the different markup being generated.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438208</commentid>
    <comment_count>5</comment_count>
      <attachid>101105</attachid>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 17:17:15 -0700</bug_when>
    <thetext>Created attachment 101105
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438209</commentid>
    <comment_count>6</comment_count>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 17:24:25 -0700</bug_when>
    <thetext>This patch will probably break the test I&apos;ve updated the results for in other platforms. I will fix the results as they become available.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438210</commentid>
    <comment_count>7</comment_count>
      <attachid>101105</attachid>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2011-07-16 17:24:28 -0700</bug_when>
    <thetext>Comment on attachment 101105
Patch

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

&gt; Source/WebCore/ChangeLog:9
&gt; +        The fix consists in detecting that the insertion point is a break element and simply insert another

insert -&gt; inserting

&gt; Source/WebCore/editing/InsertParagraphSeparatorCommand.cpp:307
&gt; +        if (visiblePos.deepEquivalent().anchorNode()-&gt;renderer()-&gt;isBR()) {

Is anchorNode() guaranteed to have a renderer?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438211</commentid>
    <comment_count>8</comment_count>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 17:28:45 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (From update of attachment 101105 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=101105&amp;action=review
&gt; 
&gt; &gt; Source/WebCore/ChangeLog:9
&gt; &gt; +        The fix consists in detecting that the insertion point is a break element and simply insert another
&gt; 
&gt; insert -&gt; inserting
&gt; 
&gt; &gt; Source/WebCore/editing/InsertParagraphSeparatorCommand.cpp:307
&gt; &gt; +        if (visiblePos.deepEquivalent().anchorNode()-&gt;renderer()-&gt;isBR()) {
&gt; 
&gt; Is anchorNode() guaranteed to have a renderer?

Yes, because is from a visibleposition.
thanks for the review!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438213</commentid>
    <comment_count>9</comment_count>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 17:34:34 -0700</bug_when>
    <thetext>Committed revision 91158.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438216</commentid>
    <comment_count>10</comment_count>
      <attachid>101105</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-07-16 17:48:59 -0700</bug_when>
    <thetext>Comment on attachment 101105
Patch

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

&gt;&gt; Source/WebCore/editing/InsertParagraphSeparatorCommand.cpp:307
&gt;&gt; +        if (visiblePos.deepEquivalent().anchorNode()-&gt;renderer()-&gt;isBR()) {
&gt; 
&gt; Is anchorNode() guaranteed to have a renderer?

Are you sure we we need to exit early both when the insertion position is before BR and after BR?  What if visiblePos was before another block element such as hr?  Shouldn&apos;t be also exit early in the case?  I&apos;m not convinced that we should be exiting early here if visiblePos was after BR.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438226</commentid>
    <comment_count>11</comment_count>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2011-07-16 18:10:06 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; (From update of attachment 101105 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=101105&amp;action=review
&gt; 
&gt; &gt;&gt; Source/WebCore/editing/InsertParagraphSeparatorCommand.cpp:307
&gt; &gt;&gt; +        if (visiblePos.deepEquivalent().anchorNode()-&gt;renderer()-&gt;isBR()) {
&gt; &gt; 
&gt; &gt; Is anchorNode() guaranteed to have a renderer?
&gt; 
&gt; Are you sure we we need to exit early both when the insertion position is before BR and after BR?  What if visiblePos was before another block element such as hr?  Shouldn&apos;t be also exit early in the case?  I&apos;m not convinced that we should be exiting early here if visiblePos was after BR.
If we are at the beginning of another block but not on a BR, then we don&apos;t exit early.
if the markup is

&lt;b&gt;one&lt;br&gt;&lt;br&gt;two&lt;/br&gt; and the insertion point is on &quot;two&quot; we don&apos;t exit early.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438231</commentid>
    <comment_count>12</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-07-16 18:57:02 -0700</bug_when>
    <thetext>(In reply to comment #11)
&gt; (In reply to comment #10)
&gt; &gt; Are you sure we we need to exit early both when the insertion position is before BR and after BR?  What if visiblePos was before another block element such as hr?  Shouldn&apos;t be also exit early in the case?  I&apos;m not convinced that we should be exiting early here if visiblePos was after BR.
&gt; If we are at the beginning of another block but not on a BR, then we don&apos;t exit early.

But we could still be anchored at BR in those cases, right?  e.g. if we had &lt;b&gt;one&lt;br&gt;&lt;br&gt;two&lt;/br&gt;, we can have (two, PositionIsOffsetInAnchor, 0) but we might as well as have (second br, PositionIsAfterAnchor).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>438375</commentid>
    <comment_count>13</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-07-18 01:12:49 -0700</bug_when>
    <thetext>Chromium rebaseline: http://trac.webkit.org/changeset/91172
GTK / Qt rebaselines: http://trac.webkit.org/changeset/91175</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>101105</attachid>
            <date>2011-07-16 17:17:15 -0700</date>
            <delta_ts>2011-07-16 17:48:59 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>insertatbreak.txt</filename>
            <type>text/plain</type>
            <size>6772</size>
            <attacher name="Enrica Casucci">enrica</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDkxMTU3KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjMgQEAKKzIwMTEtMDctMTYgIEVucmljYSBD
YXN1Y2NpICA8ZW5yaWNhQGFwcGxlLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OOiBQcmVzc2lu
ZyByZXR1cm4gaW4gYSBwYXJ0aWN1bGFyIGRvY3VtZW50IHNlbmRzIHRoZSBjdXJzb3IgdG8gdGhl
IGVuZCBvZiB0aGUgZG9jdW1lbnQuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD02NDE0MAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vOTczNzQ5MT4KKwor
ICAgICAgICBJbnNlcnRpbmcgYSBwYXJhZ3JhcGggc2VwYXJhdG9yIGF0IGEgYnJlYWsgZWxlbWVu
dCwgaW5zaWRlIGFuIGlubGluZSwgZ2VuZXJhdGVzIHJlZHVuZGFudCBtYXJrdXAKKyAgICAgICAg
YW5kIG1vdmVzIHRoZSBzZWxlY3Rpb24gYXQgdGhlIGVuZCBvZiB0aGUgaW5zZXJ0ZWQgYmxvY2su
CisgICAgICAgIFRoZSBmaXggY29uc2lzdHMgaW4gZGV0ZWN0aW5nIHRoYXQgdGhlIGluc2VydGlv
biBwb2ludCBpcyBhIGJyZWFrIGVsZW1lbnQgYW5kIHNpbXBseSBpbnNlcnQgYW5vdGhlcgorICAg
ICAgICBicmVhayBlbGVtZW50LiBUaGlzIHNvbHZlcyB0aGUgcHJvYmxlbSBvZiB0aGUgd3Jvbmcg
ZmluYWwgc2VsZWN0aW9uIGFuZCBhbHNvIHByb2R1Y2VzIGxlc3MgcmVkdW5kYW50CisgICAgICAg
IG1hcmt1cC4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICBUZXN0OiBlZGl0aW5nL2luc2VydGluZy9pbnNlcnQtcGFyYWdyYXBoLXNlcGFyYXRvci1hdC1i
cmVhay5odG1sCisKKyAgICAgICAgKiBlZGl0aW5nL0luc2VydFBhcmFncmFwaFNlcGFyYXRvckNv
bW1hbmQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6SW5zZXJ0UGFyYWdyYXBoU2VwYXJhdG9yQ29t
bWFuZDo6ZG9BcHBseSk6IE1vZGlmaWVkIGxvZ2ljIGZvciBpbnNlcnRpb24gYXQgYSBicmVhawor
ICAgICAgICBlbGVtZW50LgorCiAyMDExLTA3LTE2ICBQcmF0aWsgU29sYW5raSAgPHBzb2xhbmtp
QGFwcGxlLmNvbT4KIAogICAgICAgICBTdWJyZXNvdXJjZUxvYWRlcjo6ZGlkUmVjZWl2ZURhdGFB
cnJheSBjYW4gY3Jhc2ggd2hlbiBjYWxsaW5nIG1fY2xpZW50LT5kaWRSZWNlaXZlRGF0YSgpCklu
ZGV4OiBTb3VyY2UvV2ViQ29yZS9lZGl0aW5nL0luc2VydFBhcmFncmFwaFNlcGFyYXRvckNvbW1h
bmQuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL2VkaXRpbmcvSW5zZXJ0UGFyYWdy
YXBoU2VwYXJhdG9yQ29tbWFuZC5jcHAJKHJldmlzaW9uIDkxMTM2KQorKysgU291cmNlL1dlYkNv
cmUvZWRpdGluZy9JbnNlcnRQYXJhZ3JhcGhTZXBhcmF0b3JDb21tYW5kLmNwcAkod29ya2luZyBj
b3B5KQpAQCAtMzAyLDYgKzMwMiwxMiBAQCB2b2lkIEluc2VydFBhcmFncmFwaFNlcGFyYXRvckNv
bW1hbmQ6OmRvCiAgICAgICAgIFJlZlB0cjxFbGVtZW50PiBiciA9IGNyZWF0ZUJyZWFrRWxlbWVu
dChkb2N1bWVudCgpKTsKICAgICAgICAgaW5zZXJ0Tm9kZUF0KGJyLmdldCgpLCBpbnNlcnRpb25Q
b3NpdGlvbik7CiAgICAgICAgIGluc2VydGlvblBvc2l0aW9uID0gcG9zaXRpb25JblBhcmVudEFm
dGVyTm9kZShici5nZXQoKSk7CisgICAgICAgIC8vIElmIHRoZSBpbnNlcnRpb24gcG9pbnQgaXMg
YSBicmVhayBlbGVtZW50LCB0aGVyZSBpcyBub3RoaW5nIGVsc2UKKyAgICAgICAgLy8gd2UgbmVl
ZCB0byBkby4KKyAgICAgICAgaWYgKHZpc2libGVQb3MuZGVlcEVxdWl2YWxlbnQoKS5hbmNob3JO
b2RlKCktPnJlbmRlcmVyKCktPmlzQlIoKSkgeworICAgICAgICAgICAgc2V0RW5kaW5nU2VsZWN0
aW9uKFZpc2libGVTZWxlY3Rpb24oaW5zZXJ0aW9uUG9zaXRpb24sIERPV05TVFJFQU0pKTsKKyAg
ICAgICAgICAgIHJldHVybjsKKyAgICAgICAgfQogICAgIH0KICAgICAKICAgICAvLyBNb3ZlIGRv
d25zdHJlYW0uIFR5cGluZyBzdHlsZSBjb2RlIHdpbGwgdGFrZSBjYXJlIG9mIGNhcnJ5aW5nIGFs
b25nIHRoZSAKSW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlv
dXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDkxMTU3KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDctMTYgIEVucmljYSBD
YXN1Y2NpICA8ZW5yaWNhQGFwcGxlLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OOiBQcmVzc2lu
ZyByZXR1cm4gaW4gYSBwYXJ0aWN1bGFyIGRvY3VtZW50IHNlbmRzIHRoZSBjdXJzb3IgdG8gdGhl
IGVuZCBvZiB0aGUgZG9jdW1lbnQuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD02NDE0MAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vOTczNzQ5MT4KKwor
ICAgICAgICBBZGRlZCBuZXcgdGVzdCB0byBjb3ZlciB0aGlzIGNhc2UgYW5kIHVwZGF0ZWQgcmVz
dWx0cyBmb3Igb25lIHRlc3QgdGhhdCBub3cKKyAgICAgICAgcHJvZHVjZXMgc2ltcGxlciBtYXJr
dXAuCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBl
ZGl0aW5nL2luc2VydGluZy9pbnNlcnQtcGFyYWdyYXBoLXNlcGFyYXRvci1hdC1icmVhay1leHBl
Y3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIGVkaXRpbmcvaW5zZXJ0aW5nL2luc2VydC1wYXJh
Z3JhcGgtc2VwYXJhdG9yLWF0LWJyZWFrLmh0bWw6IEFkZGVkLgorICAgICAgICAqIHBsYXRmb3Jt
L21hYy9lZGl0aW5nL2luc2VydGluZy81MDU4MTYzLTEtZXhwZWN0ZWQudHh0OgorCiAyMDExLTA3
LTE2ICBTZXJnZXkgR2xhenVub3YgIDxzZXJnLmdsYXp1bm92QGdtYWlsLmNvbT4KIAogICAgICAg
ICBET01XaW5kb3c6Om9wZW4gcGVyZm9ybXMgYSBzZWN1cml0eSBjaGVjayBvbiBhIHdyb25nIHdp
bmRvdwpJbmRleDogTGF5b3V0VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcvaW5zZXJ0LXBhcmFncmFw
aC1zZXBhcmF0b3ItYXQtYnJlYWstZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRl
c3RzL2VkaXRpbmcvaW5zZXJ0aW5nL2luc2VydC1wYXJhZ3JhcGgtc2VwYXJhdG9yLWF0LWJyZWFr
LWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2VkaXRpbmcvaW5zZXJ0
aW5nL2luc2VydC1wYXJhZ3JhcGgtc2VwYXJhdG9yLWF0LWJyZWFrLWV4cGVjdGVkLnR4dAkocmV2
aXNpb24gMCkKQEAgLTAsMCArMSwxNCBAQAorVGhpcyB0ZXN0IGVuc3VyZXMgV2ViS2l0IGluc2Vy
dHMgb25seSBhIGJyZWFrIGVsZW1lbnQgd2hlbiBpbnNlcnRpbmcgYSBwYXJhZ3JhcGggYXQgYSBi
cmVhayBlbGVtZW50LgorT25seSBvbmUgQlIgc2hvdWxkIGJlIGluc2VydGVkIGluIHRoaXMgdGVz
dC4KKworQWZ0ZXI6Cit8IDxiPgorfCAgICJwbGFjZSB0aGUgY3Vyc29yIGJldHdlZW4gdGhlc2Ug
dHdvIGxpbmVzIgorfCAgIDxicj4KK3wgICA8YnI+Cit8ICAgPCNzZWxlY3Rpb24tY2FyZXQ+Cit8
ICAgPGJyPgorfCAgICJhbmQgcHJlc3MgcmV0dXJuIgorfCA8YnI+Cit8ICIKKyIKSW5kZXg6IExh
eW91dFRlc3RzL2VkaXRpbmcvaW5zZXJ0aW5nL2luc2VydC1wYXJhZ3JhcGgtc2VwYXJhdG9yLWF0
LWJyZWFrLmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcv
aW5zZXJ0LXBhcmFncmFwaC1zZXBhcmF0b3ItYXQtYnJlYWsuaHRtbAkocmV2aXNpb24gMCkKKysr
IExheW91dFRlc3RzL2VkaXRpbmcvaW5zZXJ0aW5nL2luc2VydC1wYXJhZ3JhcGgtc2VwYXJhdG9y
LWF0LWJyZWFrLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTkgQEAKKzwhRE9DVFlQRSBo
dG1sPgorPGh0bWw+Cis8Ym9keT4KKzxkaXYgY29udGVudGVkaXRhYmxlIGlkPSJ0ZXN0Ij48Yj5w
bGFjZSB0aGUgY3Vyc29yIGJldHdlZW4gdGhlc2UgdHdvIGxpbmVzPGJyPjxicj5hbmQgcHJlc3Mg
cmV0dXJuPC9iPjxicj4KKzwvZGl2PgorPHNjcmlwdCBzcmM9Ii4uLy4uL3Jlc291cmNlcy9kdW1w
LWFzLW1hcmt1cC5qcyI+PC9zY3JpcHQ+Cis8c2NyaXB0PgorCitNYXJrdXAuZGVzY3JpcHRpb24o
J1RoaXMgdGVzdCBlbnN1cmVzIFdlYktpdCBpbnNlcnRzIG9ubHkgYSBicmVhayBlbGVtZW50IHdo
ZW4gaW5zZXJ0aW5nIGEgcGFyYWdyYXBoIGF0IGEgYnJlYWsgZWxlbWVudC5cbicrCisgICAgJ09u
bHkgb25lIEJSIHNob3VsZCBiZSBpbnNlcnRlZCBpbiB0aGlzIHRlc3QuJyk7CisKK3ZhciBlbGVt
ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ3Rlc3QnKTsKK3dpbmRvdy5nZXRTZWxlY3Rpb24o
KS5zZXRQb3NpdGlvbihlbGVtLmZpcnN0Q2hpbGQsIDIpOworZG9jdW1lbnQuZXhlY0NvbW1hbmQo
Ikluc2VydFBhcmFncmFwaCIpOworTWFya3VwLmR1bXAoZWxlbSwgIkFmdGVyIik7CisKKzwvc2Ny
aXB0PgorPC9ib2R5PgorPC9odG1sPgpJbmRleDogTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2Vk
aXRpbmcvaW5zZXJ0aW5nLzUwNTgxNjMtMS1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5
b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2VkaXRpbmcvaW5zZXJ0aW5nLzUwNTgxNjMtMS1leHBlY3Rl
ZC50eHQJKHJldmlzaW9uIDkxMTM2KQorKysgTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2VkaXRp
bmcvaW5zZXJ0aW5nLzUwNTgxNjMtMS1leHBlY3RlZC50eHQJKHdvcmtpbmcgY29weSkKQEAgLTEz
LDE1ICsxMywxMyBAQCBsYXllciBhdCAoMCwwKSBzaXplIDgwMHg2MDAKICAgICAgICAgICAgICAg
UmVuZGVyVGFibGVDZWxsIHtURH0gYXQgKDIsMikgc2l6ZSA0Njl4MjAgW3I9MCBjPTAgcnM9MSBj
cz0xXQogICAgICAgICAgICAgICAgIFJlbmRlclRleHQgeyN0ZXh0fSBhdCAoMSwxKSBzaXplIDQ2
N3gxOAogICAgICAgICAgICAgICAgICAgdGV4dCBydW4gYXQgKDEsMSkgd2lkdGggNDY3OiAiVGhl
cmUgc2hvdWxkIGJlIHR3byBlbXB0eSBwYXJhZ3JhcGhzIGFmdGVyIHRoaXMgdGFibGUgYW5kIGJl
Zm9yZSB0aGUgbmV4dC4iCi0gICAgICAgIFJlbmRlckJsb2NrIChhbm9ueW1vdXMpIGF0ICgwLDI2
KSBzaXplIDc4NHgxOAorICAgICAgICBSZW5kZXJCbG9jayAoYW5vbnltb3VzKSBhdCAoMCwyNikg
c2l6ZSA3ODR4MzYKICAgICAgICAgICBSZW5kZXJCUiB7QlJ9IGF0ICgwLDApIHNpemUgMHgxOAot
ICAgICAgICBSZW5kZXJCbG9jayB7RElWfSBhdCAoMCw0NCkgc2l6ZSA3ODR4NDQKLSAgICAgICAg
ICBSZW5kZXJCbG9jayAoYW5vbnltb3VzKSBhdCAoMCwwKSBzaXplIDc4NHgxOAotICAgICAgICAg
ICAgUmVuZGVyQlIge0JSfSBhdCAoMCwwKSBzaXplIDB4MTgKLSAgICAgICAgICBSZW5kZXJUYWJs
ZSB7VEFCTEV9IGF0ICgwLDE4KSBzaXplIDI3NngyNiBbYm9yZGVyOiAoMXB4IHNvbGlkICNBQUFB
QUEpXQotICAgICAgICAgICAgUmVuZGVyVGFibGVTZWN0aW9uIHtUQk9EWX0gYXQgKDEsMSkgc2l6
ZSAyNzR4MjQKLSAgICAgICAgICAgICAgUmVuZGVyVGFibGVSb3cge1RSfSBhdCAoMCwyKSBzaXpl
IDI3NHgyMAotICAgICAgICAgICAgICAgIFJlbmRlclRhYmxlQ2VsbCB7VER9IGF0ICgyLDIpIHNp
emUgMjcweDIwIFtyPTAgYz0wIHJzPTEgY3M9MV0KLSAgICAgICAgICAgICAgICAgIFJlbmRlclRl
eHQgeyN0ZXh0fSBhdCAoMSwxKSBzaXplIDI2OHgxOAotICAgICAgICAgICAgICAgICAgICB0ZXh0
IHJ1biBhdCAoMSwxKSB3aWR0aCAyNjg6ICJBbmQgdGhlIGNhcmV0IHNob3VsZCBiZSBpbiB0aGUg
c2Vjb25kIG9uZS4iCi1jYXJldDogcG9zaXRpb24gMCBvZiBjaGlsZCAwIHtCUn0gb2YgY2hpbGQg
MiB7RElWfSBvZiBjaGlsZCAyIHtESVZ9IG9mIGJvZHkKKyAgICAgICAgICBSZW5kZXJCUiB7QlJ9
IGF0ICgwLDE4KSBzaXplIDB4MTgKKyAgICAgICAgUmVuZGVyVGFibGUge1RBQkxFfSBhdCAoMCw2
Mikgc2l6ZSAyNzZ4MjYgW2JvcmRlcjogKDFweCBzb2xpZCAjQUFBQUFBKV0KKyAgICAgICAgICBS
ZW5kZXJUYWJsZVNlY3Rpb24ge1RCT0RZfSBhdCAoMSwxKSBzaXplIDI3NHgyNAorICAgICAgICAg
ICAgUmVuZGVyVGFibGVSb3cge1RSfSBhdCAoMCwyKSBzaXplIDI3NHgyMAorICAgICAgICAgICAg
ICBSZW5kZXJUYWJsZUNlbGwge1REfSBhdCAoMiwyKSBzaXplIDI3MHgyMCBbcj0wIGM9MCBycz0x
IGNzPTFdCisgICAgICAgICAgICAgICAgUmVuZGVyVGV4dCB7I3RleHR9IGF0ICgxLDEpIHNpemUg
MjY4eDE4CisgICAgICAgICAgICAgICAgICB0ZXh0IHJ1biBhdCAoMSwxKSB3aWR0aCAyNjg6ICJB
bmQgdGhlIGNhcmV0IHNob3VsZCBiZSBpbiB0aGUgc2Vjb25kIG9uZS4iCitjYXJldDogcG9zaXRp
b24gMCBvZiBjaGlsZCAyIHtCUn0gb2YgY2hpbGQgMiB7RElWfSBvZiBib2R5Cg==
</data>
<flag name="review"
          id="95927"
          type_id="1"
          status="+"
          setter="simon.fraser"
    />
          </attachment>
      

    </bug>

</bugzilla>