broulik created this revision.
broulik added reviewers: Plasma, fvogt.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  This implements `CanSetFullscreen` to return `true` when it's a video and 
lets the `Fullscreen` property reflect and change the fullscreen state of the 
player.
  
  BUG: 408059

TEST PLAN
  YouTube sets the entire website as fullscreen and gets confused when just 
fullscreening the player but it works fine in e.g. Vimeo.
  If the video autostarted it refuses to fullscreen it as there hasn't benn any 
user interaction, otherwise works fine.

REPOSITORY
  R856 Plasma Browser Integration

REVISION DETAIL
  https://phabricator.kde.org/D21554

AFFECTED FILES
  extension/content-script.js
  extension/extension.js
  host/mprisplugin.cpp
  host/mprisplugin.h

To: broulik, #plasma, fvogt
Cc: plasma-devel, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to