REGRESSION(r64712): Safari fails to show most of the contents of http://www.milliyet.com.tr

[Mark Rowe (bdash)]

After <http://trac.webkit.org/changeset/64712>, which enabled the HTML5 tree builder, pages on milliyet.com.tr are displayed as mostly blank.  See <http://www.milliyet.com.tr/2010/08/25/index.html> for an example of one such page.

[Mark Rowe (bdash)]

<rdar://problem/8354793>

[Adam Barth]

Will investigate.

[Adam Barth]

The site is doing a bad User Agent detect.  A few places in the page says:

 if(navigator.userAgent.indexOf("Firefox")==-1 && navigator.userAgent.indexOf("Chrome")==-1) 

If you change that to

 if(navigator.userAgent.indexOf("Firefox")==-1 && navigator.userAgent.indexOf("Safari")==-1) 

everything works great.  :(

[Andy Estes]

Created attachment 80407 [details]
Reduction

Here is a reduction.

The site injects an unclosed object element towards the beginning of the page which causes the remainder of the document to be considered fallback content for the object. The legacy parser behavior was to close the object tag implicitly when its parent div was closed, but the HTML5 parser does not behave this way. Minefield shows the same behavior we do (if you trick it into following Safari's code path). In general, Firefox/Minefield don't suffer from this problem because the site injects an embed tag (which of course can't have fallback content).

[Andy Estes]

and Chrome gets the embed tag too, of course :)

No love for Safari, unfortunately.

[Andy Estes]

milliyet.com.tr is actually quite popular in Turkey (ranked 7th by Alexa) and surprisingly popular globally (ranked #510). We should consider adding a evangelizing the site developer as well as perhaps adding a site-specific hack.

[Adam Barth]

The site appears to work fine now.