<?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>7433</bug_id>
          
          <creation_ts>2006-02-23 14:56:54 -0800</creation_ts>
          <short_desc>REGRESSION (r12789): Second RTL text run on a line cannot be selected</short_desc>
          <delta_ts>2006-03-01 09:34:36 -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>Layout and Rendering</component>
          <version>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>VERIFIED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>HasReduction, Regression</keywords>
          <priority>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>mitz</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>harrison</cc>
    
    <cc>justin.garcia</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>33806</commentid>
    <comment_count>0</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-02-23 14:56:54 -0800</bug_when>
    <thetext>Clicking or dragging to select the second run and subsequent runs of RTL text on a line fails. See the test case for an example where you cannot drag or double-click to select the rightmost two words of Hebrew text, since they are separated from the first two words by a double space (causing them to be in a different text run).

Using nightly builds I tracked it down to somewhere between r12786 and r12791, and r12789 looks like the most likely candidate to be the cause.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>33807</commentid>
    <comment_count>1</comment_count>
      <attachid>6687</attachid>
    <who name="">mitz</who>
    <bug_when>2006-02-23 14:57:40 -0800</bug_when>
    <thetext>Created attachment 6687
Test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>33864</commentid>
    <comment_count>2</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-02-24 07:48:50 -0800</bug_when>
    <thetext>This bug is a result of Position::inRenderedText()&apos;s assumption that RenderText&apos;s inline boxes are ordered by their start offset. This assumption is false in many bidi contexts.

The same logic is used in at least two other places: Position::isRenderedCharacter() and RenderText::inlineBox().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34064</commentid>
    <comment_count>3</comment_count>
      <attachid>6746</attachid>
    <who name="">mitz</who>
    <bug_when>2006-02-26 12:10:37 -0800</bug_when>
    <thetext>Created attachment 6746
Fix the regression

This fixes inRenderedText() (which is responsible for this bug) and isRenderedCharacter(), but not other places where logical ordering of the text boxes is wrongly assumed. Those seem to be in editing code and require separate bugs and test cases.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34075</commentid>
    <comment_count>4</comment_count>
      <attachid>6746</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-02-26 15:40:20 -0800</bug_when>
    <thetext>Comment on attachment 6746
Fix the regression

These fixes seem to leave in an optimization for renderers that don&apos;t contain reversed text. But couldn&apos;t we just remove the optimization entirely?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34079</commentid>
    <comment_count>5</comment_count>
    <who name="David Harrison">harrison</who>
    <bug_when>2006-02-26 17:16:12 -0800</bug_when>
    <thetext>I would think so.  Before my patch, Position::inRenderedContent had this optimization but VisiblePosition::isCandidate() did not.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34136</commentid>
    <comment_count>6</comment_count>
      <attachid>6746</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-02-27 08:20:53 -0800</bug_when>
    <thetext>Comment on attachment 6746
Fix the regression

Lets get this fix landed -- we can ponder whether to remove the regression altogether later.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34215</commentid>
    <comment_count>7</comment_count>
      <attachid>6746</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-02-27 10:58:28 -0800</bug_when>
    <thetext>Comment on attachment 6746
Fix the regression

I meant to say &quot;we can ponder whether to remove the *optimization* altogether later&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34649</commentid>
    <comment_count>8</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-03-01 09:34:36 -0800</bug_when>
    <thetext>Verified in r13060 nightly</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>6687</attachid>
            <date>2006-02-23 14:57:40 -0800</date>
            <delta_ts>2006-02-23 14:57:40 -0800</delta_ts>
            <desc>Test case</desc>
            <filename>RTL selection regression.html</filename>
            <type>text/html</type>
            <size>380</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">/v8APABoAHQAbQBsAD4ACgA8AGgAZQBhAGQAPgAKADwAdABpAHQAbABlAD4AUgBUAEwAIABzAGUA
