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