Bug 20599 – REGRESSION: Important controls are missing from <video> element UI, when compared to QuickTime

• Status: RESOLVED
• Resolution: FIXED
• Component: Plug-ins

[Iván Garcia]

When I open a mp3, mp4 or mov file in Webkit (r36013, 1 Sept 08) quicktime plugin is in "Kiosk" mode, when I turned off in Quicktime Preference Pane. 

In this mode I can't download the file (I have QuickTime Pro), I can't change volume (only mute), and Behavior is very bad. Music keeps playing after 5 seconds I closed tab. A capture:
http://img131.imageshack.us/img131/2313/imagen1tv6.png

All this not happen in Standard Safari, mp3 load correctly. A capture:
http://img399.imageshack.us/img399/5493/imagen2fa8.png

This is not only with Seeqpod music. For example:
http://movies.apple.com/movies/paramount/tropic_thunder/tropic_thunder-tlr1a_480p.mov
Doesn't load correctly 

An interesting thing is that is file is embebed plugin loads ok. For example:
http://www.midilifecrisis.com/MortillaDemoReelMacs.html

Works perfectly while other file with the same format (m4v):
http://content.mobuzz.tv/es/dosisdiaria/ipod/es_mag_20080724.m4v
doesn't work.

That's all

[Alexey Proskuryakov]

Giving this the full regression treatment (I haven't attempted to reproduce this though).

[Alexey Proskuryakov]

Confirmed with r38032.

[Alexey Proskuryakov]

<rdar://problem/6336097>

[Simon Fraser (smfr)]

The available controls are different because you're comparing the <video> element (which is used when you load a media file directly, via the MediaDocument), with the QuickTime plug-in.

The <video> element controls do not have affordance for saving the movie. This is by design.

[Ruth Fong]

Created attachment 204740 [details]
Patch

[Early Warning System Bot]

Comment on attachment 204740 [details]
Patch

Attachment 204740 [details] did not pass qt-ews (qt):
Output: http://webkit-queues.appspot.com/results/923058

[EFL EWS Bot]

Comment on attachment 204740 [details]
Patch

Attachment 204740 [details] did not pass efl-ews (efl):
Output: http://webkit-queues.appspot.com/results/890138

[Early Warning System Bot]

Comment on attachment 204740 [details]
Patch

Attachment 204740 [details] did not pass qt-wk2-ews (qt-wk2):
Output: http://webkit-queues.appspot.com/results/921059

[EFL EWS Bot]

Comment on attachment 204740 [details]
Patch

Attachment 204740 [details] did not pass efl-wk2-ews (efl-wk2):
Output: http://webkit-queues.appspot.com/results/920060

[Ruth Fong]

Created attachment 204745 [details]
Patch

[Ruth Fong]

Created attachment 204746 [details]
Patch

[EFL EWS Bot]

Comment on attachment 204746 [details]
Patch

Attachment 204746 [details] did not pass efl-ews (efl):
Output: http://webkit-queues.appspot.com/results/906106

[Eric Carlson]

Comment on attachment 204746 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=204746&action=review

> Source/WebCore/page/ContextMenuController.cpp:258
> +        // FIXME: Some day we should be able to do this from within WebCore.

Please file a bug and reference it in the FIXME.

[Ruth Fong]

Created attachment 204843 [details]
Patch

[Ruth Fong]

(In reply to comment #13)
> (From update of attachment 204746 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=204746&action=review
> 
> > Source/WebCore/page/ContextMenuController.cpp:258
> > +        // FIXME: Some day we should be able to do this from within WebCore.
> 
> Please file a bug and reference it in the FIXME.

Bug 117709 now tracks this issue.

[WebKit Commit Bot]

Comment on attachment 204843 [details]
Patch

Clearing flags on attachment: 204843

Committed r151656: <http://trac.webkit.org/changeset/151656>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.