<?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>234600</bug_id>
          
          <creation_ts>2021-12-22 05:52:45 -0800</creation_ts>
          <short_desc>Simplify test for startOfLastParagraph in InsertListCommand::doApply</short_desc>
          <delta_ts>2021-12-24 09:09:15 -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>HTML Editing</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="Frédéric Wang Nélar">fred.wang</reporter>
          <assigned_to name="Frédéric Wang Nélar">fred.wang</assigned_to>
          <cc>darin</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>mifenton</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>wenson_hsieh</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1826273</commentid>
    <comment_count>0</comment_count>
    <who name="Frédéric Wang Nélar">fred.wang</who>
    <bug_when>2021-12-22 05:52:45 -0800</bug_when>
    <thetext>Currently, we are doing

!startOfLastParagraph.deepEquivalent().anchorNode()-&gt;isConnected()

but it could be replaced with

startOfLastParagraph.isOrphan()

which is the same as

startOfLastParagraph.deepEquivalent().isOrphan()

and so the same as

startOfLastParagraph.deepEquivalent().anchorNode() &amp;&amp; !startOfLastParagraph.deepEquivalent().anchorNode()-&gt;isConnected()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1826276</commentid>
    <comment_count>1</comment_count>
      <attachid>447798</attachid>
    <who name="Frédéric Wang Nélar">fred.wang</who>
    <bug_when>2021-12-22 06:01:48 -0800</bug_when>
    <thetext>Created attachment 447798
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1826730</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-12-24 09:08:12 -0800</bug_when>
    <thetext>Committed r287428 (245563@main): &lt;https://commits.webkit.org/245563@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 447798.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1826731</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-12-24 09:09:15 -0800</bug_when>
    <thetext>&lt;rdar://problem/86894407&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>447798</attachid>
            <date>2021-12-22 06:01:48 -0800</date>
            <delta_ts>2021-12-24 09:08:13 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>0001-Bug-234600-Simplify-test-for-startOfLastParagraph-in.patch</filename>
            <type>text/plain</type>
            <size>2605</size>
            <attacher name="Frédéric Wang Nélar">fred.wang</attacher>
            
              <data encoding="base64">RnJvbSA5YzIyYzAzODFmZjhlNTc5YzgzZTcxMDc3ZDFhNTI4OWNkOWUxNmNiIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiA9P1VURi04P3E/RnI9QzM9QTlkPUMzPUE5cmljPTIwV2FuZz89
