From b60c0efa9d0b30bc09b2d648ab6569d22ae23d9e Mon Sep 17 00:00:00 2001
From: justing <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Sat, 25 Aug 2007 01:20:33 +0000
Subject: [PATCH] WebCore:

        Reviewed by Darin.

        <rdar://problem/5432254> GoogleDocs: A hang occurs when applying list style to selected table

        * editing/DeleteSelectionCommand.cpp:
        (WebCore::DeleteSelectionCommand::handleGeneralDelete): If the position
        that marked the start of the range to delete has been removed from the
        document, and it was inside the node that holds the position that marks
        the end of the range to delete, don't remove any children of that node,
        because we don't know how many to remove.  For example, if the end is
        [a, 5] and the start was in some descendant of a and was removed, don't
        remove any of the children of a.  We will now refuse to remove some content
        incorrectly, but that's less dangerous than removing content incorrectly.
        Long term we need to update these positions as we remove content from the
        document, but that seems like a more risky change.  Added a testcase.
        * editing/InsertListCommand.cpp:
        (WebCore::InsertListCommand::modifyRange): If the end of the selection to
        modify is just after a table, and if the start of the selection is inside
        that table, the last paragraph that we'll want modify is the last one inside
        the table, not the paragraph that contains the table itself. Adjust
        startOfLastParagraph here to avoid infinite recursion.

LayoutTests:

        Reviewed by Darin.

        <rdar://problem/5432254> GoogleDocs: A hang occurs when applying list style to selected table

        * editing/execCommand/5432254-1.html: Added.
        * editing/execCommand/5432254-2.html: Added.
        * platform/mac/editing/execCommand: Added.
        * platform/mac/editing/execCommand/5432254-1-expected.checksum: Added.
        * platform/mac/editing/execCommand/5432254-1-expected.png: Added.
        * platform/mac/editing/execCommand/5432254-1-expected.txt: Added.
        * platform/mac/editing/execCommand/5432254-2-expected.checksum: Added.
        * platform/mac/editing/execCommand/5432254-2-expected.png: Added.
        * platform/mac/editing/execCommand/5432254-2-expected.txt: Added.



git-svn-id: https://svn.webkit.org/repository/webkit/trunk@25229 268f45cc-cd09-0410-ab3c-d52691b4dbfc
---
 LayoutTests/ChangeLog                         |  16 +++++++++++
 .../execCommand/5432254-1-expected.checksum   |   1 +
 .../execCommand/5432254-1-expected.png        | Bin 0 -> 12407 bytes
 .../execCommand/5432254-1-expected.txt        |  22 +++++++++++++++
 .../execCommand/5432254-2-expected.checksum   |   1 +
 .../execCommand/5432254-2-expected.png        | Bin 0 -> 9789 bytes
 .../execCommand/5432254-2-expected.txt        |  25 ++++++++++++++++++
 WebCore/ChangeLog                             |  24 +++++++++++++++++
 WebCore/editing/DeleteSelectionCommand.cpp    |  11 ++++++--
 WebCore/editing/InsertListCommand.cpp         |   8 ++++++
 10 files changed, 106 insertions(+), 2 deletions(-)
 create mode 100644 LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.checksum
 create mode 100644 LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.png
 create mode 100644 LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.txt
 create mode 100644 LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.checksum
 create mode 100644 LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.png
 create mode 100644 LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.txt

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index af1b4f4f0ed0..65fb80e0f839 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,19 @@
+2007-08-24  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by Darin.
+        
+        <rdar://problem/5432254> GoogleDocs: A hang occurs when applying list style to selected table
+
+        * editing/execCommand/5432254-1.html: Added.
+        * editing/execCommand/5432254-2.html: Added.
+        * platform/mac/editing/execCommand: Added.
+        * platform/mac/editing/execCommand/5432254-1-expected.checksum: Added.
+        * platform/mac/editing/execCommand/5432254-1-expected.png: Added.
+        * platform/mac/editing/execCommand/5432254-1-expected.txt: Added.
+        * platform/mac/editing/execCommand/5432254-2-expected.checksum: Added.
+        * platform/mac/editing/execCommand/5432254-2-expected.png: Added.
+        * platform/mac/editing/execCommand/5432254-2-expected.txt: Added.
+
 2007-08-24  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Geoff.
