[Webkit-unassigned] [Bug 23452] Add XHTMLMP support to Webkit

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=23452


hyatt at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #27307|review?                     |review-
               Flag|                            |




------- Comment #4 from hyatt at apple.com  2009-02-04 09:38 PDT -------
(From update of attachment 27307)
The code here:

if (m_doc->isXHTMLDocument() && !m_hasDocTypeDeclaration) {

... bails if you don't have a DOCTYPE.  It only asks if you're an XHTML
document, so if XHTMLMP is enabled, you'll start being more strict in your
handling of XHTML documents that aren't also the XHTMLMP MIME type.  Is that
what you really want?

The code in parseDTD and externalSubsetHandler had lines like:

(publicId == QLatin1String("-//WAPFORUM//DTD XHTML Mobile 1.0//EN"))) {

These were unconditionally removed.  Shouldn't those just be left alone but put
inside #if !ENABLE(XHTMLMP)?

The new document method isXHTMLDocument() was added inside #if ENABLE(XHTMLMP).
 It's ok for that to be outside of the #ifdef.

Can you explain what the NOSCRIPT changes are about? Why do you need an actual
element for <noscript> in the XHTMLMP case?

Thanks!


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.