D13619: Inject script on relevant video nodes not top of DOM
David Edmundson
noreply at phabricator.kde.org
Tue Jun 19 19:10:55 UTC 2018
davidedmundson created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
davidedmundson requested review of this revision.
REVISION SUMMARY
Turns out we're not the only script that messes with the top of the DOM,
and because we're in a separate context, we're potentially doing it in
parallel.
This moves injection into the found video node
TEST PLAN
Ran media sessions example
REPOSITORY
R856 Plasma Browser Integration
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D13619
AFFECTED FILES
extension/content-script.js
To: davidedmundson
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180619/c5e86e10/attachment.html>
More information about the Plasma-devel
mailing list