Also catch new Audio() elements
ClosedPublic

Authored by broulik on Oct 26 2018, 9:10 AM.

Details

Summary

Also catches audio players created using the proper new Audio() syntax rather than explicitly creating a DOM element using document.createElement('audio')

BUG: 398261

Test Plan
  • I can now control music on cinemix.radio.net and player.fm

Diff Detail

Repository
R856 Plasma Browser Integration
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Oct 26 2018, 9:10 AM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 26 2018, 9:10 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Oct 26 2018, 9:10 AM
fvogt accepted this revision.Nov 23 2018, 12:57 PM

LGTM, but I have no idea about this part of JS.

This revision is now accepted and ready to land.Nov 23 2018, 12:57 PM
This revision was automatically updated to reflect the committed changes.