Summary: | Media Element: scrubbing in full-screen mode breaks playback. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jer Noble <jer.noble> | ||||
Component: | Media | Assignee: | Jer Noble <jer.noble> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | Keywords: | InRadar | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Jer Noble
2012-01-05 14:34:41 PST
The media panel and the timeline both attempt to handle the mouseDown event during a scrub event, which results in the timeline never receiving a mouseUp event. Created attachment 121359 [details]
Patch
Comment on attachment 121359 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=121359&action=review > Source/WebCore/html/shadow/MediaControlElements.cpp:243 > + } else if (event->type() == eventNames().mousemoveEvent && m_isBeingDragged) > continueDrag(location); Is there no need to call setDefaultHandled() on the move event? > LayoutTests/ChangeLog:9 > + * fullscreen/video-controls-timeline.html: Added. > + Did you forget to add the test results? (In reply to comment #4) > (From update of attachment 121359 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=121359&action=review > > > Source/WebCore/html/shadow/MediaControlElements.cpp:243 > > + } else if (event->type() == eventNames().mousemoveEvent && m_isBeingDragged) > > continueDrag(location); > > Is there no need to call setDefaultHandled() on the move event? There is no need. The panel has already started capturing events at this point, so none of the other controls will get their default event handler called. > > LayoutTests/ChangeLog:9 > > + * fullscreen/video-controls-timeline.html: Added. > > + > > Did you forget to add the test results? I did; I'll add them before checking in. Committed r104303: <http://trac.webkit.org/changeset/104303> |