<?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>207368</bug_id>
          
          <creation_ts>2020-02-06 16:48:31 -0800</creation_ts>
          <short_desc>Add a comment in StringView::setUnderlyingString(const StringView&amp;) for clarification.</short_desc>
          <delta_ts>2020-02-06 17:00:40 -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>JavaScriptCore</component>
          <version>WebKit 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="Mark Lam">mark.lam</reporter>
          <assigned_to name="Mark Lam">mark.lam</assigned_to>
          <cc>benjamin</cc>
    
    <cc>cdumez</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>dbates</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>keith_miller</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1615933</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2020-02-06 16:48:31 -0800</bug_when>
    <thetext>&lt;rdar://problem/59211075&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1615935</commentid>
    <comment_count>1</comment_count>
      <attachid>390029</attachid>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2020-02-06 16:52:12 -0800</bug_when>
    <thetext>Created attachment 390029
proposed patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1615936</commentid>
    <comment_count>2</comment_count>
      <attachid>390029</attachid>
    <who name="Keith Miller">keith_miller</who>
    <bug_when>2020-02-06 16:54:28 -0800</bug_when>
    <thetext>Comment on attachment 390029
proposed patch.

r=me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1615941</commentid>
    <comment_count>3</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2020-02-06 17:00:33 -0800</bug_when>
    <thetext>Thanks for the review.  Landed in r255993: &lt;http://trac.webkit.org/r255993&gt;.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>390029</attachid>
            <date>2020-02-06 16:52:12 -0800</date>
            <delta_ts>2020-02-06 16:54:28 -0800</delta_ts>
            <desc>proposed patch.</desc>
            <filename>bug-207368.patch</filename>
            <type>text/plain</type>
            <size>1758</size>
            <attacher name="Mark Lam">mark.lam</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XVEYvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XVEYvQ2hh
bmdlTG9nCShyZXZpc2lvbiAyNTU5OTIpCisrKyBTb3VyY2UvV1RGL0NoYW5nZUxvZwkod29ya2lu
ZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDIwLTAyLTA2ICBNYXJrIExhbSAgPG1hcmsubGFt
QGFwcGxlLmNvbT4KKworICAgICAgICBBZGQgYSBjb21tZW50IGluIFN0cmluZ1ZpZXc6OnNldFVu
ZGVybHlpbmdTdHJpbmcoY29uc3QgU3RyaW5nVmlldyYpIGZvciBjbGFyaWZpY2F0aW9uLgorICAg
ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjA3MzY4CisgICAg
ICAgIDxyZGFyOi8vcHJvYmxlbS81OTIxMTA3NT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP
RFkgKE9PUFMhKS4KKworICAgICAgICBFdmVyeSB0aW1lIEkgcmVhZCB0aGlzIGNvZGUsIEkgaGF2
ZSB0byByZS1kZWR1Y2Ugd2h5IHRoaXMgY29kZSBpcyB0aHJlYWQgc2FmZS4KKyAgICAgICAgU28s
IEknbSBqdXN0IGdvaW5nIHRvIGFkZCBhIGNvbW1lbnQgdG8gZG9jdW1lbnQgaXQuCisKKyAgICAg
ICAgKiB3dGYvdGV4dC9TdHJpbmdWaWV3LmNwcDoKKyAgICAgICAgKFdURjo6U3RyaW5nVmlldzo6
c2V0VW5kZXJseWluZ1N0cmluZyk6CisKIDIwMjAtMDItMDYgIE15bGVzIEMuIE1heGZpZWxkICA8
bW1heGZpZWxkQGFwcGxlLmNvbT4KIAogICAgICAgICBSRUdSRVNTSU9OKHIyNTQ1MzQpOiAxLTMl
IHJlZ3Jlc3Npb24gb24gUExUCkluZGV4OiBTb3VyY2UvV1RGL3d0Zi90ZXh0L1N0cmluZ1ZpZXcu
Y3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XVEYvd3RmL3RleHQvU3RyaW5nVmlldy5jcHAJKHJl
dmlzaW9uIDI1NTk5MSkKKysrIFNvdXJjZS9XVEYvd3RmL3RleHQvU3RyaW5nVmlldy5jcHAJKHdv
cmtpbmcgY29weSkKQEAgLTM1Niw4ICszNTYsMTQgQEAgdm9pZCBTdHJpbmdWaWV3OjpzZXRVbmRl
cmx5aW5nU3RyaW5nKGNvbgogdm9pZCBTdHJpbmdWaWV3OjpzZXRVbmRlcmx5aW5nU3RyaW5nKGNv
bnN0IFN0cmluZ1ZpZXcmIG90aGVyU3RyaW5nKQogewogICAgIFVuZGVybHlpbmdTdHJpbmcqIHVu
ZGVybHlpbmdTdHJpbmcgPSBvdGhlclN0cmluZy5tX3VuZGVybHlpbmdTdHJpbmc7Ci0gICAgaWYg
KHVuZGVybHlpbmdTdHJpbmcpCisgICAgaWYgKHVuZGVybHlpbmdTdHJpbmcpIHsKKyAgICAgICAg
Ly8gSXQncyBzYWZlIHRvIGluYyB0aGUgcmVmQ291bnQgaGVyZSB3aXRob3V0IGxvY2tpbmcgdW5k
ZXJseWluZ1N0cmluZ3NNdXRleAorICAgICAgICAvLyBiZWNhdXNlIFVuZGVybHlpbmdTdHJpbmc6
OnJlZkNvdW50IGlzIGEgc3RkOjphdG9taWNfdWludCwgYW5kIHdlJ3JlCisgICAgICAgIC8vIGd1
YXJhbnRlZWQgdGhhdCB0aGUgU3RyaW5nVmlldyB3ZSdyZSBjb3B5aW5nIGl0IGZyb20gd2lsbCBh
dCBsZWFzdAorICAgICAgICAvLyBoYXZlIDEgcmVmIG9uIGl0LCB0aGVyZWJ5IGtlZXBpbmcgaXQg
YWxpdmUgcmVnYXJkbGVzcyBvZiB3aGF0IG90aGVyCisgICAgICAgIC8vIHRocmVhZHMgbWF5IGJl
IGRvaW5nLgogICAgICAgICArK3VuZGVybHlpbmdTdHJpbmctPnJlZkNvdW50OworICAgIH0KICAg
ICBhZG9wdFVuZGVybHlpbmdTdHJpbmcodW5kZXJseWluZ1N0cmluZyk7CiB9CiAK
</data>
<flag name="review"
          id="405681"
          type_id="1"
          status="+"
          setter="keith_miller"
    />
          </attachment>
      

    </bug>

</bugzilla>