diff --git a/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.checksum b/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.checksum
new file mode 100644
index 000000000000..ce3a6d234fc5
--- /dev/null
+++ b/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.checksum
@@ -0,0 +1 @@
+d4c893a03c062eb91ff0a04f0628f65f
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.png b/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.png
new file mode 100644
index 0000000000000000000000000000000000000000..87079ceb05c829514e68e46ff7b29cd2ef003b77
GIT binary patch
literal 12407
zcmeHtd03NY)_xqPbw;a}x`3<}tB42`WnZ0&P*GsmK?tctM7AIV2m%4e)`~2lwk*n8
z(GUUzWDR?&0#OhsTUbJZ5?K;RSR@ek@4WPyncp(={jT42&G%pV$N1(=-sd^bIrq8m
z`{e##Z7lchmfej)q4r+<+T0$6+IbX(`h)AAK87n9;dgGLP)AW0&Hv&UktO6rW%YY!
z@0!zm`L^Kp&o94=I8Sazm1q1g@TUKb_Tx7%zEo@JZ|O_Vv`7!6Cy4U93u|i8&jPB_
zUbJ7ICg1$a+*hCb^=LFaJy_LorsR(PFQ2@5QxPi}Yq<XXBm9x;isqxUjESt#S!2fJ
z@_NqX%7!Vvb0ynUm3$CZa>P+MgRC(~@ehAOp?*k0{Q-r#cI-A>8T!*v6zZJrzqxE8
zdrnqXcC1AjGZV5hT;<Oj^2rsxw!U8B*I)QhFUYfDeWBf`{#18Q&u5G^&TQKiI+dy#
zvoVb!_!~768<5X-#ctwDQ!)2x?)27PTQdcJRxv;Ph*1wKTU95A`}HY)Z;*DX;$)6&
zI=PMi8e3km8bUS5r!Z%#1D7V6(hBiZ9xL+$t_1IJoO=@AKHZ2B7KXRVcc<FpL&(Ip
zwy~?d;a%eQRRKQZE_v5bN6ccPXBwBe!Nb0k7FewUacwE-{62#(Ojo;;TkA(RL{n^f
z(E606S$yvLdljwFsb3}Y$+08eZkUxem$5fzyG9=Gp8Z5`G#dXVLq|<9M)FR^Br;T6
zM@Lv$XFT70Ah=n|vUzk<IGW}`=+3*Kg&V2SpDwPbtD<p@J<nc$6E{3OT=SvRD2luu
zvN5`~fw}zf%g;^4RNI#Il@Gk{<#2I?#%Z(`e&S7h+faA4s32>8(ZcRxZp;&YXRwGG
z>{gtWV|>D>NHaYH8yWe=VsZ1ILFmLg1of#ZkEJu#z550REOSkXUXr<Y^S|uee=v>a
z*=T6GwbF%L9qY~AT->)&QD|Sf98*;^d<1W<)iUwkYVDVD@Ak20FZcN`d)Rumhdal#
z*#DW56X_SZxiYyYnJ`LR4ux0JikR39v2SbQ5mj=E1xfx<)26tT;Ks_(@$Wg?W$fVh
z6Wcc+;PQuwjv`5ziJisy^XCPlj95Qk-;1i`#Ujjx-#tP10gd{Z^3a9XD(O|CumN=X
z_xlvCu<8`p1@+7#kMsBTS@%+GXR8)2xz&aG%6|FHxjKG+=CH}uSb}E$xBE@?(ZMZ;
z4yC$WbSLCv*V1KegT6>z<m+WvCF5<A+U>_@UTN_=nmk^6c>0lsy0zGkxQcYLz(Y_Y
zo@mIN9_e!rvr;9es=4I9dwGxY`T6gkR{0N9yXz8H7Z+PJW6IpcgLV)aapXc|?y<U_
zU>oj(TcIuShxl*5-LGLR;`<Mg#tD|M3Fk^07TWaR|8Ty}!_>Pi1xW>S^Tp`_&DBMI
z?|5BRRn>eoy^&V&Y2UbiHI5^T<P@*lQc2yZU!(oBvqf2;rTJ;M@-M0bhCXw0D%I%^
zW-7SYnOOPaa2in@7K;@ZS5Q)-(ML()gtKW=T0{92x&lFjIK_U0>_H}7^+7&3nG=PB
z<7RrNqglA29lPW|#cZt(U||Wu&7ICxwSk;Rox=>uFoh<nbP5vK<Z4v!PfeuQPZKCK
zt3XD@88wqmjzYQuo#SM{F*p$@zK+q*O((FH$kp}7u;G~`;jkR9npPt45Cu;st&W@O
ziJh~RmAMOP^yaRZrHa94SI^huUn*@$m=RNt?~M)nm(mIpIaNjCqQJ;Cc?Uld(WFyq
z+|Nj5cFNcgDypEgRP&!f<SuS5GZ<3~n;UD)d8QI3Vq|z&cDves{&rvd%Y8<tJtN-T
zjf<1CV>Uk<U7yh74Pukki=)<rbS42{MKYns81lxBMDUV=yG)jw0Z~5a&o+GU;<YA~
zg^ah=EVQZdnh)3}mSLH#*klvu$~T>DCw<svlIQ!!{4r}+wDt6u<aAG22Z~Kn+<@sl
z!jN}*i;vmbkp71^XL$?ln#moRdY0zR4z?zT9P%&Uk6mZRCYHu-OxwA*NGF?ZO2WO(
zByYdf3LLJidH22Cn1q*{Oum#JKGd428yJ5{qxOr<mE0`@jV8%h1mrtgU6ZA#5LSg_
z>PCucC##J-Tb80W*H^TBd!9Zt4D%ZaX;;_sYK}vqB){dgrW+>clQ9gvk*Hv#o2Kr5
zSgKTkxv@NIhJ%WTi+iZ!@7%ysn2k%&Y^ve-YeG>;D6y*)eQ}=yw&&?ZWL4!hj*OL#
zpt0D^$=JH+di;y;{wyUm|59o5nfu!EPr2(~SR^Yd_+$lJyu(*qN;`cYf}}gqmCN9)
zW{bPyN|+Aj-pt0LP0!*)2n5q?{iRuRrEl*TR%y-EgTM*9>uoeFZ6)ScPjqI>O=jV3
zqF356B+Z!l1g(I<XZlk=f9oY0sl6LkCu@sNVtVmPT^WUy4Iu(gfj+%ISW%rqGcsDr
z?qHhebS+-Th>B<I@xCtj%RZ4_ms5uays(t4TcuB|TI~%2?JvsBB;p;Y{<lSymX?-^
zx2{FM+k=S%Fr&=pZmr}>NdYRjRA%m^^ZC5mqNO5H7WNchY25Q{qvu3i?#7G{a%4)I
zbN8?ttK#5k8r=eoMmNk7XA{QT)8UBs>WzG=l6CVdZCzcQo(jQ7Q8<NUPMiqco>md<
zzQS?lyPr@nyVK9YZ1iKuFZY`ncVrvsHA==3s7+0#Xh>9mL}%KSROGa^uf6&q#oo%D
zj<bebRBKp@S1UQ$pwrj=e3w#IN$o@iyxGyw!C&mPwUyHi_*z|E-M1}8l?oLP?Xcv=
zqrmopYiz<uuw`Lkq2KaI6!$~U;uEEfljM{h$b=p{ONG?dWTjktE8ll7?)uI)F56hj
z2b<STm(=&9s5(@yJv{H=;4mD!wXr9NMilyEE-9_NvNYtxSgB?Qw;u6j6}!P|uUxs(
z6SF2v=Poxbrv@aa*srqaO{q&9!e}`wkZqxm>_Xk}Ec3b$xyByN5lm0_Li@~w5Sl$D
zK;rT95;O!MjD7lVZ)-;`PK81RKE3r}f{b9aLgdtA-)R;N9<nqVyA>X!mlG+V`i?c<
zzsuC)ggUIw_?TtF;$Q!GTvfRTQmO|z8V!1F$1-DU<De9u<sKvq$!=U4R!dSeshipG
zXS3MD(<u%%s>V1#$jB^qb*xnj(}9W|a(3BhbUZ@I4!?4x=H^uZca_}eGZ)NPJAz^y
z2I&g?JuR!DmYNN7Uq!aNbAQ}t@cxIzsVB+#_mpDv6`+app+{a;J)~rtoy3INJ**Jh
zvg$-0Pc4k1@<!~K!ES}Ldc&pcq*;YWK86BX(?}VTrNBgfTq&b@P5xldtQZ`;NL_3g
z2+pBiJDuwHlT~s{r@mh56?)Iolob|>jbs{e9|#139`OJsw*>ZNGr;cV4Y5jwOY86?
z>eEuE$d1LxhsW%#_^*y7UzYZ``ImR|m@P4e^7<zgX+?<N1|RsssDo8D2JdtxIsCd<
z(0`Yq)|jU5bBn!D|AZOz(kNzK5-}S!Dz{7i=DDoUo{M+4Hd+1sEf2M@yMwxD7=~#K
z9AYa{nsIS8Q+WyTO<{ClAjL>j1J9j@fV9pGRbT<o%Ob^si1!aHt<q{~iB@zD3%UOM
z&PNDH#YG6!%l2s092M2iHn~Ey<?(N{>@XOmW0s`E<mmUu<{M{;LG{T7NF(etoG#X1
zpL=<1tcuR5vl>Y@S#n(;lV+T3IN;>ubVy0^%dXw+300Ps<Hhx~Btykg`!y425i(j{
z=7~qno`CdLdU<bu@=vy=1~FS}qtbz0wnTx0*3}o^>GPldG$uDHu`u3R|8RF}-)tFE
zQOL!O;JZxNCYwOo<e~inA`D)=cLA#0-24+I(}pSCW1jT~%*~rUqlEqAhEc*Qmtr7B
z%6aZ<1b~iaNLyFQ{+2r*?HPlr)Vxwk3MviP$vI^pLW`HZyx(N%X&%<}l($qOLZe3z
zxoNp^wq@?!OG1riilA3PA$sn4@Y}!bYTX6xC1SaLRAKg3SCki2+B`6qR-jk$BuZy1
zufXm}Jc;48QP{I!)!*0`EGppDobG3#71l`Oi|>sqtb-QEN#5ya(+_82hXwbwocu7C
zlCtA^K6tvPlF0VkNLY+_C=07+D`r!v)cZ%yUT+rk7xB{EajNSBHMAVloO(Zopc685
z)9$rSudY!P3A%;xtq|k6Uv}P2F3S#SQw;{lIBdFGmmnH>s2}WQ9$-c%j3fuPnYci-
zj+}Yg?oD33Y(Ok@T^XO6NA*;fEpek}Z@?ctfV}{$soBjZ`_O?Cm*XyJHfrxPmApmB
z^Xm|44G$TJeHj~9--4?b9%d@VM!$J<R;87s;9mYlgsZ2)6=SG{JPae|?j*@jP!JdA
z+7J<J-k|#{9bFk(r8x>9<OLXww66%syL-HT7mP5Jb5AkyBtQ^$?(OrtlOKN~^H=An
zP{+=?z+?=Y=hPWPN1G{oz5V5H-GhUJ-bSzoS?z+kWi6)s0V%sBb&Fm5<XvU{Bqg;b
zst<a1r#6M&B&JD8m3g(q+oV%cyvEy7g)gw=H9aJ`AV|E4%piyv+^uQ4@F4jh&4u|o
zzi8g^b?|H<Lt2gZz*6x(k7QKBS)yEi^cif}Xas44CBtN)46U>@8F11FTB3W1ahQBE
znWfoZdP=?2mBTKnux8$>sHmRu%fc0HRaa~14pb4*ww-6Qg$`xJkq71Nk1n-mV}EOZ
zA+0vY>&~Vr*WrG$8ve#7J2Rff<}6LM-B-zxf%1!z4<Ed0hekJY7XFVDcVG{stPOJ?
z9#dBCG?>hCZHx?ihkbV`T|+(ki{i2}tt;;CPhz(=g#wQN?u?|SuQu<YJu#oM=IB$c
zWOroay^7iWjU~3`rWmuOgTQtKWOe+erJ=G{M}`hOQGt>b(zY*kC_B)yVae^ejz7yB
zc>X6cUmjIf24!%dzrX)(*HkJNUYC9I{Ep<^{#Y4=av5&qeBJE7>?WECk7XNPr7qnl
z`Z3H%K;6)|4q8R)=G$Ln^lyISHjRN+9t>iG`Q!0RdHMM#mY}f9&3>v8w*DE-afs?K
zbSev*X36OIIo?X0H6Lvv@<7ytZnMCCCzqCqj!d@olu=#}cE~MO)48=RRjqzKQAk?4
zYfr560O+}!a3_S>tJ}hwwP6&7t_4nW*=>)eBSUhieWWz>MYeS2_6Zo?TU<0AK7geL
zW;*tKY&1RM7{JNNaBu6En`}W>VVojK)X)YeLuUALm5G66AB)v0ndf0A-aOKZn0qUu
zhx2fBa%w4ds~Agl@f@O>U9v0edS5fa_GNt61bOFL<>z!c*VN>CSB?V0yxc0Tht+OY
zlVV?eey=&p06)AjX+S=>-(>Nxug}m^?fBK~LP)H%w6ut^`zpJ;XjiIEWn^Yz-@kg`
zojMPI#XO$w@!0^HfUXZ6dW0OGbE&67m+^<o6a-@lETP{W{{0P&aiF{LE*PQ?ccqhX
zIETH&^7b*E``iBJ3)1L=6+T@gXrAT0ngoLP;#7C=DxbnsAdGbLb#aW1rKy2Zj7QAM
zc&gBRm$YZZ)ne!>O^amBmOrax{WWAG#5N^Wez{~ds_e}7`;`>COxFU8mq+lzG<DC0
zh<j^aQBBb#pPXAKSS4sVguq?kspc(8iYhTqbsN=Tvw`ECVI~G$0ljj>TI^72HPHuu
znIP!q;J>cLQU)6aoNW`y)&E4ls63NX-Mvh_i_nE*z;x^}JEbrwWfV&qoLK+Rm25QI
zaJFe{W0rYW+EeBy6<ZP%%yvbx%q~Z%8K(=*8kDS@+I^Hpqm_BLr)k9gCAc@=y3DsX
z{{;`2E`6<aJ>NRh3v}!Q`*;wDtwQ>!qLdUOh@+>@E1h~;O!|1gqKBlVWuC`mLWW_1
zy_MlOb;z-L5XJr5;t!#55s=93+@<gL%H;u0Ji{DNu}R#yPyZ<7iG%CpYn9x@)zwvf
zVP&uKSe1KY6=#-&Fnw4Ta;TP<E?qK)77C~)qe>Q<ZY`H36HgZw74f1**F4|Vf@wi0
zyWWv;!`JsdG(UR#5_IVB3ZR`yaVKkWY~3q#9J(IC$DpBdGUtzU*dW}?FIP~`<17GK
zYkB08<0dJd0Gi+ho&^$%2tGM|tadUd0k(aZ#Sj!0jR}KdqmYB2ht_Jm1R5;dTTe%)
zMQ6aHcf2&r2C^8s+=0=}B|6(=i(<B+iUTzyc_H2OtLR?1l@P<O=3amG!{-Qts9BlF
z+(TdqtbBx{{Sl|aPO}Am9tYKUh@zNjS`%sAT6phT9pHA_z4|obSup>$OR?r9@v&pB
z$cl|Mi``t;fA`&iG2km2204Z#ru}rfz$G=!rC7us&L3<TvSTBJEurInVEa(_t^6lk
zLAyeG43vyjHL>hB&Sozh?r%yv8*bGR6*xV@DeH6kdwtKy{+zXRGR$*o(8j$wz;|FX
zP~1Ep;Q#j&8#q;aJwAf?Mv9#kv&y$~a5#WiNPqeE;}$qZd4y1W0$jfSW_K{I^qQhN
zxyOGfWVLUf+A>=J;A;&6@}ks(kN3)^lrT0|7u(cI52lTR|3NJ*Je4G?a|+5t-vqRq
zFVr-(uFjAHC+Z7_!iRucOFhW9&CLM<l4X%7^WgfMhYvQV8JlY8kk`lW*6$02DkQ3k
z`MoOWo6kS;g?7dVLkm|(rCtPI<6>A?7#|p$MqgRc8Ft$I+{^vxRYIm>)amS-Fa8Yt
zyN~M73lB>N0!pRiSUs^wvf5hE-1-s7BIfDEpH7e=EyyYM83y?4h?^s;8{k?85M#v2
zO_I~gKSt75-V5Mwonvgk0j}x`Lf~?)q*=w@!J#*Nx+}}z>|S~OFO%dA&SVpphI~PT
z@@sgcOJ{vLFRPL@*jU)3@62EsEQHYXA-fO$#=#AmLcFYcUVmSoV<<xjh`KhwYUOCm
z8u{)X>3k&F__QFRKe9*Q##<a;qiAsv#u~cL#ZgJCUQgA4XXd78c2gs8@Y&;M>ma-?
zLjm%CJ-^>NcWq>{2dv91?izRbu;}{yqps)^7tG`1`shKU+Q30WUms6DI6TLv<k}$d
zqjSRDDQsI+vKf#|{!S&yIYSV;J&X-DgUr}kW7uMuB<rntrLAsUqy5@|n>(E-WWv1~
z)4#f{je@AZz;r0%aqY^2?TQl-B!Uh}j{6$et?iz}r%#?{cQ^gD(3YSLj$tU6h)*?R
zSK9f7MI}}U-dq6zW<`}m)CsHE)pE(?R!mt;|Byt($+N63Jwwk(pr72CZA8D}P6(SZ
zN~;JQtvlQGeg?_<cXy@Zx3j(nluF5PRTj@JVCJb~G<UkV6QPnHaL1WmK3SRZ=$$DJ
zl#tFns^rRu`LWRPRG0j?ucKzlEHiWiuS5bKRJ#XRl6))?S}8KfA8B&t>-0`vCEf5B
z^ih3R7RK)~F`EnNY!hxZjRfZFhZ`<+cy;&A1qG-1z=)`}YWH?fwHi9w+9@6kp0Ft+
zBco?wyq*2gJ$rAQ+LN9@LVi`1jtO`!?65qmfZ<B&+%lkGd<lDG#G_yf8amyb>lPJC
z9;_qdidYBMINiweG^RO_95*PWz|anzfu~4s*_hqh3|j3kjY$<Twl+h#w<gI3_=$ro
zH-Kfgup54UR!AkPxs=RIc!mZ7LE!_wob=~d1^K&l25a_@CZu)`4_~Qq6$K2D77@Zg
z3+A<w)k+)A!0JP9T~<3{+rrYMV4Und4Z>#CQ@4B8q6J1&IAhOj8kFzNj+~h?c-l9p
zsyiE&|6^IDHT4#K1Y1_oY2Ct7ccSDd-ZKd^CiKTt9h-b;wQ6}BzuuX1Y}hX#;Ph(E
zLxslvKvf{#K^`9hdsWB-L$=4xAHg-POhCODRo6RlckVa-^q58B$&qnCpGa}P%M+kS
zohHj8$ou6R83MWPdak%6qAQAfU;q<KEX7guR-H=fcK%Z0Md5z5r*0asB&l7KVSK_S
zO9rww39)sM_O&jVFHM-6&z_A|LEPCvsAEJ>NSTJ8^Y{0EcecHeMp<*}M84^T(b1W5
z&s*cEh)M(|b11myqDrI29j2}(V?AU942Ai}`-vx)*g=eJbT)c8kj`<ID03g*Q<Oz4
zqEcL`%9@i1`KEKI9Vk@!kS}8=OdT3lQVyL1ZouYu^`sZ|4CrZVH|K7xPvQcL{ysXG
zu7Zig>?yU|(c@~Ep~tz2rSHZ6i?Oc8?aaH7;5AYc9L#d_aazBELjAm)dgxq9)2cwS
z(iO`UCWS^5!54XxYsTmeL#TVyY$JhQjbH4xWUfr+j?xBTA&sTQD0_Ez_eIKt+N}*1
z&7B8E*)`+UsVC@?(NYWpH(DJEh=6Z*K%pYCPaQc2smN&71D{PFdgC!=uYc|6B?D!8
zTs$!PuguEqCLGWYt^eKbb9Sx%7ak_ddqWO}C*|!L35Ww5Jn_h<bvzxTn5l;-mj6Ga
zE6wi|gy_2;cRYo5gIKLEK3V~J>NP*s61=ss%>4E7c`Uc#7AR?9)j*nfwTmdiBgP)0
z7)Qd{f!&PQEgOs;Of2+a{((iLE_kL)putaSYYo6%@>+QRIvC;P$b<tu8|e}~_pMT*
zczS?vrSphSmJ}kUJz$^^Jef^sg|i!*spx`TRCF4u?K7PEdC(`9=mmmrTMS&ir*L?e
zV#GJ^xLabecwR5|gI*a3wDf=M0lYIreXwIRu1)>=81YeM&R)NqsAOh}&{5#EA?si_
zYfg*VDXS0rC@^_>_~q5F5U*rDDJc3_PVDB&Ln8ui1dMnfeXulbgqy$oes4m5kqh%!
zvT-^32*Qv*+H<%Wx+3tk<Zz_j2E%Nm<;C4SaPqKSWaMHFC1Y+5B57M@Tj23{bMrS~
z7DCav7Cu6uZl6gJEBs(eT&>*miGtF8dHrWomMMlM3}LMo&;v)w+Cwbqeq0>vUia|7
z7u5CWBVa*C^!4{&V8m?f?otTrvja^F2p$)w50&PHxJ0v0PowL!b#Wsy`%NX^fVpV_
zYfGQ)fdOACl{<9kBoiDAEC^>rez{g$w{~=KVX9KH>=(ii3;SPcyV;!ohuC?{-oF!t
z5(+CZUzNVManfvDv~O4y6wP-Mn$9Br+rzUyHuVG#@SUI=2QJqE14bCEN=`)H+(@BQ
zQ66Lu442#@ME@O^mxKGgmcu_qp$2W-c3eaFJ22W8%la@Gv6pl<SC50M>dlCO1e!-k
zsj@QE@&Mm|I885z+pz8M0~1OH;-lwW;&BA=_d!;+!1KohMWq@e6NL6Obwp``c>)P9
zfAg0eT0WgwU%A<uu3ST=F20m(=Liwd^kisgi2nhm!$}*{mG^`GrHx)-rLjLE=jZ3w
zOl0`)7Vko#+6I<k=rBAGPn&~50Tbp#JOG{K=ilyPwkFB(TIGTg52-mf!z4t>f1o7E
zzohm=nkGgODUw7Pt-Pt}WaEWb+w>e679v~(LEgY+g@nCwy3NV4>pWy0bK3RU(VSZo
zk7{6QhNx#STfCqif;YqiOyERvbZ(@-M`-tj;66Lc7GAQMA_Oi021yz)hLZ)eI>Eul
z;5gz^4mRAv!;DJ}nMg`R1cqGDt#<d_?4rWLco?6xB6E|qnF?Fjh6)5$8cNs`%GXpV
zSb~3AEzMTD5%-S=F&=$Bu?uxuFWr>N6H?EB(zQk`rYS=;=i)f<lFY-zOBko2a_>tG
z;HGOuOXh6S>8bYdh%xWv)E~aXHxt5GK&A2*g58T;N=bEfb$RSRqEOZJUSvEyH&M~s
z^xZzBh5WZO>wh!K{=Y6e5~A~Jk6&=W`)h0KS~}q4Y3ekiC!Q$OFB%1y(ZT|%=)lhK
zQc{MXut3D7a2CRMJcZlOJK&202W9=`sx8iA@s*YN#fM<jAC4v#dG1lF+PQi`csoYx
z=lnYKpM0gyWn{Qx17ETtV*2xcTv#(jp*}r>{1(7<-pU~$!0)x^RTS!*0Tv@aXvV1E
zgz{2i46!J$E@<>cIXvs}d=<gWxuNHvlvLHu@Py}4xMfv!02LeBXrFwNySrD5nzh!A
zF63cBi=}+!+uAym8#e+-N!IFiM&Hl}2{)~0^lDB-k$0eOqkAt+XL_})=DV_=k>?s0
zs)s-N0d{+LHu@88h2fH%$APVRPV8$Jwyp?f3$EqYUfT@mqENpQliaU(1-?wX;mg^~
zclIVW36R-tsblfOdQqL)nl5}bcbsqzRuyn+_4^f&myt!FpXa-iV?rqnZ0NG@?dH`>
zB0oeOaY2VUm8R}Rp=P2@?|k%+UmwieL?NUNv0Xhudgc`DhlBpO`n(xF=0k^`CD`i4
z=z3j*`$%A%^{I|dObsBxlm~t1E2Nk}T_r+Qn1kKZ06u^;g!pz|-G)^0@k?pVz%BC%
z1y#x<aK9Cc;AaqXt$j4Yxfp0p&1eGW5KLx*f%Ud3M$JeemWnn|5ov~`1yb-nQYlDp
zdI;2b27G5JspIJYc0mE0-A55q1i=8%_S<TBPggDsTY`ayx2l+}ydJCz^KXD3Ly(|K
zj6~oOmZxoh(j;7tJ5=Gr1nb!g=5|t2-M}(x-rm_W27)*N)J~cQlHBuRSJbWbdFB&n
zbgSZNmk3e86H<I!TuXv<`af<Ms`PaL&*=pGz+!h-64>Zq7GR;Oe_Ld<$>DQgqCO#$
zkLQ4o^a5YxOpBkGn2<3Dx%PS-5?l@g@%zoI(9sPXh7|j^eub3=KpIz47babNA>nWf
zu)M=kQkV^a%eEUQ4+Eg>nXHQG{Qd1d4t~-+_;lJL*c42*Nr(yD4tb*2ghQ_=5f{7X
zVfYcSylynQ(&-A5|I?TzV;Fvo0d)Ib*uJq^)F_@??0t|EW4M|qdq~UAv7Sk^NQ$Qi
za%A~@%CU@$Hg_aYs-bGPD?N!}i*B`NdVVE7=_<|@&0$ffw;hk&F<;HX$l`%##OJH>
z_{+)0P@-J<I`O`*`wzL8TT$m~x_nFz5KTWk!kKNq6*+@sG0IA6JD2|mDRxwD2SCp&
zi*4@uAGyqTMqN}Dhnhf&>itb%g}DigNo?`kf8+n~3S3Yq9ypxL+q1V}Ekh~deXo#;
zs^7kNF1K$&Zamh5Ez|CTOZ0ILNF3f#-_rE0zZc`Vj!{Jn0$cCNV(T(5q5}ZbXF;R%
z*oLqhTlxjz%P$tfXK(j8F5ll#?_#-?(bQd2b=<i)D85x)eMk){$qZJIAE}k5-VIRn
z$zY?QU*TG%J6GfO_Um1X3B+Cf&G9atm(&6FuF<|kp`Q6qV|w+L6dVr#<h9)9uw$p6
zyfixMWM}j<#(d_;w@Wk9dy_i4djxuBu-%14n!q6f`y~7vqkxKw9W4qU-Elwe2JT{a
z--s-7UtgA&|0RFcZm5&`5&*&77-0dmBwo-YG`mxUMvh=gJ!aQ`_&ok2hx+@M1pLob
z%kP2xJ+S|I4E{Z<e$T4kv+6%eYkn`O-%INElKP)2sm!fig-YxPm61HqBq-Fy3pVB@
JUww1uzW_%M^)Ubd

literal 0
HcmV?d00001

diff --git a/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.txt b/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.txt
new file mode 100644
index 000000000000..71a70ec37f60
--- /dev/null
+++ b/LayoutTests/platform/mac/editing/execCommand/5432254-1-expected.txt
@@ -0,0 +1,22 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x600
+  RenderBlock {HTML} at (0,0) size 800x600
+    RenderBody {BODY} at (8,8) size 784x584
+      RenderBlock {P} at (0,0) size 784x36
+        RenderText {#text} at (0,0) size 778x36
+          text run at (0,0) width 182: "This tests for a deletion bug. "
+          text run at (182,0) width 390: "Only the second paragraph inside the table should be deleted. "
+          text run at (572,0) width 206: "You should see a table with 'foo'"
+          text run at (0,18) width 71: "in it below."
+      RenderBlock {DIV} at (0,52) size 784x46
+        RenderTable {TABLE} at (0,0) size 31x46 [border: (1px outset #808080)]
+          RenderTableSection {TBODY} at (1,1) size 29x44
+            RenderTableRow {TR} at (0,2) size 29x40
+              RenderTableCell {TD} at (2,2) size 25x40 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
+                RenderBlock {DIV} at (2,2) size 21x18
+                  RenderText {#text} at (0,0) size 21x18
+                    text run at (0,0) width 21: "foo"
+                RenderBlock (anonymous) at (2,20) size 21x18
+                  RenderBR {BR} at (0,0) size 0x18
+caret: position 0 of child 1 {BR} of child 0 {TD} of child 0 {TR} of child 0 {TBODY} of child 0 {TABLE} of child 2 {DIV} of child 0 {BODY} of child 0 {HTML} of document
diff --git a/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.checksum b/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.checksum
new file mode 100644
index 000000000000..20527445fdbf
--- /dev/null
+++ b/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.checksum
@@ -0,0 +1 @@
+ab5114af2304436556dc70e0f32ad275
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.png b/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.png
new file mode 100644
index 0000000000000000000000000000000000000000..65c6888835e4aa29b8872291c49d6b90fb78a4b2
GIT binary patch
literal 9789
zcmeHNX<U+7+h@k7oS90;amsQnn^sfD+;XKR(<U=ZGk2Y$Tp%MgR|KY-QA@?CTryKj
zGs}%!5Eao%h0>JVKt-V>#0^MMQ4n|!^Stl-;r+e8-}8KVzU3EhywADMxvz6w|K-A4
zr%$<T`cC^h2n4d}*W*qe5Qxfl2;^&@Z`OmqJczw<6$04~`PJ#j+4zV2aa2~pCG^xh
zd+gvJKmGEJ#vj}NNZb9}Kf2sBPS+P-cyasiJiXXOJFTkfY*hvCa7VSrzQ+27;`zPn
zG&XI`y>a8r_Zywsdpcl${r0KhyUy>w`S$$rgACr<tFFqto_DS9PGa76(0HciC>n_^
zPAKA`SDQnZEwVjMfQ|eV#$wrl-)r;VgWVn0{_1N8<cY?=y4*GZnVy<TrRt+D4^tUc
z{#tDcnUK1$YoIcaWOaqSM30}$3vCcrJ0X!s`_C^`VjPHdb>^7W?qJ+hOvBRZ44AA~
zEmDkxFb#|dla{7o*B(;fn4!^AJ7-IC_?BAms_ec>5ggxSoJpWNGHj8s1lgMwYcG9q
ze50vVb8>7BViMQ7e!qi!g%bmf3$<}c54KdfC>U!pi<udWpY>;JhZe+~tFN!`c$a6+
zNjL9J)jV}*ho7I{L_T{hBuUC-<H*aSM%As7ne)V$kAG@W?AWWlW)8n^43>^EE_{6b
zXU)a8cMr|IXfLj-ize~A9mHdrF1nFUYeT^@uO1#Nh7lOK*ylo>E-P{@W{l?W5U*~m
zGm99lUlGK$k=z=ByM1~Q7o0d2=R_5wvD&_-gv<D*(}=3{*aG!S{sxFDbaf1iB9YvQ
zL2A!5#w|)Olpw-Fn?eLrh)4b9UKJBw_(ZQj+PH782FJe#GA$6`xZSzbiOw`m^K5n6
zx0<^<bXcF0ed@!`90+W-Fd;AY57%CDZ4r}&88R`uZKlCdIzpqUV?N(Zj;jUHze3_S
zB)1_CG|aDwXw+3+kpWg@jn&i&ymlr@h+?NLFE5|rcZcalzcZUpwr<%mIywq>6gP5k
zY0<qlWr7HAT;Fk;x5H#lzLsa%sqx*+YU=-a!=u|8cT*Bg22pP1gmc<~yrp5fsP&(x
zhn5ZJFq*F?+X@ncQ3nJ!cb=hXk8050`XFI0!7ACy!?4UFh~iN*624~Vvu)dXMOe(c
zhc?l}R32C^MK9LJ$*G<|IOgd&@Pbxg72&5F^=5~uByr!RVWBcI>Pa6|BB^?DgQG*|
ziM|m!LqT?i7r0&7eM3W=dwNVa4$V^y*zYle{R0D@;%KQvVrW=QAP}%vc;3(Fe;JCd
z9XbuO7+ci_x#&uU7qmZ4=eFoShfBxbv^-l{yOZu-#V~;9XW#~o^SrqTbz;Gt%G~S8
zewr{#t@6No$=XssyuZ*E|1>IqIMl+1H=?V>Eg4*S#t@;pFvdS&Y-vo(CDEWVt|-})
z)-)V6`{cnL+<3gdiVB~=2r@&^D^s5d@6S3k^t~&(Glp8>^!d$xi%jVDFL!hd?|%|M
zJ{!bP2wPiQ8$Z9^!09r=r{}hO+BM&9Apbd}_u)3|O1r_tOa&e%#44mUmq%(-QyZ5@
z5uCix!dn?S)|4eiF!U4c7tDNGaJtTw#vS@epI^mKJx-Ze=r+y7;c$(g?z&ERw@JLN
zUAxwi+qTAy??D8a&sn96QZg}%B@ghjB|O$)i?QF24h|0UeVG`G-KQRI%?*t8RZb{_
z%BI7R$jlV|#JxfCg<R^c+<+f6_nb;srt6J$8!ku|r_CZKI#wpM+O}G=;$ga|xtf~|
zvbrMK0RdUe3WLLM&*Ix0w;b^Kd5&3@ZxwOdJ!wL-M$AG{HQnts+GKnr3@kQ{U3<Hy
zNH^Ag|0eNRmszti6ezZo^+@E!Z$yIR_IdH!k$3mac81Uj^|XSMB0<7*W@6Vk<%-8`
zOT)@azi$R}P6F6)d!#Ou3L>wg%(EoPKoPD@#%#;@bAuLTZB@kibk7}!$1GLI*wW|j
z=>0*G{#;6bsVgkgVL5eYru{)2RK9X(zC$ww-IH3A&&Wtu7!${SV6RZDCqCR%#|Z?2
z#)U2;*y}&OjqKd)-BBc;E^_YpEV|H`urwA;+hmtno<V!^>pkt!AO~p>;^F%87X3kU
zY_cEoXrMO*PTm+ZzO|z1=fMVHi>SAp@yADc*J<sh-DgQ^-gb02<7OfqS?b5D8WnG}
z+`WW*b$s$dsdF`NC<xUkG8;_Fvk3PXiclxIFX;yF<Qbw0(mQ2sQFD}b?Pt6evU~||
z5IdIqz;p-~Q7;O8A`X$KX07MfcAIb?M)z)8r~zp(#Qof^Zb}@Ba!r`4p>|dJ)^(*o
z*FJij*oR<L8yl>?ap@{V34BlND<zRgy6D9|5U2S}Y|G;EvRc!)irS$^0ZH?nrCA<{
ziHTiI)^bB_B!|+^WcmZ>@P6uE6hy}d2gl^A5c}RBg5$<xN}HMz-ZM_wPCr@bL1OB%
zJWhCfd)oj==pJi|cHLKcz`vpLWPvR~H?l+HQeW}0ORScJOG`6DH(T54>gwi5!d5Qb
zA8A~gwU9?kdfFl#7wRu8x$figkyw;(<JI%~O0!rjRxQm@nq}R#+G9yA$?|AjEH%vr
z$w#W@sp*z<iM(bT8I``CLG!o0vwZ79vJK>)Q@DLD2FCa#u2FXruh4ETvgf{8iEixh
zd2X`KmAyn(BEsgw?<!r_RrP(ENhGk(zjqQW=LSoPuwDDgvdn4rbqcm(-av6i{?S1G
zILo7}E8ED`Da+NQ>508$s-(Sj@~X2f9+5G9IMaUNc*3-6upg*rUjYE!TEIS0D<4yG
zWy?%_mIpd$qa|JdUbS5GY;?t<ne3;`oc&(Cct25Sqov)Ty<Sh^bBn8P>D+*4W3g3;
zU#bZGZ)^>ojS^naS1V&3_LeS7^crcUXQ9ZW&#iW6q#r4XuJI^gh^|w|-=J%vOwM^m
z6wTPQI@6BPZMUUKv8D61>t}3`?hZ?Xe{rm`;8d&8Rr4HiNNf;HZq^db>&>txPrI4H
zDCvh*Ki!i;U!7Ohd1&uLIq{R#<>Bw!K&o+`=3lv;ni@@WAkHvj{Mq|k1kJO>SwF;$
zMY<daYB?0NT$|hJ>F%(Z$i|?0BCd=(m*5@V1B!!9_m@U4kA%8(Y(T;=Q%4PM8_1?y
zl+dPxCe6B#p1(_&#FhzGW<bgo6B*3W(TT+Y_sn*6sJ&rO{07WWx!2KFRzVzVoAvlL
zPlW=;9vjmwD@=qdI75^Y4+q+~t#dZ7f*vUZND%q{h-xPwoC`of)F|XqstzdUm2n})
zJH}oyVJbH~Wyh~qz9=WUNE%?j_PIT>+nDI&)X{NEQ<Z$IWwx56CJ_&L2kQ+ybxTW4
z^(N1FnV}_%si_eQbye*>v(6O7?<#7k;51EH^SHQ<Z}&X>eow(Qj{^aXKkEZP^dZ2#
zh7!yMTWlBKAFb#ufpZ3?mf?I<Rb=^Dn>Kh)(MS4aZY(8=eA#Oi_8rVjYd<lNX8Cy3
zZ`-pwvS%^Ix&j|Wpt>O(&%O>XAb#9v{Xa*%lOgiM2MQf3bKugmvn$StCI<oSjwy^X
zaxG8OS^yw9r09YPahFT-pc+l$5DMN*{jckqn3=)neSLjqQ#DnYLQnvalIb#Pcv-;~
z<w<aq?~;>~^MD!HaM|}{zLoLrllQ;Z+<h{YK^DYBih8W6XI%8WqVsp6`PT)lQo(&k
zwB%9nMy7=0Wpu7GHC4;R)?aJvDs`dM6unQGF1Z0wzaH?d{RtpT(~csks7PTS((_<5
z$as{3vD>{cfDwPToY$m6W4{!6(IVB8Fiss=`#djn=NPx+jyiP!bwgLQ5nD5Ygf1SZ
z@(z|NelgJan?ryu@Megu;Ef66Z&r}wy)T=r0O8)Zje(CvbsJmqj&WPcm)880@h(@!
zTl>m^<b0&R7(2W6ydgvt!}h7peT;iPYK9A8E(K`L1R7<`xsVy}x?okj#P_Lwyof{X
z8E=o%FOj1op9!nPi9xdj9UB8egD=8ZJuYddgVwg$iw6XXY|4Ok*rGo3jg3ycGS>cz
zf*54TTKTHoc4j{ju-tHH5qg5Hkg;K_3*7m8MQc9=E!@pLD=RE3!r1!x`>V>vR*yH>
z+-2b6KZfcQVI)9;Po=)Sx5ve)1EekxG?s-}`Km2WvTnnc1F5C1xs=V`0UE&y34;Tm
zLw_M56Dp6lmn{yMN+k5oV1)g8_@}09%1mR$2v#A3b!F`{yVRA{T&MtOZr^Led+5^?
zb*Y|6q*jvzqYCpEzRu=P{VlDnT@bSyTx(bfHC_Z>=EWp!wCuXHwQ8s?gc?P*fY&px
zT%4<@%61mReqy|?Y1JRJMJ{bUw41qLuU^_0<Jogu=msOW!y-+}xAin_tB@*ulGwP$
z^_2dw$wk~%WC)cwF8@LDz!GtRKT4SY(==(v7SkYIBk$^TK+tN^wW_qMlkf7BNv2-T
z_c*b(#)IOUda9UViv(3L>m1-A3Q+3<3IYMaaCUd6y=eR1Z(1%DW{P_4Oif60wGRwx
zh=2|{l+aV*jn`k8hjYyC^3>vLbkaE0+*^AJQ+b6b6P*3R8#z8gX7ShKhlw-KBLz)7
zic){GP{MH&gEJ#<R@9X|RC8s#dF38-^<MJiF(-b);M9tRM4}=_`&Y#E;JAq+<tznc
zYfpoYp7b%j>-aM43FCDlUUi5rJWS~IcUX^m8^*Y_Q`;eKXb`2&%#m93N8hPAMVXKX
zhll#9WIJn{mJ|%wy(Z`B1t0yYpTW=y3=C|STYh%Sx9WWAmM>9|6GW54EeWDWRBI$l
zw=~O#cqK3Z#V2TJL;@P>D3Xm7Eh*Z*$&O{C#qze#HgSTn_=UH7V1PzzFfxe`DK;W$
zxvnVVsG7}_87tK*oR296)l!1>m9ed8G+IWD0bY)hzRi4MG9M|x_5#+_Q35crb*V~Y
z{xy}p$zE`x;#}2jr9K7FSwbXQ=N$0;!N1}1Esw4=j24%sSfiB?2r=)=LX>839>BB<
zForEcv`{D3-q5hQxw)7`vNJ56j8?_<NtR|+3-F0JQbh&geP=Ad(>^BCZtjp%$0I6I
zX)Sqn>=8wDS@h()TO>iV5tu!pJTayIq*wVf6l-ZF&3lzh-a&O-u89ZUMhD1I-Fkj+
zZ*SwuL>fXKGu-!X)(>Wzb)9;Z8A0p>)3*Zm5YUrw#s&{}Xrn6`7OrI2WF1D8$f#HX
zprAqZN@)FTY5!@&h04j-FC*9hIW4KJqkiER3)Igd0#fYgiPfqEy6~ma*(IBg-M2_I
z(auZD)@~~F){o;N7$|lBHVoQcJQf}KYMoZc0=HX-RR*V8-;KUQP#$<2IE^Dr19!xu
zI|;u0`v^T|*ra>M@Ob{n+aW1)Mm!!DoM(*;x|LeXyh6&RVdWz3NN5osn*>A=5K&FK
ziBpd+b>7~53E;=2<++hkU~$H-wr}Es`rVI5^ASLMm?nMPpd7<}bW?3VfQh|~U@Y1O
zC`}2pue2X38-Qj%efo6au{!#hJ)k?TQSj=Ca3s>y&CTr+Ab75nt-vT8)rdXUnLf1_
ze+|T=c?EGfK&RI@Ijwf=aXh0BHUXq!cdn@^IFYcX#eUarH|z1Uy^hkkTKG^oxW@>H
zc2mmKgpux`bJf6x@5tl<KWeJ9*Ln5{QL7(Rp*~ky<|(y+R&&Ilw@_MFCpt!Jn|y!(
zAm!OLH&WN_8(QF{B;}~Jd1@}Ay6I__N3vcfNQesEn87o<47<3w!9limb}8s9+^w<|
zDpdbkEBIKUFTT6WwF}^35YQJzwLnrV%myo(b%7@;C6mc(i)B!~DBVd~Q7`}*Pe*YR
zEgLIiV2!cjIIOqTm9b&P3JtSdcMv%Js1MgwbKEft_qaO^R*xqP<g*49Z*{Q>F*ew+
z*vTn8JbWK0L+`(%OZ^N0NEt<Lc}2B0(5KsEE!9@?qc%VXc5Arko-t|sa3iYkm{zvu
zDVtKDxt~t9G|rS)1X<wWJwOeZ5877|lsQQ|wCQYuLnAlGzq}WSf>>^BgFf>(z5Dd~
z>RZSpjt<OLYyB1J9T)xQw#!5G6D6}rk3Lceg(6>m;^`AhOLl|GZb~*h5IUoG)0y|T
zb;zcYuTA$i3)8a2uQnT<%bJ{=jCyBAwWwEmh!UP5Lg^tCO7m4O1FBG)Og(2e>G!uF
z+yVq4W)pU!&}T$q7B|blo9gN5Sr$P*bGRoGKEfuHR`bZm(};Y3V&4=(y%dkfR|b)r
z?N>hB+$oWJoEWJO2hvP1GgB@ZD=aLO%h&+O()11B_Q{_7rfg0?_yHWLU#Z5=i%CH3
zV;}CPPEDklngG}~0b(B6WeRL<6eq(vr){mf?SM5qDOwRtuMV^ek@~ukIatu5$raI{
zZNRB+Bq1~2rNL?V&?M~<V7*$aESi;+H3)os6W@;qN_pE`O!~5)&urr92$@<bJwfM0
z)vODdccET8ka+aDSifN?d6Q08sG}$^qSig5FzsE;!3RJp7o6X;v8*rJBm9Xd)TP<F
zjsxG{SyX;;ZFQ8<x`h!KH=gF0ytoIt`m64>_eUZ>zfwh(IaCPl!*I9mowD8Ms92>T
z2N)w;IQIy`!|3Y9iQ9*O`<a!Lr2KWhg_Zx77zkar!Fh&&M@J;h!3@%Xu`tKk&O8I4
zAFV7>Gl1xN%U?6Vk0SadbJQo(eX><$DS<&^ZC18wSfsCH6b?C$-)^@(-5(I&<uQzu
zUjy`&Qqjh(q%pm`1BVo@ajN?9zqRljgd<P!);XMeWFaKoox^9nTi$UP0{Pv7$rt_(
zfk3cH|M&(1`O8^(9sciKJ|!${a3JrpMM;WbuI|^Cv!(=mroMC?1d@I<!nmi8HTvVd
zxbTc+`eax{xc1`8Bgpi$abMqv=B#ZqEYKzH+U4n6tk%2D?GVVzm7)H%Uy@Gdk8K&)
zP3-)%Oh2M=aPML8=nG(=eM_uoWkQ2_p>WQPeI5e&Eh&GH^#e*s*ViC9`@2ThH`Fge
zt-$bxmd^s}gHXGjh8IRZeOn`~HkIr%1~0iIVt7!~zZnJVsVk^qR!5jMk$3GTsB3O7
zaM)$ru-G9gAe%)j{4tCYFAs-`mlq2_H<wbj$Lbp8!KQ+m>XHhDxJy<0$L-@}w5&}8
zo?BHz6Y%G&@Bt5qM9!PyV`<916cc3)_LfptT47C(xpoZ6PJ<4XR~O+QK(TVGk(&9-
z0MVH!Eih(mKJKtG_M2qMt3c!#8csYjHyPe-SGQ9-RE))$_a15|9f3f8{%#YtV+hzw
z4n?Y1kz2jV7SV149t=YX!3NV2n15`5K#uSH`%!l2-!J<Rt>o+iA|+a<aD+tyJzwl%
z><l!zb)^}9%$t!j+;~1PK$0m;Q#7ykh2?)xH~Jd#a=CV3{hJ&&8dV`9ia4Y=WI6zf
zLo+c*4Fd?|MFGDadShuy=<Vzt=WSu#b~Uvw(;j0d)ByL|TUBa5J^gcjWv6;ax=PSC
zJXVU*mavKrND|j3%F>F7WqV8%yi8}Qi6#e&UZ@>jOL7TV{nOor`N#4~I~MxmfH)x`
z>GPUvF*^5_hK8kqDg@#jnzHdPce0{mMTQh+6c!5hqS29@IFn4Ql&%+pRrvR=Ts`@q
z*1vLv+Y`xJ7c5XnLgf<ve2%%JE;5O#T3kl5b6mQvIT6h(+AK0voDa-Yfj~;?+GXF+
z@8)lz$c!)=<ADWdl)b_WOK36Wu1=eIIbS}5>x@(^lt%V5dvU3$^oE9b5y|mK@qs3@
zbFxI=9_m++-}{&!Szjfr-BB%m-qZwjG&D>^kvQ`%SnG0rbx=awTmyA6ECMmB7&Qt)
zUv_e`Z-F1X9H6*1MBTZj_ANLqx|&$+O?qBkacOD9d&Y_2;CJ_J5NDbEvD$rB)mU3A
zD7#NVP*Ka%>@ha3xg-+(Gaks44Sz9;5@U52%1c0d^LdkvA9ED8e1}S%b$39WWfqu4
zpIcm7>J$n|WeS8&p|1{_Os-*HuR}PhfkjzrTH7@=axsYTq=kMYb;Mv5IzOMLFfx(R
z`;&<^&tb4Ej|<(CZa`JO_U}y(NAY!xa>mCL583QE<h%Q;U594=Akvd`A&{+PJwe>z
z2JId@{WfV&;C!7yz-A2%#u{%eUOIx+LkZ)43s~XmOw?kNuqQ_%GUHcYrF8PG(R<|L
z+*SkQs;a*JQNN`~ko=3WU5Z)^w<0smwYpVyEEpQLd?#{sN!+Qako{(1ZAmRNk)AsG
z&vWx!f7}w#YmHEzlD?vx{3-Sg`>U@Ykl0gS|Kopi|37tEm4RtLrWN?3n`$fCf391N
z&l0j4$#GzL3OV`fi^`IRLn+rwh++`ymyJA^7s{fJA*&n&9g_$$D4>@H*B^X!5G!B1
z55;0AtpHo?Y@jy)LjLP-WB%pWHUIyBaJ`kxhMdW;aAl8XQ-0#atxW~YOJ!hV^V8t6
z=Depn=(<~2AM@}2Ahut>?l=nJd6HFXxtl=j$Puu7>K>anTw9vt1K9kz`hQ^41Ldk5
zGwPqe-obJ2;oZW1Z6!>zNCYL?0xb8goSix`ejXgq|3dTMI3BoumkZZ!dyiQ}l)uxQ
zHi<vFMbAI6n25h9FaitU2o}~$Z8gff^+YwfM&(nFu1Bw@085MvXekHQl{v*BgJ;x0
z5p%o&(kt+>^+YQdME3`SSBYnS2l2dY<d_T~^}kW@Ka~kzp8HRh`4WdOaj^K31ph`7
z>=LFHLm=x`l>ZB$WQ8w5`!|C2B~br)QS&7cza-)pH2M$gMTNoRfYf25$^B3uQ6aw`
MJLOdU%WpUS3)4Yp;Q#;t

literal 0
HcmV?d00001

diff --git a/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.txt b/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.txt
new file mode 100644
index 000000000000..0641428c92e1
--- /dev/null
+++ b/LayoutTests/platform/mac/editing/execCommand/5432254-2-expected.txt
@@ -0,0 +1,25 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x600
+  RenderBlock {HTML} at (0,0) size 800x600
+    RenderBody {BODY} at (8,8) size 784x584
+      RenderBlock {P} at (0,0) size 784x18
+        RenderText {#text} at (0,0) size 665x18
+          text run at (0,0) width 182: "This tests for a deletion bug. "
+          text run at (182,0) width 483: "The paragraph inside the table should be in a list, and the test shouldn't hang."
+      RenderBlock {DIV} at (0,34) size 784x62
+        RenderBlock {DIV} at (0,0) size 784x44
+          RenderTable {TABLE} at (0,0) size 71x44 [border: (1px outset #808080)]
+            RenderTableSection {TBODY} at (1,1) size 69x42
+              RenderTableRow {TR} at (0,2) size 69x38
+                RenderTableCell {TD} at (2,2) size 65x38 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
+                  RenderBlock {UL} at (2,2) size 61x18
+                    RenderListItem {LI} at (40,0) size 21x18
+                      RenderListMarker at (-17,0) size 7x18: bullet
+                      RenderText {#text} at (0,0) size 21x18
+                        text run at (0,0) width 21: "foo"
+                  RenderBlock (anonymous) at (2,36) size 61x0
+        RenderBlock (anonymous) at (0,44) size 784x18
+          RenderBR {BR} at (0,0) size 0x18
+selection start: position 0 of child 0 {TABLE} of child 0 {DIV} of child 2 {DIV} of child 0 {BODY} of child 0 {HTML} of document
+selection end:   position 1 of child 0 {TABLE} of child 0 {DIV} of child 2 {DIV} of child 0 {BODY} of child 0 {HTML} of document
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index a2328f681f9a..304a36ba8858 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,27 @@
+2007-08-23  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by Darin.
+        
+        <rdar://problem/5432254> GoogleDocs: A hang occurs when applying list style to selected table
+        
+        * editing/DeleteSelectionCommand.cpp:
+        (WebCore::DeleteSelectionCommand::handleGeneralDelete): If the position
+        that marked the start of the range to delete has been removed from the
+        document, and it was inside the node that holds the position that marks
+        the end of the range to delete, don't remove any children of that node,
+        because we don't know how many to remove.  For example, if the end is
+        [a, 5] and the start was in some descendant of a and was removed, don't
+        remove any of the children of a.  We will now refuse to remove some content
+        incorrectly, but that's less dangerous than removing content incorrectly.
+        Long term we need to update these positions as we remove content from the 
+        document, but that seems like a more risky change.  Added a testcase.
+        * editing/InsertListCommand.cpp:
+        (WebCore::InsertListCommand::modifyRange): If the end of the selection to 
+        modify is just after a table, and if the start of the selection is inside 
+        that table, the last paragraph that we'll want modify is the last one inside 
+        the table, not the paragraph that contains the table itself. Adjust 
+        startOfLastParagraph here to avoid infinite recursion.
+
 2007-08-24  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Geoff.
diff --git a/WebCore/editing/DeleteSelectionCommand.cpp b/WebCore/editing/DeleteSelectionCommand.cpp
index a6bf480a17b6..faf8562e3802 100644
--- a/WebCore/editing/DeleteSelectionCommand.cpp
+++ b/WebCore/editing/DeleteSelectionCommand.cpp
@@ -404,8 +404,9 @@ void DeleteSelectionCommand::handleGeneralDelete()
         }
     }
     else {
+        bool startNodeWasDescendantOfEndNode = m_upstreamStart.node()->isDescendantOf(m_downstreamEnd.node());
         // The selection to delete spans more than one node.
-        RefPtr<Node> node = startNode;
+        RefPtr<Node> node(startNode);
         
         if (startOffset > 0) {
             if (startNode->isTextNode()) {
@@ -456,7 +457,13 @@ void DeleteSelectionCommand::handleGeneralDelete()
                         deleteTextFromNode(text, 0, m_downstreamEnd.offset());
                         m_downstreamEnd = Position(text, 0);
                     }
-                } else {
+                // Remove children of m_downstreamEnd.node() that come after m_upstreamStart.
+                // Don't try to remove children if m_upstreamStart was inside m_downstreamEnd.node()
+                // and m_upstreamStart has been removed from the document, because then we don't 
+                // know how many children to remove.
+                // FIXME: Make m_upstreamStart a position we update as we remove content, then we can
+                // always know which children to remove.
+                } else if (!(startNodeWasDescendantOfEndNode && !m_upstreamStart.node()->inDocument())) {
                     int offset = 0;
                     if (m_upstreamStart.node()->isDescendantOf(m_downstreamEnd.node())) {
                         Node *n = m_upstreamStart.node();
diff --git a/WebCore/editing/InsertListCommand.cpp b/WebCore/editing/InsertListCommand.cpp
index 063580bcd019..093b637fa1fd 100644
--- a/WebCore/editing/InsertListCommand.cpp
+++ b/WebCore/editing/InsertListCommand.cpp
@@ -65,6 +65,14 @@ bool InsertListCommand::modifyRange()
     VisiblePosition visibleEnd = endingSelection().visibleEnd();
     VisiblePosition startOfLastParagraph = startOfParagraph(visibleEnd);
     
+    // If the end of the selection to modify is just after a table, and
+    // if the start of the selection is inside that table, the last paragraph
+    // that we'll want modify is the last one inside the table, not the table itself.
+    // Adjust startOfLastParagraph here to avoid infinite recursion.
+    if (Node* table = isFirstPositionAfterTable(visibleEnd))
+        if (visibleStart.deepEquivalent().node()->isDescendantOf(table))
+            startOfLastParagraph = startOfParagraph(visibleEnd.previous(true));
+        
     if (startOfParagraph(visibleStart) == startOfLastParagraph)
         return false;
     
-- 
2.36.0