From 7dc8ae9944b1b7e9686b730d3ddb4516dca7280e Mon Sep 17 00:00:00 2001
From: ap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri, 5 May 2006 20:35:57 +0000
Subject: [PATCH]         Reviewed by Maciej.

        - http://bugzilla.opendarwin.org/show_bug.cgi?id=8626
          Strict mode erroneously triggered by a broken comment

        Test: fast/parser/broken-comments-vs-parsing-mode.html

        * page/Frame.cpp:
        (WebCore::Frame::endIfNotLoading): Call determineParseMode() if needed.



git-svn-id: https://svn.webkit.org/repository/webkit/trunk@14211 268f45cc-cd09-0410-ab3c-d52691b4dbfc
---
 LayoutTests/ChangeLog                           |  12 ++++++++++++
 ...n-comments-vs-parsing-mode-expected.checksum |   2 ++
 ...broken-comments-vs-parsing-mode-expected.png | Bin 0 -> 15576 bytes
 ...broken-comments-vs-parsing-mode-expected.txt |  15 +++++++++++++++
 .../parser/broken-comments-vs-parsing-mode.html |   9 +++++++++
 WebCore/ChangeLog                               |  12 ++++++++++++
 WebCore/page/Frame.cpp                          |  10 ++++++++--
 7 files changed, 58 insertions(+), 2 deletions(-)
 create mode 100644 LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.checksum
 create mode 100644 LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.png
 create mode 100644 LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.txt
 create mode 100644 LayoutTests/fast/parser/broken-comments-vs-parsing-mode.html

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 2b748d1b78f7..b91e25c16be6 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,15 @@
+2006-05-05  Alexey Proskuryakov  <ap@nypop.com>
+
+        Reviewed by Maciej.
+
+        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=8626
+          Strict mode erroneously triggered by a broken comment
+
+        * fast/parser/broken-comments-vs-parsing-mode-expected.checksum: Added.
+        * fast/parser/broken-comments-vs-parsing-mode-expected.png: Added.
+        * fast/parser/broken-comments-vs-parsing-mode-expected.txt: Added.
+        * fast/parser/broken-comments-vs-parsing-mode.html: Added.
+
 2006-05-05  Darin Adler  <darin@apple.com>
 
         - test update for http://bugzilla.opendarwin.org/show_bug.cgi?id=8722
