361892 - bump application/xhtml+xml back to q=1.0 in the Accept: header when ready

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

Once bug 18333 is fixed, application/xhtml+xml should be bumped back to 1.0 in the Accept header. Compare bug 309438

Comment 1

[Aaron Leventhal]

Is this what's causing bug 364352? We have an .htaccess file that is supposed to look to see if application/xml+xhtml is accepted and thus exposes it as XHTML. It's broken in FF3 trunk builds.
Example: http://www.mozilla.org/access/dhtml/maincontent

Comment 2

[Dão Gottwald [:dao]]

Attachment: patch

bug 18333 is fixed

Comment 3

[Kai de Leeuw]

So, shouldn't this patch be checked in as bug 18333 is fixed?

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

it's still waiting for superreview.

Comment 5

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 253612 [details] [diff] [review]
patch

sr=dbaron

Comment 6

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in all.js;
/cvsroot/mozilla/modules/libpref/src/init/all.js,v  <--  all.js
new revision: 3.666; previous revision: 3.665
done

Comment 7

[Steffen Wilberg]

Comment on attachment 253612 [details] [diff] [review]
patch

>-pref("network.http.accept.default", "text/html,application/xhtml+xml;q=0.9,application/xml;q=0.8,*/*;q=0.7");
>+pref("network.http.accept.default", "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8");
So this bumps application/xml from 0.8 to 0.9. It doesn't bump application/xhtml+xml "back to q=1.0" (bug's summary). Neither does it move that to the beginning of the accept header. Should it?

Besides, */* should rather be 0.1 or 0.01 to allow extensions/plugins to add something in between.

Comment 8

[Kai de Leeuw]

the absence of q=x.x means 1.0 so it has really been bumped back to q=1.0

Comment 9

[u81239]

http://www.mozilla.org/docs/web-developer/faq.html#accept should be updated as well.

~Grauw

Comment 10

[Christian :Biesinger (don't email me, ping me on IRC)]

done