broulik created this revision. broulik added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision.
REVISION SUMMARY So far it was unused and we typically just read the user's preferences rather than checking what modules have been loaded. For hiding unsupported settings (D21113 <https://phabricator.kde.org/D21113>) I am going to query for the status explicitly anyway since at the point the options page comes up the extension is most likely already running. TEST PLAN - Played a video, disabled media controls, reloaded page, media controls were gone, re-enabled them, reloaded page, media controls are there again REPOSITORY R856 Plasma Browser Integration REVISION DETAIL https://phabricator.kde.org/D21785 AFFECTED FILES extension/extension.js host/abstractbrowserplugin.cpp To: broulik, #plasma Cc: plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
