New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 610363 link

Starred by 1 user

Issue metadata

Status: Archived
Owner: ----
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Media document uses <video> for wav files

Project Member Reported by mlamouri@chromium.org, May 9 2016

Issue description

I believe WAV files are audio but for some reasons, we use a <video> element for these when opened as a document.

For example, you can open this link in Chrome:
http://themushroomkingdom.net/sounds/wav/smw/smw_coin.wav

I was able to reproduce this on Beta (M51) and trunk (M52).
 
Yeah, I looked at this ages ago, it's roughly impossible to fix unless we decide we want to go with extension based mappings (we shouldn't). If you ever choose audio and guess wrong you get no video, but if you choose video and guess wrong you still get audio controls -- so video is the safest bet.

What particularly is the issue here?
What triggered me filing the issue is that right clicking on the player shows "Download the video file" or something similar instead of "Download the audio file". That was confusing.

Maybe we could simply change the text so it is agnostic of the type of media?
sgtm, that's much easier to fix :)
Cc: rolfe@chromium.org renganat...@chromium.org
Labels: Hotlist-Polish
+rolfe@, +renganathan@

Rebecca, would you be fine to have context menu items using "media" instead of "video" or "audio" when we don't know if the <video> might be used for an audio file. In practice, that would be for MediaDocument that has a <video> because it can be used for audio files.

Comment 5 by rolfe@chromium.org, May 13 2016

Cc: talo@chromium.org k...@chromium.org rachelis@chromium.org sidv@chromium.org
It could just be "download" if there's no need to disambiguate (unlike for linked images - where we have options for the link and the image itself) But "download media" is also fine with me for when we don't know.

Adding the downloads/some Chrome on Android people (+sidv & +rachelis with +talo +ktam as FYI) if they are going in a different direction.
No different direction here. :)
Project Member

Comment 7 by sheriffbot@chromium.org, Jun 1 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 8 by sheriffbot@chromium.org, Jul 10 2016

Labels: -M-53 MovedFrom-53
This issue has been moved once and is lower than Pri-1. Removing the milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Needs-BlinkMediaTriage
Components: -Blink>Media Blink>Media>Controls
Labels: -MovedFrom-52 -MovedFrom-53 -Needs-BlinkMediaTriage
Labels: M-58
There's some cruddy logic to be unmangled here. Marking as M-58, given it is a polish milestone. 
Cc: foolip@chromium.org
Cc: -phil...@opera.com

Comment 14 by rolfe@chromium.org, Mar 29 2017

Cc: -rolfe@chromium.org
Status: Archived (was: Available)
Think this will be made redundant by the context menu redesign:
https://bugs.chromium.org/p/chromium/issues/detail?id=655359

Sign in to add a comment