fvogt updated this revision to Diff 17287. fvogt added a comment.
braces + const A hasOwnProperty check is not needed here, that's accounted for by the if (!favIconUrl) below REPOSITORY R856 Plasma Browser Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6717?vs=16759&id=17287 BRANCH favicons (branched from master) REVISION DETAIL https://phabricator.kde.org/D6717 AFFECTED FILES extension/extension.js extension/manifest.json tabsrunner/tabsrunner.cpp To: fvogt, #plasma, broulik, davidedmundson Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
