D7587: kfileaudiopreview: don't autoplay media files by default

2017-08-29 Thread Elvis Angelaccio
This revision was automatically updated to reflect the committed changes. Closed by commit R320:c1a7f07d4629: kfileaudiopreview: don't autoplay media files by default (authored by elvisangelaccio). REPOSITORY R320 KIO Extras CHANGES SINCE LAST UPDATE

D7587: kfileaudiopreview: don't autoplay media files by default

2017-08-29 Thread Bhushan Shah
bshah accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras BRANCH master REVISION DETAIL https://phabricator.kde.org/D7587 To: elvisangelaccio, #plasma, bshah Cc: broulik, graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai,

D7587: kfileaudiopreview: don't autoplay media files by default

2017-08-29 Thread Kai Uwe Broulik
broulik added a comment. +1 from me also REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D7587 To: elvisangelaccio, #plasma Cc: broulik, graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D7587: kfileaudiopreview: don't autoplay media files by default

2017-08-28 Thread Martin Flöser
graesslin added a comment. obviously +1 from me, but as I'm biased towards security I want someone else to accept it REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D7587 To: elvisangelaccio, #plasma Cc: graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai,

D7587: kfileaudiopreview: don't autoplay media files by default

2017-08-28 Thread Elvis Angelaccio
elvisangelaccio created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Not only it's annoying, but it's also a potential security risk (basically, a variant of [1]). [1]: