<?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>83841</bug_id>
          
          <creation_ts>2012-04-12 18:14:59 -0700</creation_ts>
          <short_desc>replaceUsingStringSearch: delay the creation of the replace string until needed</short_desc>
          <delta_ts>2012-04-18 10:08:51 -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>JavaScriptCore</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Benjamin Poulain">benjamin</reporter>
          <assigned_to name="Benjamin Poulain">benjamin</assigned_to>
          <cc>barraclough</cc>
    
    <cc>ggaren</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>601699</commentid>
    <comment_count>0</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2012-04-12 18:14:59 -0700</bug_when>
    <thetext>To avoid a couple of instructions when there is no match.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>601716</commentid>
    <comment_count>1</comment_count>
      <attachid>137016</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2012-04-12 18:26:48 -0700</bug_when>
    <thetext>Created attachment 137016
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>604762</commentid>
    <comment_count>2</comment_count>
      <attachid>137016</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2012-04-18 08:26:07 -0700</bug_when>
    <thetext>Comment on attachment 137016
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>604850</commentid>
    <comment_count>3</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2012-04-18 10:08:51 -0700</bug_when>
    <thetext>Committed r114521: &lt;http://trac.webkit.org/changeset/114521&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>137016</attachid>
            <date>2012-04-12 18:26:48 -0700</date>
            <delta_ts>2012-04-18 08:26:07 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-83841-20120412182647.patch</filename>
            <type>text/plain</type>
            <size>3687</size>
            <attacher name="Benjamin Poulain">benjamin</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTEzOTEzCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBi
MTE3ZDg5NTVkMTUzY2VmZTQ4NGMxMTk0NDU4NTAzZDU3ZmM2MzlhLi5iOTI3ZGI3ZDU3MTc0MDBj
NGM2NzYxNjM4ZTI3YjM0YWM3NzdiMjJmIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxOCBAQAorMjAxMi0wNC0xMiAgQmVuamFtaW4gUG91bGFpbiAgPGJwb3VsYWluQGFwcGxl
LmNvbT4KKworICAgICAgICByZXBsYWNlVXNpbmdTdHJpbmdTZWFyY2g6IGRlbGF5IHRoZSBjcmVh
dGlvbiBvZiB0aGUgcmVwbGFjZSBzdHJpbmcgdW50aWwgbmVlZGVkCisgICAgICAgIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04Mzg0MQorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFdlIGRvIG5vdCBuZWVkIHRvIG9idGFpbiB0
aGUgcmVwbGFjZVZhbHVlIHVudGlsIHdlIGhhdmUgYSBtYXRjaC4gQnkgbW92aW5nIHRoZSBpbnRp
YWxpemF0aW9uCisgICAgICAgIG9mIHJlcGxhY2VWYWx1ZSB3aGVuIG5lZWRlZCwgd2Ugc2F2ZSBh
IGZldyBpbnN0cnVjdGlvbnMgd2hlbiB0aGVyZSBpcyBubyBtYXRjaC4KKworICAgICAgICAqIHJ1
bnRpbWUvU3RyaW5nUHJvdG90eXBlLmNwcDoKKyAgICAgICAgKEpTQzo6cmVwbGFjZVVzaW5nUmVn
RXhwU2VhcmNoKToKKyAgICAgICAgKEpTQzo6cmVwbGFjZVVzaW5nU3RyaW5nU2VhcmNoKToKKyAg
ICAgICAgKEpTQzo6c3RyaW5nUHJvdG9GdW5jUmVwbGFjZSk6CisKIDIwMTItMDQtMTEgIEJlbmph
bWluIFBvdWxhaW4gIDxicG91bGFpbkBhcHBsZS5jb20+CiAKICAgICAgICAgSW1wcm92ZSByZXBs
YWNlVXNpbmdTdHJpbmdTZWFyY2goKSBmb3IgY2FzZSBvZiBhIHNpbmdsZSBjaGFyYWN0ZXIgc2Vh
cmNoVmFsdWUKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL1N0cmlu
Z1Byb3RvdHlwZS5jcHAgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvcnVudGltZS9TdHJpbmdQcm90
b3R5cGUuY3BwCmluZGV4IGYxMWU2NDFjYWE1OTdhMTNhNmM2ZDlmYWUwYjFjOTU0NjEyOGRiNzku
LjExOWEwMjE1NzA5YjgxMmJhZTk2NDQ3YTRkNTI5YjM0ZDI1NDIzZDggMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL1N0cmluZ1Byb3RvdHlwZS5jcHAKKysrIGIvU291
cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvU3RyaW5nUHJvdG90eXBlLmNwcApAQCAtNDQxLDgg
KzQ0MSw5IEBAIHN0YXRpYyBORVZFUl9JTkxJTkUgRW5jb2RlZEpTVmFsdWUgcmVtb3ZlVXNpbmdS
ZWdFeHBTZWFyY2goRXhlY1N0YXRlKiBleGVjLCBKU1N0CiAgICAgcmV0dXJuIEpTVmFsdWU6OmVu
Y29kZShqc1NwbGljZVN1YnN0cmluZ3MoZXhlYywgc3RyaW5nLCBzb3VyY2UsIHNvdXJjZVJhbmdl
cy5kYXRhKCksIHNvdXJjZVJhbmdlcy5zaXplKCkpKTsKIH0KIAotc3RhdGljIE5FVkVSX0lOTElO
RSBFbmNvZGVkSlNWYWx1ZSByZXBsYWNlVXNpbmdSZWdFeHBTZWFyY2goRXhlY1N0YXRlKiBleGVj
LCBKU1N0cmluZyogc3RyaW5nLCBKU1ZhbHVlIHNlYXJjaFZhbHVlLCBKU1ZhbHVlIHJlcGxhY2VW
YWx1ZSkKK3N0YXRpYyBORVZFUl9JTkxJTkUgRW5jb2RlZEpTVmFsdWUgcmVwbGFjZVVzaW5nUmVn
RXhwU2VhcmNoKEV4ZWNTdGF0ZSogZXhlYywgSlNTdHJpbmcqIHN0cmluZywgSlNWYWx1ZSBzZWFy
Y2hWYWx1ZSkKIHsKKyAgICBKU1ZhbHVlIHJlcGxhY2VWYWx1ZSA9IGV4ZWMtPmFyZ3VtZW50KDEp
OwogICAgIFVTdHJpbmcgcmVwbGFjZW1lbnRTdHJpbmc7CiAgICAgQ2FsbERhdGEgY2FsbERhdGE7
CiAgICAgQ2FsbFR5cGUgY2FsbFR5cGUgPSBnZXRDYWxsRGF0YShyZXBsYWNlVmFsdWUsIGNhbGxE
YXRhKTsKQEAgLTYyNSw3ICs2MjYsNyBAQCBzdGF0aWMgTkVWRVJfSU5MSU5FIEVuY29kZWRKU1Zh
bHVlIHJlcGxhY2VVc2luZ1JlZ0V4cFNlYXJjaChFeGVjU3RhdGUqIGV4ZWMsIEpTUwogICAgIHJl
dHVybiBKU1ZhbHVlOjplbmNvZGUoanNTcGxpY2VTdWJzdHJpbmdzV2l0aFNlcGFyYXRvcnMoZXhl
Yywgc3RyaW5nLCBzb3VyY2UsIHNvdXJjZVJhbmdlcy5kYXRhKCksIHNvdXJjZVJhbmdlcy5zaXpl
KCksIHJlcGxhY2VtZW50cy5kYXRhKCksIHJlcGxhY2VtZW50cy5zaXplKCkpKTsKIH0KIAotc3Rh
dGljIGlubGluZSBFbmNvZGVkSlNWYWx1ZSByZXBsYWNlVXNpbmdTdHJpbmdTZWFyY2goRXhlY1N0
YXRlKiBleGVjLCBKU1N0cmluZyoganNTdHJpbmcsIEpTVmFsdWUgc2VhcmNoVmFsdWUsIEpTVmFs
dWUgcmVwbGFjZVZhbHVlKQorc3RhdGljIGlubGluZSBFbmNvZGVkSlNWYWx1ZSByZXBsYWNlVXNp
bmdTdHJpbmdTZWFyY2goRXhlY1N0YXRlKiBleGVjLCBKU1N0cmluZyoganNTdHJpbmcsIEpTVmFs
dWUgc2VhcmNoVmFsdWUpCiB7CiAgICAgY29uc3QgVVN0cmluZyYgc3RyaW5nID0ganNTdHJpbmct
PnZhbHVlKGV4ZWMpOwogICAgIFVTdHJpbmcgc2VhcmNoU3RyaW5nID0gc2VhcmNoVmFsdWUudG9V
U3RyaW5nKGV4ZWMpOwpAQCAtNjUwLDYgKzY1MSw3IEBAIHN0YXRpYyBpbmxpbmUgRW5jb2RlZEpT
VmFsdWUgcmVwbGFjZVVzaW5nU3RyaW5nU2VhcmNoKEV4ZWNTdGF0ZSogZXhlYywgSlNTdHJpbmcq
CiAgICAgaWYgKG1hdGNoU3RhcnQgPT0gbm90Rm91bmQpCiAgICAgICAgIHJldHVybiBKU1ZhbHVl
OjplbmNvZGUoanNTdHJpbmcpOwogCisgICAgSlNWYWx1ZSByZXBsYWNlVmFsdWUgPSBleGVjLT5h
cmd1bWVudCgxKTsKICAgICBDYWxsRGF0YSBjYWxsRGF0YTsKICAgICBDYWxsVHlwZSBjYWxsVHlw
ZSA9IGdldENhbGxEYXRhKHJlcGxhY2VWYWx1ZSwgY2FsbERhdGEpOwogICAgIGlmIChjYWxsVHlw
ZSAhPSBDYWxsVHlwZU5vbmUpIHsKQEAgLTY4NSwxMSArNjg3LDEwIEBAIEVuY29kZWRKU1ZhbHVl
IEpTQ19IT1NUX0NBTEwgc3RyaW5nUHJvdG9GdW5jUmVwbGFjZShFeGVjU3RhdGUqIGV4ZWMpCiAg
ICAgICAgIHJldHVybiB0aHJvd1ZNVHlwZUVycm9yKGV4ZWMpOwogICAgIEpTU3RyaW5nKiBzdHJp
bmcgPSB0aGlzVmFsdWUudG9TdHJpbmcoZXhlYyk7CiAgICAgSlNWYWx1ZSBzZWFyY2hWYWx1ZSA9
IGV4ZWMtPmFyZ3VtZW50KDApOwotICAgIEpTVmFsdWUgcmVwbGFjZVZhbHVlID0gZXhlYy0+YXJn
dW1lbnQoMSk7CiAKICAgICBpZiAoc2VhcmNoVmFsdWUuaW5oZXJpdHMoJlJlZ0V4cE9iamVjdDo6
c19pbmZvKSkKLSAgICAgICAgcmV0dXJuIHJlcGxhY2VVc2luZ1JlZ0V4cFNlYXJjaChleGVjLCBz
dHJpbmcsIHNlYXJjaFZhbHVlLCByZXBsYWNlVmFsdWUpOwotICAgIHJldHVybiByZXBsYWNlVXNp
bmdTdHJpbmdTZWFyY2goZXhlYywgc3RyaW5nLCBzZWFyY2hWYWx1ZSwgcmVwbGFjZVZhbHVlKTsK
KyAgICAgICAgcmV0dXJuIHJlcGxhY2VVc2luZ1JlZ0V4cFNlYXJjaChleGVjLCBzdHJpbmcsIHNl
YXJjaFZhbHVlKTsKKyAgICByZXR1cm4gcmVwbGFjZVVzaW5nU3RyaW5nU2VhcmNoKGV4ZWMsIHN0
cmluZywgc2VhcmNoVmFsdWUpOwogfQogCiBFbmNvZGVkSlNWYWx1ZSBKU0NfSE9TVF9DQUxMIHN0
cmluZ1Byb3RvRnVuY1RvU3RyaW5nKEV4ZWNTdGF0ZSogZXhlYykK
</data>
<flag name="review"
          id="141982"
          type_id="1"
          status="+"
          setter="ggaren"
    />
          </attachment>
      

    </bug>

</bugzilla>