472417 - view-source links to audio and video documents should display them regularly, not as text

Status: NEW

(Toolkit :: View Source, defect, P3)

Description

[Curtis Bartley [:cbartley]]

When a "view-source:" URL refers to an audio or video document, it should display the document normally, not as text, since these types of documents don't have a meaningful source representation anyway.

See bug 464339 which addresses similar handling of images.

Comment 1

[Nochum Sossonko [:Natch]]

You can use a method similar to http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js?mark=3683-3690#3674 to detect any non-text mime type.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:Honza, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 6

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Simple test to demonstrate this issue:

// Video file URL
https://media.w3.org/2010/05/sintel/trailer.mp4

// View source of the video file
view-source:https://media.w3.org/2010/05/sintel/trailer.mp4

Opening the view-source URL shows text representing the video cotent.

Note that Tools -> Browser Tools -> Page Source is disabled for video files and the user needs to add view-source protocol manually.

Gijs, this was filed 13 years ago, is this a bug or a feature?

Honza

Comment 7

[:Gijs (out for now; he/him)]

(In reply to Jan Honza Odvarko [:Honza] (always need-info? me) from comment #6)

Simple test to demonstrate this issue:

// Video file URL
https://media.w3.org/2010/05/sintel/trailer.mp4

// View source of the video file
view-source:https://media.w3.org/2010/05/sintel/trailer.mp4

Opening the view-source URL shows text representing the video cotent.

Note that Tools -> Browser Tools -> Page Source is disabled for video files and the user needs to add view-source protocol manually.

The shortcut works before the file finishes loading, fwiw.

Gijs, this was filed 13 years ago, is this a bug or a feature?

It's a bug but fixing it is probably not super high priority. I think we'd need to change https://searchfox.org/mozilla-central/rev/3269d4c928ef0d8310c2f57634e9b6057aa636e9/layout/build/nsContentDLF.cpp#109-112 to also make an exception for media content (not just images).

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

The severity field for this bug is relatively low, S3. However, the bug has 3 duplicates.
:Honza, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Comment 9

[Jan Honza Odvarko [:Honza] (always need-info? me)]

No need to increase the severity/priority in this case.