Closed Bug 361892 Opened 15 years ago Closed 15 years ago

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

Categories

(Core :: Networking: HTTP, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Biesinger, Assigned: dao)

References

Details

Attachments

(1 file)

Once bug 18333 is fixed, application/xhtml+xml should be bumped back to 1.0 in the Accept header. Compare bug 309438
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
Blocks: 364352
Flags: blocking1.9?
Summary: bump application/xhtml+xml back to 1.0 in the Accept: header → bump application/xhtml+xml back to q=1.0 in the Accept: header when ready
Flags: blocking1.9? → blocking1.9+
Attached patch patchSplinter Review
bug 18333 is fixed
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #253612 - Flags: superreview?(dbaron)
Attachment #253612 - Flags: review?(cbiesinger)
Attachment #253612 - Flags: review?(cbiesinger) → review+
So, shouldn't this patch be checked in as bug 18333 is fixed?
it's still waiting for superreview.
Comment on attachment 253612 [details] [diff] [review]
patch

sr=dbaron
Attachment #253612 - Flags: superreview?(dbaron) → superreview+
Checking in all.js;
/cvsroot/mozilla/modules/libpref/src/init/all.js,v  <--  all.js
new revision: 3.666; previous revision: 3.665
done
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
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.
the absence of q=x.x means 1.0 so it has really been bumped back to q=1.0
http://www.mozilla.org/docs/web-developer/faq.html#accept should be updated as well.

~Grauw
Keywords: dev-doc-needed
Whiteboard: See comment 9 for documentation stuff
Keywords: dev-doc-needed
Whiteboard: See comment 9 for documentation stuff
You need to log in before you can comment on or make changes to this bug.