bABlAGMAdABpAG8AbgAgAHAAcgBvAGIAbABlAG0APAAvAHQAaQB0AGwAZQA+AAoAPAAvAGgAZQBh
AGQAPgAKADwAYgBvAGQAeQA+AAoAPABwAD4ACgBUAHIAeQAgAHMAZQBsAGUAYwB0AGkAbgBnACAA
ZQB2AGUAcgB5AHQAaABpAG4AZwAgAG8AbgAgAHQAaABlACAAbgBlAHgAdAAgAGwAaQBuAGUAIABi
AHkAIABkAHIAYQBnAGcAaQBuAGcALAAgAGMAbABpAGMAawBpAG4AZwAsACAAZQB0AGMALgAKADwA
LwBwAD4ACgA8AHAAPgAKBdAF2wXcBeoAIAXkBdwF5AXcACAAIAXpBeoF1AAgBd4F2QXlAAoAPAAv
AHAAPgAKADwALwBiAG8AZAB5AD4ACgA8AC8AaAB0AG0AbAA+AAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>6746</attachid>
            <date>2006-02-26 12:10:37 -0800</date>
            <delta_ts>2006-02-27 08:20:53 -0800</delta_ts>
            <desc>Fix the regression</desc>
            <filename>7433_r1.patch</filename>
            <type>text/plain</type>
            <size>7269</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDEyOTkwKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMDYtIyMtIyMgIE1pdHogUGV0dGVsICA8b3Bl
bmRhcndpbi5vcmdAbWl0enBldHRlbC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgLSB0ZXN0IGZvciBodHRwOi8vYnVnemlsbGEub3BlbmRhcndp
bi5vcmcvc2hvd19idWcuY2dpP2lkPTc0MzMKKyAgICAgICAgICBSRUdSRVNTSU9OIChyMTI3ODkp
OiBTZWNvbmQgUlRMIHRleHQgcnVuIG9uIGEgbGluZSBjYW5ub3QgYmUgc2VsZWN0ZWQKKworICAg
ICAgICAqIGZhc3QvdGV4dC9pbi1yZW5kZXJlZC10ZXh0LXJ0bC1leHBlY3RlZC5jaGVja3N1bTog
QWRkZWQuCisgICAgICAgICogZmFzdC90ZXh0L2luLXJlbmRlcmVkLXRleHQtcnRsLWV4cGVjdGVk
LnBuZzogQWRkZWQuCisgICAgICAgICogZmFzdC90ZXh0L2luLXJlbmRlcmVkLXRleHQtcnRsLWV4
cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC90ZXh0L2luLXJlbmRlcmVkLXRleHQt
cnRsLmh0bWw6IEFkZGVkLgorCiAyMDA2LTAyLTI1ICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBh
cHBsZS5jb20+CiAKICAgICAgICAgUlMgYnkgbWpzLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC90
ZXh0L2luLXJlbmRlcmVkLXRleHQtcnRsLWV4cGVjdGVkLmNoZWNrc3VtCj09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t
IExheW91dFRlc3RzL2Zhc3QvdGV4dC9pbi1yZW5kZXJlZC10ZXh0LXJ0bC1leHBlY3RlZC5jaGVj
a3N1bQkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvdGV4dC9pbi1yZW5kZXJlZC10
ZXh0LXJ0bC1leHBlY3RlZC5jaGVja3N1bQkocmV2aXNpb24gMCkKQEAgLTAsMCArMSBAQAorYjM4
ZDYyOGMwOTY3ZmMyYWQ3NTdmNjU0NDczNTQ1YWUKXCBObyBuZXdsaW5lIGF0IGVuZCBvZiBmaWxl
CkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L3RleHQvaW4tcmVuZGVyZWQtdGV4dC1ydGwtZXhwZWN0
ZWQucG5nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT0KQ2Fubm90IGRpc3BsYXk6IGZpbGUgbWFya2VkIGFzIGEgYmluYXJ5
IHR5cGUuCnN2bjptaW1lLXR5cGUgPSBhcHBsaWNhdGlvbi9vY3RldC1zdHJlYW0KClByb3BlcnR5
IGNoYW5nZXMgb246IGluLXJlbmRlcmVkLXRleHQtcnRsLWV4cGVjdGVkLnBuZwpfX19fX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
Ck5hbWU6IHN2bjptaW1lLXR5cGUKICAgKyBhcHBsaWNhdGlvbi9vY3RldC1zdHJlYW0KCkluZGV4
OiBMYXlvdXRUZXN0cy9mYXN0L3RleHQvaW4tcmVuZGVyZWQtdGV4dC1ydGwtZXhwZWN0ZWQudHh0
Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvdGV4dC9pbi1yZW5kZXJlZC10ZXh0LXJ0
bC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L3RleHQvaW4t
cmVuZGVyZWQtdGV4dC1ydGwtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDMy
IEBACitFRElUSU5HIERFTEVHQVRFOiBzaG91bGRDaGFuZ2VTZWxlY3RlZERPTVJhbmdlOihudWxs
KSB0b0RPTVJhbmdlOnJhbmdlIGZyb20gMyBvZiAjdGV4dCA+IFNQQU4gPiBCT0RZID4gSFRNTCA+
ICNkb2N1bWVudCB0byAzIG9mICN0ZXh0ID4gU1BBTiA+IEJPRFkgPiBIVE1MID4gI2RvY3VtZW50
IGFmZmluaXR5Ok5TU2VsZWN0aW9uQWZmaW5pdHlEb3duc3RyZWFtIHN0aWxsU2VsZWN0aW5nOkZB
TFNFCitFRElUSU5HIERFTEVHQVRFOiB3ZWJWaWV3RGlkQ2hhbmdlU2VsZWN0aW9uOldlYlZpZXdE
aWRDaGFuZ2VTZWxlY3Rpb25Ob3RpZmljYXRpb24KK2xheWVyIGF0ICgwLDApIHNpemUgODAweDYw
MAorICBSZW5kZXJDYW52YXMgYXQgKDAsMCkgc2l6ZSA4MDB4NjAwCitsYXllciBhdCAoMCwwKSBz
aXplIDgwMHg2MDAKKyAgUmVuZGVyQmxvY2sge0hUTUx9IGF0ICgwLDApIHNpemUgODAweDYwMAor
ICAgIFJlbmRlckJvZHkge0JPRFl9IGF0ICg4LDgpIHNpemUgNzg0eDU4NAorICAgICAgUmVuZGVy
QmxvY2sge1B9IGF0ICgwLDApIHNpemUgNzg0eDM2CisgICAgICAgIFJlbmRlclRleHQge1RFWFR9
IGF0ICgwLDApIHNpemUgNTN4MTgKKyAgICAgICAgICB0ZXh0IHJ1biBhdCAoMCwwKSB3aWR0aCA1
MzogIlRlc3QgZm9yICIKKyAgICAgICAgUmVuZGVySW5saW5lIHtJfSBhdCAoMCwwKSBzaXplIDc1
OXgzNgorICAgICAgICAgIFJlbmRlcklubGluZSB7QX0gYXQgKDAsMCkgc2l6ZSAzNDh4MTggW2Nv
bG9yPSMwMDAwRUVdCisgICAgICAgICAgICBSZW5kZXJUZXh0IHtURVhUfSBhdCAoNTMsMCkgc2l6
ZSAzNDh4MTgKKyAgICAgICAgICAgICAgdGV4dCBydW4gYXQgKDUzLDApIHdpZHRoIDM0ODogImh0
dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzQzMyIKKyAgICAg
ICAgICBSZW5kZXJUZXh0IHtURVhUfSBhdCAoNDAxLDApIHNpemUgNzU5eDM2CisgICAgICAgICAg
ICB0ZXh0IHJ1biBhdCAoNDAxLDApIHdpZHRoIDM1ODogIiBSRUdSRVNTSU9OIChyMTI3ODkpOiBT
ZWNvbmQgUlRMIHRleHQgcnVuIG9uIGEgbGluZSIKKyAgICAgICAgICAgIHRleHQgcnVuIGF0ICgw
LDE4KSB3aWR0aCAxMTY6ICJjYW5ub3QgYmUgc2VsZWN0ZWQiCisgICAgICAgIFJlbmRlclRleHQg
e1RFWFR9IGF0ICgxMTYsMTgpIHNpemUgNHgxOAorICAgICAgICAgIHRleHQgcnVuIGF0ICgxMTYs
MTgpIHdpZHRoIDQ6ICIuIgorICAgICAgUmVuZGVyQmxvY2sge1B9IGF0ICgwLDUyKSBzaXplIDc4
NHgxOAorICAgICAgICBSZW5kZXJUZXh0IHtURVhUfSBhdCAoMCwwKSBzaXplIDY0OHgxOAorICAg
ICAgICAgIHRleHQgcnVuIGF0ICgwLDApIHdpZHRoIDY0ODogIlRoZSByaWdodG1vc3QgdHdvIHdv
cmRzIGluIHRoZSBIZWJyZXcgdGV4dCBzaG91bGQgYmUgc2VsZWN0YWJsZSBieSBkcmFnZ2luZyBv
ciBkb3VibGUtY2xpY2tpbmcgdGhlbS4iCisgICAgICBSZW5kZXJCbG9jayB7SFJ9IGF0ICgwLDg2
KSBzaXplIDc4NHgyIFtib3JkZXI6ICgxcHggaW5zZXQgIzAwMDAwMCldCisgICAgICBSZW5kZXJC
bG9jayAoYW5vbnltb3VzKSBhdCAoMCw5Nikgc2l6ZSA3ODR4MTgKKyAgICAgICAgUmVuZGVySW5s
aW5lIHtTUEFOfSBhdCAoMCwwKSBzaXplIDE0NngxOAorICAgICAgICAgIFJlbmRlclRleHQge1RF
WFR9IGF0ICgwLDApIHNpemUgMTQ2eDE4CisgICAgICAgICAgICB0ZXh0IHJ1biBhdCAoMCwwKSB3
aWR0aCA2MiBSVEw6ICJceHs1RTl9XHh7NUVBfVx4ezVENH0gXHh7NURFfVx4ezVEOX1ceHs1RTV9
IgorICAgICAgICAgICAgdGV4dCBydW4gYXQgKDYyLDApIHdpZHRoIDg0IFJUTDogIlx4ezVEMH1c
eHs1REJ9XHh7NURDfVx4ezVFQX0gXHh7NUU0fVx4ezVEQ31ceHs1RTR9XHh7NURDfSAiCisgICAg
ICAgIFJlbmRlclRleHQge1RFWFR9IGF0ICgwLDApIHNpemUgMHgwCisgICAgICAgIFJlbmRlclRl
eHQge1RFWFR9IGF0ICgwLDApIHNpemUgMHgwCisgICAgICAgIFJlbmRlclRleHQge1RFWFR9IGF0
ICgwLDApIHNpemUgMHgwCitjYXJldDogcG9zaXRpb24gMyBvZiBjaGlsZCAwIHtURVhUfSBvZiBj
aGlsZCA3IHtTUEFOfSBvZiBjaGlsZCAxIHtCT0RZfSBvZiBjaGlsZCAwIHtIVE1MfSBvZiBkb2N1
bWVudApJbmRleDogTGF5b3V0VGVzdHMvZmFzdC90ZXh0L2luLXJlbmRlcmVkLXRleHQtcnRsLmh0
bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC90ZXh0L2luLXJlbmRlcmVkLXRleHQt
cnRsLmh0bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L3RleHQvaW4tcmVuZGVy
ZWQtdGV4dC1ydGwuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwzMSBAQAorPGh0bWw+Cis8
aGVhZD4KKzx0aXRsZT48L3RpdGxlPgorPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgor
ZnVuY3Rpb24gdGVzdCgpCit7CisgICAgaWYgKCF3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIp
CisgICAgICAgIHJldHVybjsKKyAgICAvLyBKdXN0IGNsaWNrIG9uIHRoZSBmaXJzdCAocmlnaHRt
b3N0KSB3b3JkIHRvIGdldCBhIHJlc3BvbnNlIGZyb20gdGhlCisgICAgLy8gZWRpdGluZyBkZWxl
Z2F0ZS4KKyAgICB2YXIgdCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCd0Jyk7CisgICAgZXZl
bnRTZW5kZXIubW91c2VNb3ZlVG8odC5vZmZzZXRXaWR0aCAtIDgsIHQub2Zmc2V0VG9wICsgMTYp
OworICAgIGV2ZW50U2VuZGVyLm1vdXNlRG93bigpOworICAgIGV2ZW50U2VuZGVyLm1vdXNlVXAo
KTsKK30KKzwvc2NyaXB0PgorPC9oZWFkPgorPGJvZHkgb25sb2FkPSJ0ZXN0KCkiPgorPHA+CitU
ZXN0IGZvciA8aT48YSBocmVmPSJodHRwOi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19i
dWcuY2dpP2lkPTc0MzMiPmh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9NzQzMzwvYT4KK1JFR1JFU1NJT04gKHIxMjc4OSk6IFNlY29uZCBSVEwgdGV4dCBydW4g
b24gYSBsaW5lIGNhbm5vdCBiZSBzZWxlY3RlZDwvaT4uCis8L3A+Cis8cD4KK1RoZSByaWdodG1v
c3QgdHdvIHdvcmRzIGluIHRoZSBIZWJyZXcgdGV4dCBzaG91bGQgYmUgc2VsZWN0YWJsZSBieSBk
cmFnZ2luZyBvciBkb3VibGUtY2xpY2tpbmcgdGhlbS4KKzwvcD4KKzxocj4KKzxzcGFuIGlkPSJ0
Ij4KKyYjeDA1ZDA7JiN4MDVkYjsmI3gwNWRjOyYjeDA1ZWEgJiN4MDVlNDsmI3gwNWRjOyYjeDA1
ZTQ7JiN4MDVkYzsgICYjeDA1ZTk7JiN4MDVlYTsmI3gwNWQ0OyAmI3gwNWRlOyYjeDA1ZDk7JiN4
MDVlNQorPC9zcGFuPgorPC9ib2R5PgorPC9odG1sPgpJbmRleDogV2ViQ29yZS9DaGFuZ2VMb2cK
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PQotLS0gV2ViQ29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEyOTkwKQorKysgV2Vi
Q29yZS9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxOCBAQAorMjAwNi0jIy0j
IyAgTWl0eiBQZXR0ZWwgIDxvcGVuZGFyd2luLm9yZ0BtaXR6cGV0dGVsLmNvbT4KKworICAgICAg
ICBUZXN0OiBmYXN0L3RleHQvaW4tcmVuZGVyZWQtdGV4dC1ydGwuaHRtbAorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIC0gZml4IGh0dHA6Ly9idWd6aWxs
YS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzQzMworICAgICAgICAgIFJFR1JFU1NJ
T04gKHIxMjc4OSk6IFNlY29uZCBSVEwgdGV4dCBydW4gb24gYSBsaW5lIGNhbm5vdCBiZSBzZWxl
Y3RlZAorCisgICAgICAgICogZG9tL2RvbV9wb3NpdGlvbi5jcHA6CisgICAgICAgIChXZWJDb3Jl
OjpQb3NpdGlvbjo6aW5SZW5kZXJlZFRleHQpOiBJZiB0aGUgUmVuZGVyVGV4dCBjb250YWlucwor
ICAgICAgICByZXZlcnNlZCB0ZXh0LCB0aGUgdGV4dCBib3hlcyBhcmUgbm90IG5lY2Vzc2FyaWx5
IG9yZGVyZWQgYnkgc3RhcnQKKyAgICAgICAgcG9zaXRpb24sIHNvIGl0ZXJhdGUgdGhyb3VnaCBh
bGwgb2YgdGhlbS4KKyAgICAgICAgKFdlYkNvcmU6OlBvc2l0aW9uOjppc1JlbmRlcmVkQ2hhcmFj
dGVyKTogRGl0dG8uCisKIDIwMDYtMDItMjQgIEFudHRpIEtvaXZpc3RvICA8a29pdmlzdG9AaWtp
LmZpPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEh5YXR0LgpJbmRleDogV2ViQ29yZS9kb20vZG9t
X3Bvc2l0aW9uLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2RvbS9kb21fcG9zaXRpb24uY3Bw
CShyZXZpc2lvbiAxMjk4OCkKKysrIFdlYkNvcmUvZG9tL2RvbV9wb3NpdGlvbi5jcHAJKHdvcmtp
bmcgY29weSkKQEAgLTUzMiw3ICs1MzIsNyBAQCBib29sIFBvc2l0aW9uOjppblJlbmRlcmVkVGV4
dCgpIGNvbnN0CiAgICAgCiAgICAgUmVuZGVyVGV4dCAqdGV4dFJlbmRlcmVyID0gc3RhdGljX2Nh
c3Q8UmVuZGVyVGV4dCAqPihyZW5kZXJlcik7CiAgICAgZm9yIChJbmxpbmVUZXh0Qm94ICpib3gg
PSB0ZXh0UmVuZGVyZXItPmZpcnN0VGV4dEJveCgpOyBib3g7IGJveCA9IGJveC0+bmV4dFRleHRC
b3goKSkgewotICAgICAgICBpZiAob2Zmc2V0KCkgPCBib3gtPm1fc3RhcnQpIHsKKyAgICAgICAg
aWYgKG9mZnNldCgpIDwgYm94LT5tX3N0YXJ0ICYmICF0ZXh0UmVuZGVyZXItPmNvbnRhaW5zUmV2
ZXJzZWRUZXh0KCkpIHsKICAgICAgICAgICAgIC8vIFRoZSBvZmZzZXQgd2UncmUgbG9va2luZyBm
b3IgaXMgYmVmb3JlIHRoaXMgbm9kZQogICAgICAgICAgICAgLy8gdGhpcyBtZWFucyB0aGUgb2Zm
c2V0IG11c3QgYmUgaW4gY29udGVudCB0aGF0IGlzCiAgICAgICAgICAgICAvLyBub3QgcmVuZGVy
ZWQuIFJldHVybiBmYWxzZS4KQEAgLTU1Niw3ICs1NTYsNyBAQCBib29sIFBvc2l0aW9uOjppc1Jl
bmRlcmVkQ2hhcmFjdGVyKCkgY29uCiAgICAgCiAgICAgUmVuZGVyVGV4dCAqdGV4dFJlbmRlcmVy
ID0gc3RhdGljX2Nhc3Q8UmVuZGVyVGV4dCAqPihyZW5kZXJlcik7CiAgICAgZm9yIChJbmxpbmVU
ZXh0Qm94ICpib3ggPSB0ZXh0UmVuZGVyZXItPmZpcnN0VGV4dEJveCgpOyBib3g7IGJveCA9IGJv
eC0+bmV4dFRleHRCb3goKSkgewotICAgICAgICBpZiAob2Zmc2V0KCkgPCBib3gtPm1fc3RhcnQp
IHsKKyAgICAgICAgaWYgKG9mZnNldCgpIDwgYm94LT5tX3N0YXJ0ICYmICF0ZXh0UmVuZGVyZXIt
PmNvbnRhaW5zUmV2ZXJzZWRUZXh0KCkpIHsKICAgICAgICAgICAgIC8vIFRoZSBvZmZzZXQgd2Un
cmUgbG9va2luZyBmb3IgaXMgYmVmb3JlIHRoaXMgbm9kZQogICAgICAgICAgICAgLy8gdGhpcyBt
ZWFucyB0aGUgb2Zmc2V0IG11c3QgYmUgaW4gY29udGVudCB0aGF0IGlzCiAgICAgICAgICAgICAv
LyBub3QgcmVuZGVyZWQuIFJldHVybiBmYWxzZS4K
</data>
<flag name="review"
          id="1715"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>