Currently, if ENABLE(XHTMLMP) == true, then the ACCEPT header for network request is hardcoded to include mobile content types. This should be runtime configurable as just because WebKit has support for XHTMLMP in a particular build does not mean that it should always be used.
Created attachment 44705 [details] Adds a new setting Adds a new runtime setting for enable/disable of xhtmlmp content.
style-queue ran check-webkit-style on attachment 44705 [details] without any errors.
Comment on attachment 44705 [details] Adds a new setting This should have a test, no? We could test this with things like layoutTestController.overridePreference()
You'd need a server configured to respond to the ACCEPT header. We don't have that.
Hum... I take it our Apache2 setup doesn't do that?
(In reply to comment #5) > Hum... I take it our Apache2 setup doesn't do that? I'll take a look and see.
Thank you.
Comment on attachment 44705 [details] Adds a new setting There seems to be some confusion (not sure if it's in the bug, in the patch, or just in my head) about what this change does. If it's only about changing Accept header, then the setting should not be called XHTMLMPEnabled. And if it disables XHTMLMP support as a whole, then bug and description should say so. Style nit - setXHTMLMPEnabled is fine, but XHTMLMPEnabled should be xhtmlmpEnabled, or maybe xhtmlMPEnabled.
Comment on attachment 44705 [details] Adds a new setting It sounds like there a number of issues in the bug that need to be addressed (or responded to) before this can be reviewed, so I'm marking this as r- for now to move this out of the review queue. Also, please add a link to the bug in the changelog.