103893 2012-12-03 07:40:52 -0800 [Gstreamer] changing MedialElement.src resets volume and mute 2013-02-12 01:46:49 -0800 1 1 1 Unclassified WebKit Media 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 jussi.kukkonen webkit-unassigned jussi.kukkonen pnormand zan oldest_to_newest 781439 0 jussi.kukkonen 2012-12-03 07:40:52 -0800 Bug 96912 updated the MediaElement volume and mute to more accurately match reality. This causes media/video-volume.html fail: When the element gets a new source url, the volume and mute of the element get updated (once state reaches PAUSED) from the playbin. Previously the volume and mute were only updated when playback started so the test did not show the problem. We may want to try to make sure that MediaElement volume does not change when we change the src. 829549 1 zan 2013-02-11 03:24:19 -0800 The media/video-volume.html layout test (of which the failure expectation is pointing to this bug) was fixed with r142005. http://trac.webkit.org/changeset/142005 It was the only failure pointing to this bug on the GTK port. The test is now passing on EFL as well. Can we close this bug? http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&showAllRuns=true&tests=media%2Fvideo-volume.html 829645 2 pnormand 2013-02-11 07:05:31 -0800 (In reply to comment #1) > The media/video-volume.html layout test (of which the failure expectation is pointing to this bug) was fixed with r142005. > http://trac.webkit.org/changeset/142005 > > It was the only failure pointing to this bug on the GTK port. The test is now passing on EFL as well. Can we close this bug? > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&showAllRuns=true&tests=media%2Fvideo-volume.html One change that patch did was connecting to notify::volume and mute before updating the playbin mute property. Are you ok with that Jussi? I'd be fine closing this bug, FWIW :) 830705 3 jussi.kukkonen 2013-02-12 01:46:49 -0800 (In reply to comment #2) > > It was the only failure pointing to this bug on the GTK port. The test is now passing on EFL as well. Can we close this bug? > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&showAllRuns=true&tests=media%2Fvideo-volume.html > > One change that patch did was connecting to notify::volume and mute before updating the playbin mute property. Are you ok with that Jussi? > > I'd be fine closing this bug, FWIW :) That looks just fine to me. Resolving then.