IDxmd2FuZ0BpZ2FsaWEuY29tPgpEYXRlOiBXZWQsIDIyIERlYyAyMDIxIDE1OjAwOjQzICswMTAw
ClN1YmplY3Q6IFtQQVRDSF0gQnVnIDIzNDYwMCAtIFNpbXBsaWZ5IHRlc3QgZm9yIHN0YXJ0T2ZM
YXN0UGFyYWdyYXBoIGluCiBJbnNlcnRMaXN0Q29tbWFuZDo6ZG9BcHBseQoKLS0tCiBTb3VyY2Uv
V2ViQ29yZS9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAgICB8IDIwICsrKysrKysrKysrKysr
KysrKysrCiBTb3VyY2UvV2ViQ29yZS9lZGl0aW5nL0luc2VydExpc3RDb21tYW5kLmNwcCB8ICAy
ICstCiAyIGZpbGVzIGNoYW5nZWQsIDIxIGluc2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkKCmRp
ZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFu
Z2VMb2cKaW5kZXggZDM0ZGQ0NjEwYy4uNjY4NDU4MDBmOCAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YkNvcmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsx
LDIzIEBACisyMDIxLTEyLTIyICBGcmVkZXJpYyBXYW5nICA8ZndhbmdAaWdhbGlhLmNvbT4KKwor
ICAgICAgICBTaW1wbGlmeSB0ZXN0IGZvciBzdGFydE9mTGFzdFBhcmFncmFwaCBpbiBJbnNlcnRM
aXN0Q29tbWFuZDo6ZG9BcHBseQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MjM0NjAwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgQ3VycmVudCB0ZXN0IGlzICFzdGFydE9mTGFzdFBhcmFncmFwaC5kZWVwRXF1
aXZhbGVudCgpLmFuY2hvck5vZGUoKS0+aXNDb25uZWN0ZWQoKS4gSXQgY291bGQKKyAgICAgICAg
YmUgcmVwbGFjZWQgd2l0aCB0aGUgbGVzcyB2ZXJib3NlIHN0YXJ0T2ZMYXN0UGFyYWdyYXBoLmlz
T3JwaGFuKCkgd2hpY2ggaXMgZXF1aXZhbGVudCB0bworICAgICAgICBzdGFydE9mTGFzdFBhcmFn
cmFwaC5kZWVwRXF1aXZhbGVudCgpLmlzT3JwaGFuKCkgYW5kIHNvIHRvIHRoZSBjb25kaXRpb24K
KyAgICAgICAgc3RhcnRPZkxhc3RQYXJhZ3JhcGguZGVlcEVxdWl2YWxlbnQoKS5hbmNob3JOb2Rl
KCkgJiYKKyAgICAgICAgIXN0YXJ0T2ZMYXN0UGFyYWdyYXBoLmRlZXBFcXVpdmFsZW50KCkuYW5j
aG9yTm9kZSgpLT5pc0Nvbm5lY3RlZCgpLiBBIGRpZmZlcmVuY2UgY2FuIG9ubHkKKyAgICAgICAg
aGFwcGVuIGZvciBhIHBhZ2Ugd2hlcmUgYW5jaG9yTm9kZSgpIGlzIGEgbnVsbCBwb2ludGVyLiBJ
biBzdWNoIGEgdGVzdCBjYXNlIGV4aXN0cywgdGhlbiBpdAorICAgICAgICBpcyBhbHJlYWR5IGNy
YXNoaW5nIHdpdGggYSBudWxsIHB0ciBkZXJlZiBhbnl3YXkuCisKKyAgICAgICAgTm8gbmV3IHRl
c3RzLCBiZWhhdmlvciBlc3NlbnRpYWxseSB1bmNoYW5nZWQuCisKKyAgICAgICAgKiBlZGl0aW5n
L0luc2VydExpc3RDb21tYW5kLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Okluc2VydExpc3RDb21t
YW5kOjpkb0FwcGx5KTogVXNlIHN0YXJ0T2ZMYXN0UGFyYWdyYXBoLmlzT3JwaGFuKCkuCisKIDIw
MjEtMTItMTkgIEJyYWR5IEVpZHNvbiAgPGJlaWRzb25AYXBwbGUuY29tPgogCiAgICAgICAgIFN0
dWIgb3V0IE5vdGlmaWNhdGlvbkV2ZW50IGFuZCByZWxhdGVkCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViQ29yZS9lZGl0aW5nL0luc2VydExpc3RDb21tYW5kLmNwcCBiL1NvdXJjZS9XZWJDb3JlL2Vk
aXRpbmcvSW5zZXJ0TGlzdENvbW1hbmQuY3BwCmluZGV4IGZiNzRkNmQ4ZDQuLjY5M2ZiODUyMWUg
MTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2VkaXRpbmcvSW5zZXJ0TGlzdENvbW1hbmQuY3Bw
CisrKyBiL1NvdXJjZS9XZWJDb3JlL2VkaXRpbmcvSW5zZXJ0TGlzdENvbW1hbmQuY3BwCkBAIC0x
NTYsNyArMTU2LDcgQEAgdm9pZCBJbnNlcnRMaXN0Q29tbWFuZDo6ZG9BcHBseSgpCiAgICAgICAg
ICAgICAgICAgICAgIC8vIGluZmluaXRlIGxvb3AgYW5kIGJlY2F1c2UgdGhlcmUgaXMgbm8gbW9y
ZSB3b3JrIHRvIGJlIGRvbmUuCiAgICAgICAgICAgICAgICAgICAgIC8vIEZJWE1FKDxyZGFyOi8v
cHJvYmxlbS81OTgzOTc0Pik6IFRoZSBlbmRpbmdTZWxlY3Rpb24oKSBtYXkgYmUgaW5jb3JyZWN0
IGhlcmUuIENvbXB1dGUKICAgICAgICAgICAgICAgICAgICAgLy8gdGhlIG5ldyBsb2NhdGlvbiBv
ZiBlbmRPZlNlbGVjdGlvbiBhbmQgdXNlIGl0IGFzIHRoZSBlbmQgb2YgdGhlIG5ldyBzZWxlY3Rp
b24uCi0gICAgICAgICAgICAgICAgICAgIGlmICghc3RhcnRPZkxhc3RQYXJhZ3JhcGguZGVlcEVx
dWl2YWxlbnQoKS5hbmNob3JOb2RlKCktPmlzQ29ubmVjdGVkKCkpCisgICAgICAgICAgICAgICAg
ICAgIGlmIChzdGFydE9mTGFzdFBhcmFncmFwaC5pc09ycGhhbigpKQogICAgICAgICAgICAgICAg
ICAgICAgICAgcmV0dXJuOwogICAgICAgICAgICAgICAgICAgICBzZXRFbmRpbmdTZWxlY3Rpb24o
c3RhcnRPZkN1cnJlbnRQYXJhZ3JhcGgpOwogCi0tIAoyLjI1LjEKCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>