Changeset 202572 in webkit
- Timestamp:
- Jun 28, 2016 10:01:29 AM (8 years ago)
- Location:
- trunk
- Files:
-
- 5 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r202571 r202572 1 2016-06-28 Frederic Wang <fwang@igalia.com> 2 3 Phrasing content should be accepted in <mo> elements 4 https://bugs.webkit.org/show_bug.cgi?id=130245 5 6 Reviewed by Brent Fulgham. 7 8 * mathml/presentation/foreign-element-in-token.html: Copy test cases to 9 check <mo> elements too. 10 * mathml/presentation/foreign-element-in-token-expected.txt: Add the 11 expectations for <mo>. 12 1 13 2016-06-28 Alexey Proskuryakov <ap@apple.com> 2 14 -
trunk/LayoutTests/mathml/presentation/foreign-element-in-token-expected.txt
r166065 r202572 325 325 mtext 326 326 327 a: 328 mo 329 330 abbr: 331 mo 332 333 b: 334 mo 335 336 bdi: 337 mo 338 339 bdo: 340 mo 341 342 button: 343 mo 344 345 cite: 346 mo 347 348 code: 349 mo 350 351 datalist: 352 mo 353 354 del: 355 mo 356 357 dfn: 358 mo 359 360 em: 361 mo 362 363 embed: 364 mo 365 366 i: 367 mo 368 369 img: 370 mo 371 372 input: 373 mo 374 375 ins: 376 mo 377 378 kbd: 379 mo 380 381 keygen: 382 mo 383 384 label: 385 mo 386 387 mark: 388 mo 389 390 math: 391 mo 392 393 object: 394 mo 395 396 output: 397 mo 398 399 q: 400 mo 401 402 ruby: 403 mo 404 405 s: 406 mo 407 408 samp: 409 mo 410 411 small: 412 mo 413 414 span: 415 mo 416 417 strong: 418 mo 419 420 sub: 421 mo 422 423 sup: 424 mo 425 426 svg: 427 mo 428 429 u: 430 mo 431 432 var: 433 mo 434 327 435 Foreign non-phrasing content should not create renderers: 328 436 -
trunk/LayoutTests/mathml/presentation/foreign-element-in-token.html
r166065 r202572 127 127 <p>var: <math><mtext><var>mtext</var></mtext></math></p> 128 128 129 <p>a: <math><mo><a>mo</a></mo></math></p> 130 <p>abbr: <math><mo><abbr>mo</abbr></mo></mth></p> 131 <p>b: <math><mo><b>mo</b></mo></math></p> 132 <p>bdi: <math><mo><bdi>mo</bdi></mo></math></p> 133 <p>bdo: <math><mo><bdo>mo</bdo></mo></math></p> 134 <p>button: <math><mo><button>mo</button></mo></math></p> 135 <p>cite: <math><mo><cite>mo</cite></mo></math></p> 136 <p>code: <math><mo><code>mo</code></mo></math></p> 137 <p>datalist: <math><mo><datalist>mo</datalist></mo></math></p> 138 <p>del: <math><mo><del>mo</del></mo></math></p> 139 <p>dfn: <math><mo><dfn>mo</dfn></mo></math></p> 140 <p>em: <math><mo><em>mo</em></mo></math></p> 141 <p>embed: <math><mo><embed>mo</embed></mo></math></p> 142 <p>i: <math><mo><i>mo</i></mo></math></p> 143 <p>img: <math><mo><img>mo</img></mo></math></p> 144 <p>input: <math><mo><input>mo</input></mo></math></p> 145 <p>ins: <math><mo><ins>mo</ins></mo></math></p> 146 <p>kbd: <math><mo><kbd>mo</kbd></mo></math></p> 147 <p>keygen: <math><mo><keygen>mo</keygen></mo></math></p> 148 <p>label: <math><mo><label>mo</label></mo></math></p> 149 <p>mark: <math><mo><mark>mo</mark></mo></math></p> 150 <p>math: <math><mo><math><mo>mo</mo></math></mo></math></p> 151 <p>object: <math><mo><object>mo</object></mo></math></p> 152 <p>output: <math><mo><output>mo</output></mo></math></p> 153 <p>q: <math><mo><q>mo</q></mo></math></p> 154 <p>ruby: <math><mo><ruby>mo</ruby></mo></math></p> 155 <p>s: <math><mo><s>mo</s></mo></math></p> 156 <p>samp: <math><mo><samp>mo</samp></mo></math></p> 157 <p>small: <math><mo><small>mo</small></mo></math></p> 158 <p>span: <math><mo><span>mo</span></mo></math></p> 159 <p>strong: <math><mo><strong>mo</strong></mo></math></p> 160 <p>sub: <math><mo><sub>mo</sub></mo></math></p> 161 <p>sup: <math><mo><sup>mo</sup></mo></math></p> 162 <p>svg: <math><mo><svg><text>mo</text></svg></mo></math></p> 163 <p>u: <math><mo><u>mo</u></mo></math></p> 164 <p>var: <math><mo><var>mo</var></mo></math></p> 165 129 166 <p>Foreign non-phrasing content should not create renderers:</p> 130 167 131 <p>p: <math><mi><p>mi</p></mi><mtext><p>mi</p></mtext><mn><p>mn</p></mn>< /math></p>132 <p>div: <math><mi><div>mi</div></mi><mtext><div>mi</div></mtext><mn><div>mn</div></mn>< /math></p>133 <p>h1: <math><mi><h1>mi</h1></mi><mtext><h1>mi</h1></mtext><mn><h1>mn</h1></mn>< /math></p>134 <p>h2: <math><mi><h2>mi</h2></mi><mtext><h2>mi</h2></mtext><mn><h2>mn</h2></mn>< /math></p>135 <p>h3: <math><mi><h3>mi</h3></mi><mtext><h3>mi</h3></mtext><mn><h3>mn</h3></mn>< /math></p>136 <p>h4: <math><mi><h4>mi</h4></mi><mtext><h4>mi</h4></mtext><mn><h4>mn</h4></mn>< /math></p>137 <p>h6: <math><mi><h6>mi</h6></mi><mtext><h6>mi</h6></mtext><mn><h6>mn</h6></mn>< /math></p>138 <p>blockquote: <math><mi><blockquote>mi</blockquote></mi><mtext><blockquote>mi</blockquote></mtext><mn><blockquote>mn</blockquote></mn>< /math></p>168 <p>p: <math><mi><p>mi</p></mi><mtext><p>mi</p></mtext><mn><p>mn</p></mn><mo><p>mo</p></mo></math></p> 169 <p>div: <math><mi><div>mi</div></mi><mtext><div>mi</div></mtext><mn><div>mn</div></mn><mo><div>mo</div></mo></math></p> 170 <p>h1: <math><mi><h1>mi</h1></mi><mtext><h1>mi</h1></mtext><mn><h1>mn</h1></mn><mo><h1>mo</h1></mo></math></p> 171 <p>h2: <math><mi><h2>mi</h2></mi><mtext><h2>mi</h2></mtext><mn><h2>mn</h2></mn><mo><h2>mo</h2></mo></math></p> 172 <p>h3: <math><mi><h3>mi</h3></mi><mtext><h3>mi</h3></mtext><mn><h3>mn</h3></mn><mo><h3>mo</h3></mo></math></p> 173 <p>h4: <math><mi><h4>mi</h4></mi><mtext><h4>mi</h4></mtext><mn><h4>mn</h4></mn><mo><h4>mo</h4></mo></math></p> 174 <p>h6: <math><mi><h6>mi</h6></mi><mtext><h6>mi</h6></mtext><mn><h6>mn</h6></mn><mo><h6>mo</h6></mo></math></p> 175 <p>blockquote: <math><mi><blockquote>mi</blockquote></mi><mtext><blockquote>mi</blockquote></mtext><mn><blockquote>mn</blockquote></mn><mo><blockquote>mo</blockquote></mo></math></p> 139 176 140 177 </body> -
trunk/Source/WebCore/ChangeLog
r202570 r202572 1 2016-06-28 Frederic Wang <fwang@igalia.com> 2 3 Phrasing content should be accepted in <mo> elements 4 https://bugs.webkit.org/show_bug.cgi?id=130245 5 6 Reviewed by Brent Fulgham. 7 8 After r202420, the RenderMathMLOperator element no longer messes with anonymous block and 9 text nodes. Hence it is now safe to allow foreign content inside <mo>. 10 11 We extend foreign-element-in-token.html to cover the mo case. 12 13 * mathml/MathMLTextElement.cpp: 14 (WebCore::MathMLTextElement::childShouldCreateRenderer): Remove the early return for <mo> so 15 that it accepts phrasing content children. 16 1 17 2016-06-27 Anders Carlsson <andersca@apple.com> 2 18 -
trunk/Source/WebCore/mathml/MathMLTextElement.cpp
r200938 r202572 95 95 return false; 96 96 97 // FIXME: phrasing content should be accepted in <mo> elements too (https://bugs.webkit.org/show_bug.cgi?id=130245). 98 if (hasTagName(MathMLNames::annotationTag) || hasTagName(MathMLNames::moTag)) 97 if (hasTagName(MathMLNames::annotationTag)) 99 98 return child.isTextNode(); 100 99
Note: See TracChangeset
for help on using the changeset viewer.