D16435: Also catch new Audio() elements

2018-11-26 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R856:c947306c0c23: Also catch new Audio() elements (authored by broulik). REPOSITORY R856 Plasma Browser Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16435?vs=44243=46247

D16435: Also catch new Audio() elements

2018-11-23 Thread Fabian Vogt
fvogt accepted this revision. fvogt added a comment. This revision is now accepted and ready to land. LGTM, but I have no idea about this part of JS. REPOSITORY R856 Plasma Browser Integration REVISION DETAIL https://phabricator.kde.org/D16435 To: broulik, davidedmundson, fvogt Cc:

D16435: Also catch new Audio() elements

2018-10-26 Thread Kai Uwe Broulik
broulik created this revision. broulik added reviewers: davidedmundson, fvogt. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision. REVISION SUMMARY Also catches audio players created using the proper `new Audio()` syntax rather