Open Bug 1500507 Opened 7 years ago Updated 3 years ago

Don't always construct videocontrols UA Widget in Fennec/GeckoView

Categories

(Core :: Audio/Video: Playback, enhancement, P5)

enhancement

Tracking

()

Tracking Status
firefox64 --- affected

People

(Reporter: timdream, Unassigned)

References

Details

In bug 1431255 it was done in HTMLMediaElement::BindToTree that we'll always dispatch UAWidgetBindToTree event on Fennec, because I thought we will need the UI there. We actually only need the UI if the <video> is auto-blocked && needs controls. It's not hard if we instead construct UA Widget only until the video is auto-blocked. This will make <video> in Fennec as fast as on the desktop.
Severity: normal → enhancement
Priority: -- → P5
Summary: Don't always construct UA Widget in Fennec → Don't always construct videocontrols UA Widget in Fennec
Summary: Don't always construct videocontrols UA Widget in Fennec → Don't always construct videocontrols UA Widget in Fennec/GeckoView

FYI the event is being used here

https://searchfox.org/mozilla-central/rev/c07aaf12f13037c1f5a343d31f8291549e57373f/mobile/android/chrome/geckoview/GeckoViewMediaChild.js#31

so we can't really stop dispatching the event on video element w/o controls without finding a replacement for this one.

See Also: → 1498246
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.