diff --git a/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.checksum b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.checksum
new file mode 100644
index 000000000000..65c7c731a5e6
--- /dev/null
+++ b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.checksum
@@ -0,0 +1,2 @@
+4906e178fb48b465dcc1bc2ea4a3a008
+\ No newline at end of file
diff --git a/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.png b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.png
new file mode 100644
index 0000000000000000000000000000000000000000..4697a23eaaa8c2eff75689a5c835c3d6bce11dbc
GIT binary patch
literal 15576
zcmeIZX*`te`#-MbZjVYRLJKLvUA7P<BxK1lma$Ya42BkC8D_Lgg_0p;UuVXcY$MB<
zyA(y%*%+acZN`vgOlBH>r_cBMdVQb%pM0MDU-vw@%yo^K^E}SucrV9wUI~u&Ry()J
zZ<COa*lB&y!bw76!$Ap&_3m3XfisyA_iji?9F(xOFms7|Jv|g1>oPVZm<!X_H_Nxq
zIhR)zoa$6`LbmL%7X2@!n^$iV-`;(^cO!+t{IO{tf71*66#_)l^aL4l;_=~wHVFp{
z1k%3WG|Qsyn{GV3bz_wSrd#=B{Fm#nu1goS=$PTASjy{apNWQw+nvj>uGiLo>qvt!
zNSuBiRla@gk8dNJ*KGvHJ31S8t-bHsb#Co&+FW8CIDFgj-?@0{(z>5Lho-nBVr9pk
z2x`3KvbLLx3S|{@PF3|CZDGMqBoL(=_X#nQN@vPQ%|cEiVlS?yMz?}b6)!;%Bh4nw
z*-A(5o;#O|5l&TVwMyA|cx32@^!JpBexKy$o5xFn@1{n$7qkvVT3aVX%zPcl8{8=I
zt?eI!J15*7E9ms|n%<r>1!q!<G@<K-L$fxuvC$8e;+kAo-fjo<FNO?x7I$bFmrvws
z(-PE<;-eQhQ&|R^!-u2e>15uz;u6`Q#mSa!={pY-XD7ffG4)tpg>YI4eCBIChegWW
zv}0dh^4XQs6{CMzlvnb0AGz?MwyW~jyUogzH8gu#r@=^U`P2FPl{ZH(@Ogu7={_iU
z;mkx^U%c!I_dGh?Mbo>1e)F?@z{pcMZ3qlj0_)Bhy7He1yf?V>Oo3&AV5_`6_H<&t
zLn`>w^XrL9*##r3%^6P4(u==G{mnZ5OgW4-e7kqpL>PG6E|cz(k-_jkp`J2Bsk5jX
zV!AX&=H|uBz$;X3+p*c+@-$)Pt5g0@z-#e~PIuta^Z@(K)%z(gU)IJ5Lc=nus!kET
z8}&I6N>Sg|iAR-JAH#DiLr9g2T$V#XHZsl0$tmgcO59`nv*K{?$RVUvZQ`RD^R^UU
zWqE_p*X$bj=v$Y>y4l`BD}q(B($0p?IIFf>UpGkT=DZ4hXJh?hkNm;=?mHrH{<XgE
z-Te|N)3U`9Z2IQeII+-X2o+o@s~w8d^1df^NUhYV+cf6&ieT0L@*sIDYBC?j88)$2
z(06IA!JIG^FFNzswYD-cGATu=se4UU|17Ynaa0vtLBV;J*X_jNm|05MptYTyt6*jb
zEs*0b$(y^nVpWbC8hRuo#QGnT-#+!tE>rjNc@u}%g*@|o|Aa`4`IB0y;-VsYV{-wn
zynNBS<mA#&4V-qtFegs#e<o4!igfJhQ#)D-_SlipJHo}|ePjefPUfkcTNzrkA=7{)
zlzw9QR6TtD^JYCv>`fxFI6>NW@!S1kd85VWt<Mb0r>9R*1U^IC?-!{|Wd>(XHj8X(
z11~=$hina87`L&@=%Yi&pJ+%D;#O5$A(`Wy8njG3E3!^_;H{q0{ad&8;YyF<Zclu8
z#Y`{StQ_ko7V@9E%lWlF>rK;yf`pwouC48mLT`)=n_Wt2Q;zfMNYkX9U73>-j~h^X
zsVjW#L<IwT>Z*8F&#!eyacODr$0stVpD(O*ulyqs)~Kqg3d#dhIz1hdsAS@qY_jCu
z&r>vwJzaLjIAXddJ8~E|7)JFbQCAvJf*E2nTBNR~W>sHb-|%O?gHhpa-1*v&2A$~1
zH_tqMv!KL28xM*hu&>Bk9miLF_2b7iyH|gkNtu99BrB6TL5=Zrz<BFMFpZm}l*&!f
z!b!1UB-!2fMcPo6zil=udhVkf_tIbu+?hUK;oS&Xn<!Y2#p&v9d@!q`y}i9J-50ap
zIl2%LGxy_T&`2|7yfaJ9+OsC`g}v$W^+e^f!_YWz1tpv+<$pSo9Qf^@yH~m2#?pNu
z$x0^Sy_Ry(Blkabb~5g)+wgwnVO=x2?Co23X*GNLeUO>}T7tg3I@#XUwWRgq&v+XR
zlBJJ$;z`1wkG)=-OqRag{{8zow9;fUVh}<SzAR!8+%$}hq16H1%sjLEi=WkU>A@o}
zl7?mv;CY3~rcP9hSQp80nT2#-M9SI5Ny@;dpBD&bquP>$&-C~v_SBnF62CUUPEPjZ
z_n;O}U+GB04rc1fFmbeX+`h3pt}ZU!VI$2wurJ5v(v*#AK3zyU^OKTeK%$q}YwGJ%
zfBkyLTBK>*NROD=@nFc;I$_}XA<MMqj#sF^KbjebUw2;mD97MR$%prhcWYN0&m1Ek
zwc8fbSM=k}ARd2;+dp>){rjr_7w*~RpD!ao?5)a^H*A*d_Xx=LPdTBf*$I*i5|Ck^
zZ8Y#DL#Ol02{pAgQ1O1H*%6tP`F0Kc(66UsR>cCupWkz7X=&rc+bhcxP<#8Nj10tZ
z(}2oAyfVf;ASORzy8kLAMnLY$HXhU<HC_Ao@uU8&pBK4IV%Ay$%`Yq<go{&Y&!1lr
z31$s+s&%^0(0pR&EOm4=w6*J2#G=7BS6iA8K4AUGfxU-ybhd9?pO%pUpU95P$;fbp
zK-hoDd0p8orBrk7;FZzV#OlSxaE)Wf%ACo89ghg<s<vtLjHQ8qTuaT_9Fv&oqL=nn
z<>e=0#C#TFqANRM{<GSkox#vv=V&pf04-Y52c<I02T6_i`RN4=2DyCsDmcp@x63&7
zas6EmM*KnA!F`a7phO4cM3-d8%+#A9Zx8sVKSVfJ3;Frc3*!THoHGcQfdNa!BO-##
z`&v(n#55sl5rgD1zC2OtAuZp0Gq<-eUK2|G!Yudj%Qiw*X?Zuee~jE<cZTl<fxvi!
zb#;2d%vTu$Xu*(NZABr^_-mzzZ>>{Z-EBH})GqTn47M=QT_@)gb*OCm1*eO2HeMPr
z{TR{(k}E3xEPM2s{>(ZF%-d^{2L}d<hq;zd+wIt4VNI9OS9^$q>#__NE=ue_`Qv$_
zJW@q(aW?MnKnx1?Yw3Lk$S!-MDUet^skqZsGkwJj(Qpl}FDqgMt-6+5_4UXNoqfe)
z5gnTGTna{IY-}t-Fcb-Oa!OG>dX)Xr!F0ba)w$GOKbU!9mJbST`iVxx7SC}|95sOb
zbabE;RIUH@>+S%b4dPb*ET`q>Do4cRw<akngwIvq_<r8j+Ir4MZtvb#Dx%54<mp6%
z;Wr;XSb$uRVyCNsT)02>^QZgy^XDD&-p;iuP@c8{P#EYsyBZ5^j;+`9ZT>pbIBWXy
zI*GF6nSE<nQ=!7=eCaZl^=10rcCBCM?`8~W)_uLUwsv9ruMI?WesoO)u+?97x$w)E
zkdbC|+UwUL9qHO25sA<`Pk(>p$hMsaflk4zs;KnK`Nm#_!DgEllElm1aTU%1dOvvP
zFql{*kWvpup&I-H0@`%X4wIjdn|zhGl>)J1=I;&Q$-2s5IO@oZkbuWm4Td&$HUp`n
zLy+_-%|QiG1m3x(9@6J>FhV%AAspVb@`q#O3D5$pXYuQ~XNq%LaW!pI25am2>RR((
z+y~N0Uzv*?zDKQ7$YD|+YC;K+t?i!;t3m*C8i^N+9(SA-c-7y!rCy8V&Sjpi%utHT
zvsjpLJNu9@4tF1FP&_XD^uq2sjQ#fHZwUVVd0L9<Sf)<tLj%nmexbFsbuTFQS`<GB
z5C)eC4I?h>&1MVY48n%JrlzKl<E#@4Ai_ZaTg|x70KedHxT|$?>ft^F2=Shea@?P}
zlp;_uKr-3cI5<cN&)-;G5v#h6sLyv9&mPmzumy2l8m|i(=o_>GXyN4S%qX&!)`Rz1
z+~2gb5ARB#mpbGcjm<;#y(kat7c{~;=U5f@Aw<vUWXmMTBX_c*ou_A}7G=?Cr?RP6
z-7tz^gya+z75zfT{QmV_IOPzB2djG_WwASM#W~$O!h_Chqs0d$83X)^86qQ~*+5vH
z>`bmB7HH10;tv~se<0nox;myEciei|@6jlZ`ryGvlerJl>L*UPfcj+;p``@{1*q9Z
zQ?yeGq})A!zYc7YMO{JmH2>d9@xS6U+5Nj`=Xc(G9h<yK0u!6Ut#)clN(f@8n-M)f
zF17ycZ{zG8>0Np2_J$y~u9>E%O5m?=pBgcL=A2CUF56XAtry1i+<bhnIWZNNM%xVL
znma0EBC+fmTyqbXsd?GjdaF_RvENT?sUmM2tUENn#H!hJ4@>xc`fI&2ScKuMnQ!mz
z^99sZ@3lbo72E3F`uXV<N5#d*Cui1#Gk5XQr5&iz3h^pH;P&or79c!2AYN;y?i>^B
z>(}Ki@lwG+KY>f6>ds8W&xr{Qpr0ndaKa3Kc<Aq;&MVpNPl_0H;tQlt>FDT1&UhgP
zsw~5rmrEjdL)H)=w0=z1Waam0!|IC2)yZj3_}$@MC6_V^xD}1|KsN+|xd5`xe|LZL
z|5eEo%eEv%2HsU;*6V5A%9=>#b6MQ_i5yz)>Yo4v_?Kakq)af?aP*l@P3Vx3UCtSg
zZc|aY0p(EFQ~BQ3B;n$0pr2J!hr!RA>m@K}IDGfQu;!C8G8rg=>3sU|tWvl%^PybH
zWJy+L&{(?-V%Q5;8{9(0+4eP~O8|%V^(F1)8BpN`z%Jm+Q!#T-RTLw=?wS7i^+`?L
z(z4{-Kj9plRr^A}pMS&mM@Kn5-#=wt^~w3zgi(o(j?OfW4DG48yJ0gPfF8Rx*sW(V
zN}Eg$8>#3%yZSW?1~ZGZLzGcB!u<V@@7gj6gvPHD-cX|(7K=3+OLb1o$iRVOnF6rB
zy%7WWs5Xd!&!f6zW&nAYUFb<}9^*?BcT|Qps-NFHQXP=IG&4jnygeQ`){bolN~XZ_
z>EX1oR)sK^t5@@9>L--4Z?^7l_D`<s)NcB9ekhz~=UIQNSRd5x#CCS?c-tVPH%-7b
zulzlp73SjNQZ!77iJ@I1QbA1yy??M3HC@@p;s2hzA*#@1VrXDhD2sizsP7Nu7VD_u
z&aB~G6PAgAer>%u##V$wNiQAF0pq1MF0p6Jf#vWkotfJb_w`y(P_Djj1k)Ksi3%Ik
zwMmj{xp7K8bt6h1D=Q*fWS@0HrY;Uw(NtwkSN>ybYwHMu6+kM=f0f9{LgKT3UO%p<
zXZBLffCddy%%I-%J7<DYnR<O2f7Uhn`@>r@dtbG85aPbawmiC%6I*o&O5S+QbCn_(
zroA@2{XUIko353q6JX!-kO8>926!CrsP7MlYd{UhP`5idI_3ctY2@(fxRw?Kn8z=d
z3ayeM)v!+8h-r5~&vt;<X=}PScV)2@fEph7i@X24a0DWw<lMn;ChBj;FBiGGy2=~g
zDl!V6%yT6`qC3Kdn>@hF2$&K*)MB8jnp!ZRwc$SwXT>hd%gfs~u5aMr390Gnr8h@g
zad~;?Vgxgc4N_-zE!}*BSyL!IaLgFCXnuY^@42Iw*EGYb=G2EO9vFWRP&4Y<+ALrx
z3qdw{jbKE?B98B2-=?Ta6*y3h4ZnXprBHR#)6--4zDG1Q?bakS&`*WmTjE(b5*v_H
zN;r^cTp)UYE`J$5hIPY7fBGzo0D43Q5luDSYt4%eW|sFp-hG5la3j6thJg^^@%TOv
zF_|qJ*9s1sme!7^3S<S%T$-AOhK4^({`8H!nFg-b(2bUz!_rOv2ZH}cSNl<tsaPz0
z{7Z%zwSJ?-?{!bco*t%^rn`6NY@YreZ;N1-?-B9Bp7OrEJF?(Ls(1rz`<mt*x@lFJ
zs2tbdWmx)&S&myt5&r!A@hHy#PE}QvalddO-|m%Th`;}LK^@$=*gJ-IHOsL216Zt6
zjT1S0M2oZ|zPm`}x>@?fexQ-b@yc4Y@9sYh5iN0+gIN%F8=Jdz)XddWI@@`Tvzx5E
zN4@|#Y|R9mFx&AVzpAP#<kRy*`u^>EvkY&S@Hvc*(B|dwJZ;qQ?QWAbfy(lYqnPmN
z&Jka4ws|y0@IrvufWgSX>MQ(x<&Jd~C`@<It)K`wgtVNT(3En&wj?q}EPONA(b2J8
z-2u>@KMdx1;ld-}x|z*?=4}=9Lta@|P0>oBpngpjd(4l1zK|&I-xxVd2I{aMSDF))
zY$_}xx>vq$eYW2ZEY1SC9*Cr11|FMztNpls5dD~fY3$pb2Tu3MYIz01U?A;k^bl<N
zo;`a`n3xRtp!tkniTnKr9&VRo;?i%>zMKe(W)YzQpyJ72zFg|~{J9deGx{5lTo|x8
zwSZ4M>Y6afN+1Ai?d)oSF6~#0{Cah?ExGP?mtotQUd@f2PeAh63;@<wfXvDSKJT5n
zx_SsmS7xb$^2_TECj3+^gEI{@el?K%r>>zD?UXamL|0j#vQA4)y$*bo{?+#z!&Wg1
zqi_u>iqAFAFLB5XHHlfO0j{AR)|u%~74yP)S9o`iCJBvWf4|9&V8fY!trUzS%GX35
z;E}6tzG$Q)^h4FGaPf1oZMrA0zeHdZ$yZx8iT*5f6D^<0GJ#DMfxgDR+L>!p-T4KH
zioNaoXZz>a!K9BS%mbY_A0@>=>q2WEJa~|&_5>)bs6Ah<ty)%@#9V^Ix7tW9@Du0f
zIm)|3hQ07*Wh-;<<*Q*>&U{B!Sq<IACH!Z{cQ@Z5i-{bC<;H`SmW)9-^TeM&eSI9D
z_Mz0|$s@!tn7XDWae*}|58MsQ0W&WxBPYl0(5BK8)N=MM`ON+K6I)zd{L%HO4h?`Y
z_Qi|dgM$l$tRzL_)K{;{fDdM7C~w*50-y^t$v`dQJuaF9Fah)Q=JZz<rnDfRptPVP
z^TGtkCMp6zfyS;aWx$1i;r<Hc;_!H<Yu5_4ZQqVfNhugjGVZE7(0yj%*_vbm{4dl}
z2+Qiu)UyQ|y57dt7Ff=r_wUbZdQ@3};MR7T2)ss?_1feepq9{KlGH<Bb(O$qEx$$r
zpW@{7BfG73kk~AeWV}M3@&R02s5#MVlsUS?Wb+c}eN9~5A#q1pc0?lOxuYN=S6Qc8
zzpY`*?-w9foPat+GF;u@pHozKV!}jgpaq=TB(H8nGIIrx)0*S=tcKaqmxmgV!9DZ3
zHLVqALD%xpY;Hl)(!DbV26-6q6W!RTC&EQ*C+mdNbhqNGEl#>2?tgqG|H3@ZL2-`T
zy*_;zI4RbeXF4X=y<yXi9sq*6z<lzjXjY*8(caNP1Ll%Wgc@K|Q!TPr(c=Xxpkb<T
zbLgI{?`FwY!{|Dvn59>bc34XacS?R6c?axhZPeT~@7TGI?Z7Y8h!32;ac;I*Q~^3`
zg@?=^(YMNC7N{gIhO&f2V66T6Eo&Xui~K)||LMS%N&U}>fGr2t&c8l%XYFv~k;Hm%
z_-Frr=VHdDod@H?FE-X_{P+{bCz69>%xC)cQ~iuR{ncrk5RXrv&AVgDDBV4w+;<2~
zBq<aIt}_2{7^I<Vt_6E|^ez+;TK?k=ZFMAaHE5c(V$^YfZiX%<saRX8JK=1&wU^8T
z%NjzCjnvKSK4?9lCXb3;ZKnAC#;Oeq9uKjHj&<m#ZF3DIkkx&c6UoVbL*tdxtAVXS
z>3=Hr2OVHu-CVqmGr#j-i|!=^{lWw{HnH!R>T&CAhDSR0me_{sl#;So;6v{~dF*fR
zu|*yKL0-u_8~EUY?&P|+yE-Zr6BF5G%^ZR)65hY8Jh>segce>MDrp>dSN51I&J$mp
z=+qqNhT(Hqp_QKHAKP6@V8Ibzxy>X&tyc$y^_O~&dm<yl$XSYewUgnMO}LFDcS)*G
zu!NqhNqIY(E;z||KD)C=0$rXw>RFPn>*B4SItz`KKkt_;z0;&QZe&=y;6GDEuuF2M
z0+1ymgc$yVoaZV?w@8_?gFmV&Vv9+-UM)ys1;dlb2<e2J<<rh<?(m$kqu_j}+-w~0
zVJdQiU&vag=)+$M3Dgv9`5>}!g|rg(0j<c@7YwfUK4WMtd+X4bu+4qfG$S0<4?bAl
z7i7-cE_xp`^d)lu8Idgzp5>iI+|~SBr`maXZ`M_v!=V}@^E@5Vm#}nXswa+Esdtma
zdOkRBOfKZVGJ)+++kJDzA}cUpaA5aDfcvA)O2fp<HOhL_m2Yk&3{RF|rvE9ldhmy5
zK!WC0^+CvB5RXXl4rX~~V54b-^7fgk#%UGf&sq9?7uX*oDg_N++llG?s$#Nkg}=O$
zIm6$4xf!b_Yc-ggd;#4S={9KWnLoCb$c-94ved7{y`IgvH0w{92IIS>$?AHeQ=r9t
zueOv>Ta%{WJ^q*7h<{`GOJvn?ebqpr#?q6|*KZ?2d#XIr{0Sl0*_9+@R$JAo@EN1+
z-iV$#B6lDlB6*1RNp=0bWz70pjiGJ<$fVP8uUm2PsEa91sRbKVExhzyah!@7r1|p`
zKa6DXJ$elnW~WT31;Qr*(0u>3s4y(--iK0cTvLj!t7@EV-wa8-+#JZ8_uFL~_lv-~
z$b;b8-?sO;*o_8JP1uF;{5MB~tBN?ugCs_+jZ1-S;N8r!gu0M>dvwip!c*+$OVD_S
zTH8jK{B72Zw&kWAsRlcZG!UAihVVmPF?tuG<riKDJzhER^kUmCcr1w|3?)@^-bO^f
zXCTiYv+{TNAJ+(EYw<vVF}#OBs_J>Q9MMv+Z}LmZy@lnfdl#SZhZh{()XB}0%*@}P
zS!jrI4nxtPgjy9`BKu+rIk@pc;DZ%BmE)DoZR`d;`?)9ntCdNNHjY?_8T^RdKxkaQ
zZ6i8UQdfUucsW>X#Q8HC{(2MqxuC(uRd|0TPa0W(0SRr3K8_P&7OfT^ZwkCqH+@Z$
zNke%iPww7H|L4tMoL6#iM%t9JFdkDlGWkYQ!y&rCyDs?6CPK1s5>3&z7CI^il|918
zqb3z8g;M-{6fWJTXZ!9t8FkN$!-`oh=+#*ZJGzS<V@qO>ik(w}O%sCb7Sl;PW8HUH
zv+uK!$Ecp^HHs|3U0T;nYU)rTD+<9J|CHz$oDI(`+2LQ7EDX#dM=u;^cogw2(Jf5M
zHtV^V>s|0mC5BK+1vQpXUHj3+7_pCzuKAP=>nP$$gZSbCGl<w4_^bq;nZu~&;49jl
zAdZv^(jCHHwrbj1`sYJ}YpbxtflQ*Ae38Rqf&s}wRhW$_4e<}IxxhVXA{(KgJaq)1
zf*phYxXyTp5OY6N3S}Dxuh!?Hyh*SiNWij%D#wyCNl_|1$u7_|a+o;o5S>yq%u!r?
zfj-gk;7P=@UiRVFfgM(y_*${^stdH%6`+D7@3{-w_@Wh(4n^5IEiU0T({7(;8}90G
z7oaD*dy@{a-|jMM7AZa*`4d$);nwUJ#~Q&_gnJj*x|E$4TgF{(=tJgoEeP8~I+^>8
z%!}2D&N&y6iae9U^ea2lT+mW0t_?N-Rjuc{vXGvoP(Z6y-z_*T{QZPrk)tw)y6C$*
z%o@M-V)s=2n}yq0La#hgj$a@wbNDdNDs#w+S9isMuqiQH8Z6QSU$?@awO*V*8*eU9
z?${?CTAnoKHIlE`;2g<`7^-0L8&HHgA)a(Awa{yfbIZFTBB(SYbBZ`>fDwA@{H_(r
zG@Or|()w%fV|VJg`%eT%UiLuW1c~a!Ur`>~F@wWPrV*6hP+gsB3Ypb97-d_ntD{4B
zvyB^!rXWlE_ilGTI2~c^p_T;)o$2yEb>8~Me<ayv+ui?Z9XM@U7q0PUy^ap}8vTsv
zoiQzt{%u)^>LZ~VE;XktjX~>Nhj%Ha*GIRS19cTb;}l>mP0KtYx~FZy98s27^J18*
zHoY}C7(rpl4Oo&t$!27D{Ifm{o)M9gEl2(egDIBZ=f6=Re@PpVhx%m$sH)V0T$O1?
z$_C|hR^#*;U)EsldR0gVIh#zob)t>!u3+40U+p$6qYy9}l7)EsRvaiA_KR7=(5e$1
zK6NZaDX-=w?|S3s&?ix^Yd=;s`!-r|r7-*;*{`AgpdU=6L_%B95xL&5^yopO5uoBa
zPU=*5k_WC=N%72ebc_eP$kzIwzkC@VjE#Bc^B6U)*1vVBfigqE(3SKuVp)Wn|A5wh
zMB3-;@HALQhzBKYs|;_Sn0<Khz+-akTg<*0W`Y9m0ylJs!jgepb?cXbjgR=`k0nG%
zT5%^TlaOr<XGMw9y{!*>=7wh7!Tpg&6YBiKh3%q;-eIg5Qg>B#NO>aPN4supsmVl{
zeC9icJqt3=bWm<lxT*2o3M{Vrlt%TAjxC%c`5ny1%=l47g1Pwf%JowNzSG*flfOq9
z52M_rW|5tuSz-i1`~h<<u8wNuW#EElpP~P)09$Qaap>V<nB6goyaSR$aYZMOvfUNA
z_0262?qOU+HnEy=jpSY_Gk8$}K2)C7S(A|#wyZx?0==i&j~NVutxRBQf*-k8tZOHJ
zbvp4;pWXVHzg@RzQ{0Z(zjx9MjjR;uX6CvdVz!w#^f8=GD9NhZUCWaBm5{)PDpN;?
z_VSN(WG0_U4@|gJB3R6U1QoM!*-P-;VMb$0(e3c03&Axry(TDYZFtI2%de1Q_}~d&
z35f$Q*Y*NXPgxp~v7R`-=d8F`)$(|Q4~v^CEWMR_#5$;A+)RYXm)+el*F4PHM*L-p
z27IDLf4G}l(puX9Y0>jC*R=rh1f8$FNFH7yRy%4>sTQldrY!1+FHPB`s~}yh#7<O~
zw5sqY_PzcMmlpJVm8S|H4<?gm0YnlS@TI7rSC4RoMx2mr;wm#vq(o2|;rqemj^A3N
zHpk;Oy~PbK6uPYwzinAB&fBL@Pk2#zDm+`unp0?+Nm}NVC7BJrA}!mv;wu=nyF04g
zVvy%HFH#q&O>jL2eH&*_x*zUlQxXfnRwOr6rtI7i-SZApxVPz`C3>=zEvxq@O^PMe
zGGuh}vUIKYApfrEa}^4)PS5EG1#uyWbfX}<rB@hT?l)99(~TLmsJVG4JXT;VOI@A|
z?Iy*3f5vd9;0!2GNEf3X0BlU%DaMQj$V?a3i{T^hx0<+(FdB`_112}1^L!`e%<8%=
zj7Ww2?@?_W$Cz3-C;sVOpXZExL^Y(Y#mL1Z0AeGYy5Z7N42L#eOJt#lJ=?tVwl~4O
z8G3;kffF|-SH!KaZulX~Cs1xk2HeQbgl?WbbxP0ec4E&KAbi8?W*~ScJ&%|Er@*_|
zZZ!2jSp|fKeo$feqQ?jQ_3}>lH$P{qMJm+x9#Y(v>85km4&LCB9`yDR7czvm=M<@^
zE{7Ue8G?vr#tOooZ-QeXf1y8!UMbN6gqP5CwQ2n$6f+U_cwZ%=?`W}x_i>xKae-T$
zHAA_#l?%A^gnzVvX5(aAyGVI++Dt8Kb=6kv<it05TS@mo5b5fZ_ezLtlpNNHf6!-P
zT+1>{nu=!4Nb8A`CQg!inaQ=@94P5~`}~6($NX(rVhE9DB!j!?#Q*s5+|C_r*6G9r
ztye1WhmgQWu;0T=zaKU~R}ovBohv~)saZea?k0eoxztGyB0>P=5JHIy4~pE=aJzZ?
zMop})d6?EHKcd-E=vL@OH?gY||0PPR&oZT;&aJ6{zYx{h;5U#f?sBExJXAA{nTTjw
z5T^+=e9JV#AA)L%e9BZ4TErvW3ALUe$+7tG*GW&`u4ckt_{2A_TFzIRGS!KFR~4D|
zJh}$T5|k_u{Ufu;-(bC4IgUegJJ6l(e19@yK2=Wt_iFr#xi;4yO(Y=`v@U0~$8z(>
zqkg~_*i3kKRjLe~+EVLrMwG1EY9UV;HDLt@7emZQ!S_Z?c?%{2amD^?2mvIBPeCo}
zVZNp$k9GMnJAi{252_#sJ{_iKU0!7$wq9;`t4b-v<ZD_Uw{dGw>OIn6X0S0kHSu!E
zH5m=xWuqeVAY58XQ4R-pU6sY%KiF7nJYCHk-G9*-U+9SzXr~zw5Kl@bjl4^}#_+gI
z{6kL*%4S3M(&$j6dC+9Ui+fzX*ME8D@jXcsi?3J8&&CLLfcySG1lxmN=j+tib9Fsl
z^2>7)-+FZ_T;mz=osoDIIDsrwYO&|J;GHOb`^w)58&@iTmMQkczmVTgJC$Fc=+}hk
zeU6BvRItPizCc;&IalXtPg(_$qL%p!!e<)!@{MOjisntD_k!orv|VxXC09Mm9tIJa
z976M&+=ai8oV-)r_O2PIEFUhAsQjE^+9^$%YJ<gUAC>NgMzfMjrP_3qSZBPB_Jw6l
zGkjdj-YdkM_bz(6rjYAqn05`vCyfIWzbl|=fl{Bg$RNrqX6tjxJ{Wx`JQ}H=jqKbN
z+~`(zUX%X5uKb+&X&CiEvsFxuTa2F7?HVTyEWvBG{`+_O>x;lC5^B^ZNe7S8j8Sbd
zXwnR08`1IJo^aAk;#*>&rgP#6|KoXgBD3OMOisJuok}P7hIw{ci7}_JI5a!bMfoA7
zFwCXsFds7eic?PBj?543v-8AZ+2x&{gxbpj8H8wVrt#26w$|LnMmAqQh(HdycXSfy
zz{PZRobY^DBbj>}J!7Zi(c@bI#bJFwzuZiFGE4~la4H-Upy#Zk*t0H-4fq8>Y&SPr
z2LJaoN-xf?(>Z56D72^Pu6&t2=cjkbhpJ0ufo=POq4dX*%YhYitWnwh2D`ecJ>Ea;
zl9@g)ER5m{!P+GK&)Q@Y=90EoFiK-tQ%0H$C$V*0Ut+^$KRpPE<4Pgc`E7ICIhR^9
z7@p;X#b<@TJPJ>d<i3Fgl>pZ=yEMvcnS0g-U`=jvy=HI98!@I*R>Si6)v((YH`8bO
z_9gb(qO5_)ADLH71hNhIoz>peDBAm>mAzci4c!;5oJU41^W|(2hP9)T9FsHO<UqUI
zD(fbJbHcW}D$S90d>j!IF3%59*$cKVhhI)X(OmKGPRE^$j}85{OZgzK0^2vh@kh7Q
zPA59w-3FGGe@E$;=_Vf>7T7c+%hF@e0hFi1X78VjVaVNW_m-rImwJ;BMHle*jRLWd
z=mmq|>Z;V@1!Qv(hYAUyUjX?qGO2*B=;jlf`Dxm2^@6uFAYZSm1~-q#ESzPOw!2~I
zYA*Dgx(mU&=a0~BDdE_PC!PWMGc2)@wHn~;DI0Ixc#x7V)d_3HlSha!A}zP*9)<Ij
zPwpobUbJj*69hdaKhO-W0o0iuD1Jxs310H0g$~x(x#IgE2-kD#0ElAoE=Os{5}kDP
zXrQk^o?RKj8S|aRbb|9sc-|(C)=;rRxcLWBn>4m8mpfq=rk`L#mqwijj+Ecf+(oqT
z<=7788@u3akeB5;dtc}A1%b&C!F7=%H-K%BC-&}6KAA=?^G>fkq0E*#0w*%pSfx-O
zRi}Becd+BS+1QJM+%+ye1Gsd$8#93)dFvPhxHQfEfA!k!yIPLUdafxc;rHx&28^I>
z+^AaTMkBe^HG3XR6;K0#J$e-Q*tBOe2bW`(+h;k-iXYuiXvruvY<FwUp}#bvMF!Ww
zg2^l9tGgGEB<PK7#tjK_Xt#wKn3BeDw`^tY9Wr5AEXZw1dQ(^$?oxgG&TWS<vn}}|
zDT7uE1;VHyOT%S%_z8b?=X2}AzTlScU&iA9mqpoPD4MX%wJcW{J(hvX@KN3@?^Qkm
zS^Z#Ic~igZdX@C_Mv;Ta!T8GNre#vUMp|G(1U@@RdBBSNk=%bg;)yD>O0(LZ+^f%)
zP`Gkx;6RoixW;5Ou-8MckOqSF-yMx|kllt%9`f6oc7-*_PYZ}%sxd*>5pk;S_lltn
zlps=O#+s%_5Q*pz<tz`rRJE!{3X<G4{8I6Joeb*P_x7KWAut#;<Y{(LDZ6l~jQB=I
zzqPere{dWTQ%fYf*w$(>KhC@{4rxlPyFGmX>K7bRh^R4Twq36(oj$HneF_@5ortZi
z0&N-v|H6>BJL~T1_sFu}zd`B(sp`OmOAWT@cfbb1cy$pylBk6v43E<<5w|);>0luK
z?vL!=Z1t&6sva%RuC4w^K3&4I%B<^RdH}PMkp?$R^HUydV|F%pI0pxX4M1VAr5EVy
z32FNlRbI+hmd~=ZdWH9WDUZm|yE6Uoh|INq3$vAd3mU3f{?HrY2F<_@5SH*D5}&Ut
z5(|lSd~kPzp5|#A4EH7y5N~!a)uY!wT3a2lWF_+J*Y^zVKs-yt>4-rmXVc&n?lte}
z3yr-c1G;N|q>&e@QR@^|f1;!6kp72~{=HdnowY7e77wOTH~$tGkf}8T^3`w`V)>%@
z?-T}^4I3!WgU9o3d|2`f<8o8g*oGmp^|@B^Zs!%bD+hS{z7!;H&mR>HE=AD&=8T#D
zvB81h=LWADH?ah4iW#Z1+&;R!{6twrZfyX1<jCRW*5*oJtbiHpQF_MR90r7;-1Nh@
zZJZk`Z7XeBHQRaH8sD!3#?^q)a{bWLq$O6~7=&%35V*naC>L(R3tKYK(Gh8jwAWl>
zL`KL<L>;f9I#*Z|UACELW?N&?QFE!0T|s`Ke`=D_GK+q@w9>-J9a(cGIevz~G^SnM
zPi8XN=}X0KRy0I|jAy>db*YR?`t276LI-&<Got_4Vc>!E-(QF#)v+PAZyvnLE#LA%
z<K35}&!jPc1_?Hxk%TQs&C?b=d`bF~r*=1$;MG`TtH`+g`6dRd8V==aIv;xpkB>NZ
zIRTl|eA#*Y&U)Z1@Qx2>t;W~=<&w9)__$rM`c9EZGJh#Bxi3|gY=4bd?U7mBfvg8A
z^so%h@u6pdYJ;bn6(wGmy_8%+0SvFi8qKVjx=D0vA$cF>$FmC<!DV}}QE}*<h=7-H
z(vrIQ{+mjc-f~liO^Bt)0{e0ul1nkW1YrZIY9Pd9H&^?L_|jsFkLt{Hp_3ZtgV9KE
zf`7)10>wgK=mFk*Zz}GR9iygeF_0(IK6^lt)twySwUyB?HtehS9J%=pD@@a_@wMQn
zegELZ()vJ)FBT@Yjj6LD5N-{9_jCRcpl>^F#+F&sy4VgCKta2w=CT51d9GQituViv
z?}(Xb2K~XB2&%hWa2o6{bZJ2D(z5B~hBA_$?73g!#b-Iado*ziuLcbGB@K9j;QHQl
zfMXoeikULMP?+EbbP>SdSMQHI`i`{S%qP#rEZ+!uu>D{eS0JQx(V|BP@iyPuk|pjP
ztZ!fI-`v>grp&*bV3S&UW~XRoq|z~OeeLBezpB~UMiRevxDP5%sBCLE_ItO;G-><U
zqWSOrzp<EN3Y!2-nZc(G8!*p?8I3^&M~9EpuXEdSux#{><fJ!+0O<FS{Ci)ib+NN;
z55X9|F@Y$u=cT%cTT<n?Km4EvCCvW!eu5#4x?$~B!nYBr|IX&_|I(iEf4}y>w;%uC
z*!~;a|LOxD|0dYK3HEP-{kI>h`nSsdt+Ics?El|Y_Gfj8>%O)l=*n{{|3n1VQbOX*
zZm@Ca%8`R$d(ee*U}sX|9dKD;9e99Ke?3t8AsfL1p8Z?EE~9@x_-}arza>SML}cWa
Wz{8zmM?fk`SX<g#RGhzZ@Bab4qPnpF

literal 0
HcmV?d00001

diff --git a/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.txt b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.txt
new file mode 100644
index 000000000000..b2dac57b2f63
--- /dev/null
+++ b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode-expected.txt
@@ -0,0 +1,15 @@
+layer at (0,0) size 800x600
+  RenderCanvas at (0,0) size 800x600
+layer at (0,0) size 800x84
+  RenderBlock {HTML} at (0,0) size 800x84
+    RenderBody {BODY} at (8,16) size 784x52
+      RenderBlock {P} at (0,0) size 784x18
+        RenderInline {A} at (0,0) size 60x18 [color=#0000EE]
+          RenderText {#text} at (0,0) size 60x18
+            text run at (0,0) width 60: "bug 8626"
+        RenderText {#text} at (60,0) size 363x18
+          text run at (60,0) width 8: ": "
+          text run at (68,0) width 355: "Strict mode erroneously triggered by a broken comment."
+      RenderBlock {P} at (0,34) size 784x18 [color=#00FF00]
+        RenderText {#text} at (0,0) size 642x18
+          text run at (0,0) width 642: "This text should be green, not black (CSS color values not beginning with '#' are OK in quirks mode)."
diff --git a/LayoutTests/fast/parser/broken-comments-vs-parsing-mode.html b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode.html
new file mode 100644
index 000000000000..45b1b8b986a0
--- /dev/null
+++ b/LayoutTests/fast/parser/broken-comments-vs-parsing-mode.html
@@ -0,0 +1,9 @@
+<html>
+<!-- A broken comment ->
+<body>
+<p><a href="http://bugzilla.opendarwin.org/show_bug.cgi?id=8626">bug 8626</a>: 
+Strict mode erroneously triggered by a broken comment.</p>
+
+<p style="color:00ff00">This text should be green, not black (CSS color values not beginning with '#' are OK in quirks mode).</p>
+</body>
+</html>
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index a043cffa19f9..6b62324d75c0 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,15 @@
+2006-05-05  Alexey Proskuryakov  <ap@nypop.com>
+
+        Reviewed by Maciej.
+
+        - http://bugzilla.opendarwin.org/show_bug.cgi?id=8626
+          Strict mode erroneously triggered by a broken comment
+
+        Test: fast/parser/broken-comments-vs-parsing-mode.html
+
+        * page/Frame.cpp:
+        (WebCore::Frame::endIfNotLoading): Call determineParseMode() if needed.
+
 2006-05-05  Darin Adler  <darin@apple.com>
 
         - fix the Windows build (this time for sure)
diff --git a/WebCore/page/Frame.cpp b/WebCore/page/Frame.cpp
index 13a277374198..a110573501a7 100644
--- a/WebCore/page/Frame.cpp
+++ b/WebCore/page/Frame.cpp
@@ -683,8 +683,14 @@ void Frame::endIfNotLoading()
 
     // make sure nothing's left in there...
     if (d->m_doc) {
-        if (d->m_decoder)
-            write(d->m_decoder->flush());
+        if (d->m_decoder) {
+            DeprecatedString decoded = d->m_decoder->flush();
+            if (d->m_bFirstData) {
+                d->m_doc->determineParseMode(decoded);
+                d->m_bFirstData = false;
+            }
+            write(decoded);
+        }
         d->m_doc->finishParsing();
     } else
         // WebKit partially uses WebCore when loading non-HTML docs.  In these cases doc==nil, but
-- 
2.36.0