Bug 39048 – REGRESSION (r55802): XML errors on Google maps ("Entity 'nbsp' not defined”) with JavaScript disabled

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Simon Fraser (smfr)]

Created attachment 55947 [details]
Screenshot

When I go to http://maps.google.com/ with a current build, I see an XML error (see screenshot)

[Simon Fraser (smfr)]

I see this in the r59204 nightly.

[Simon Fraser (smfr)]

I had JS turned off, and in that situation google redirect to their mobile site, which is XML errors.

[Simon Fraser (smfr)]

Mark Rowe points out that there is a behavior change, which is that we didn't used to see an XML error with nbsp on this page, and now we do.

[Mark Rowe (bdash)]

This appears to have been introduced by <http://trac.webkit.org/changeset/55802>.

[Mark Rowe (bdash)]

Prior to r55802 we were parsing the document as XHTML.  The change within externalSubsetHandler in XMLTokenizerLibxml2.cpp made it so that we no longer treat the XHTML Mobile DTD as indicating XHTML unless the ENABLE(XHTMLMP) define is set.

[Mark Rowe (bdash)]

<rdar://problem/7978384>

[Alexey Proskuryakov]

Created attachment 56136 [details]
proposed fix

[Alexey Proskuryakov]

I don't really know what it means to have HTMLMP enabled. Looks like we handle this particular example of XHTMLMP just fine even when it's disabled.

[George Staikos]

This seems like a site bug.  The change above will cause xhtmlmp to be parsed with the xhtml parser but xhtmlmp isn't always going to work in that case.  If it did, we wouldn't need to have xhtmlmp support in the first place because it would be a no-op.

[Alexey Proskuryakov]

My understanding is that:
1) This patch only affects entity substitution, nothing else.
2) Firefox doesn't support XHTMLMP, but this page works in it.
3) This patch doesn't change behavior when XHTMLMP is enabled, it only affects builds with XHTMLMP disabled. It reverts this aspect to what it was in Safari 4.0.5 - compatible with Firefox and Google Maps.

[Alexey Proskuryakov]

Committed <http://trac.webkit.org/changeset/59556>.

George: I'm interested in further discussing consequences of this change, but wanted to get the maps.google.com issue resolved ASAP.