Bug 28211 – Can't click and drag the media controller thumb

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Simon Fraser (smfr)]

With TOT, the media slider doesn't move with the cursor when I mouse down and drag it over the controller. The media slider only updates it's position after I mouse up after I drag it.

This is a regression on TOT since this doesn't happen on stock SL 10A342. This regression was caused by http://trac.webkit.org/changeset/46915 since it doesn't happen with previous checkin 46914.

* STEPS TO REPRODUCE
1. Launch a nightly webkit build and go to http://www.mirageband.tv/Night%20and%20Day.mp3
2. Wait for entire file to load then pause it
3. Mouse down directly on the slider and drag the mouse to another section on the controller. Notice the slider doesn't move until you mouse up.

[Simon Fraser (smfr)]

<rdar://problem/7135838>

[Hin-Chung Lam]

Created attachment 34647 [details]
patch

[Hin-Chung Lam]

The cause of the regression is that r46915 rejects non mouse actions and mouse actions not from left button. This logic caused trouble because it discards a lot of mouse move events during drag.

[Simon Fraser (smfr)]

Comment on attachment 34647 [details]
patch

Patch looks reasonable but you can make a testcase; in DumpRenderTree, there is an eventSender API that you can use to synthesize mouse events. I think it's important to add a testcase so that this doesn't regress again.

[Hin-Chung Lam]

OK, I'll provide a test case with fix shortly.

[Hin-Chung Lam]

Created attachment 34703 [details]
patch + test

[Simon Fraser (smfr)]

Note that layout tests that use eventSender make assumptions about locations of buttons on the media controller that may not hold across platorms.

[Eric Carlson]

https://bugs.webkit.org/show_bug.cgi?id=28220 will fix the problem Simon mentioned, but for now you may need to land platform specific test results.

[Eric Seidel (no email)]

Comment on attachment 34703 [details]
patch + test

Clearing flags on attachment: 34703

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	A	LayoutTests/media/controls-drag-timebar-expected.txt
	A	LayoutTests/media/controls-drag-timebar.html
	M	WebCore/ChangeLog
	M	WebCore/rendering/MediaControlElements.cpp
Committed r47171
	M	WebCore/ChangeLog
	M	WebCore/rendering/MediaControlElements.cpp
	A	LayoutTests/media/controls-drag-timebar-expected.txt
	A	LayoutTests/media/controls-drag-timebar.html
	M	LayoutTests/ChangeLog
r47171 = 5d54d4d1aa0dd34b843952ff291dcdbf59c39a24 (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/47171

[Eric Seidel (no email)]

All reviewed patches have been landed.  Closing bug.

[Dmitry Titov]

It turned out that expected.txt is different for windows so landed http://trac.webkit.org/changeset/47174 to split the results.