188376 - Chimera claims to accept MNG but doesn't

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[Henri Sivonen (:hsivonen)]

Build ID: 2003010804

Reproducible: Always.

Steps to reproduce:
1) Find out what Accept header Chimera sends eg. by loading
http://www.delorie.com:81/some/url.html
2) Observe that Chimera does not support MNG. See bug 188116.

Actual results:
Chimera list video/x-mng in the Accept header even though Chimera does not
support MNG (bug 188116).

Expected results:
Until bug 188116 is fixed, expected video/x-mng not to appear in the Accept header.

Additional information:
This bug breaks MNG content negotiation, hence the major severity.

Comment 1

[Henri Sivonen (:hsivonen)]

Attachment: Patch for removing video/x-mng from Accept

Comment 2

[saari (gone)]

->sfraser for patch eval

Comment 3

[Stephane Moureau]

Duplicate of bug 188116 (patch here)

Comment 4

[Henri Sivonen (:hsivonen)]

Any chance of getting the fix in? A patch exists and leaving this bug unfixed
effectively makes content negotiation of MNG useless.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 111069 [details] [diff] [review]
Patch for removing video/x-mng from Accept

Simon, could you please review this patch and land it in Camino?

Comment 6

[Simon Fraser [no longer active]]

Camino is on the trunk now. Land this for Mozilla, and Camino will get it.

Comment 7

[Boris Zbarsky [:bzbarsky]]

This just landed on trunk (see bug 195280).  But up till that point, the whole
point was that MOZILLA supported MNG, but CAMINO did not.  Both advertised
support, though.

Feel free to mark fixed unless there are other things that Camino compiles out
support for but still sends Accept headers for.

Comment 8

[Ludovic Hirlimann [:Usul]]

video/x-mng does not appears on the Accept line with 2003090102 on 10.2.6.

Comment 9

[Asa Dotzler [:asa]]

Comment on attachment 111069 [details] [diff] [review]
Patch for removing video/x-mng from Accept

We have a new Camino request flag which can be set multiple times for a patch.
Moving old review requests to the new flag. Sorry for the spam.

Comment 10

[David Haas (gone, not reading mail)]

this patch already landed.  taking advice from comment 7 and closing bug.