<table><tr><td style="">fvogt requested changes to this revision.<br />fvogt added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21112">View Revision</a></tr></table><br /><div><div><p><tt style="background: #ebebeb; font-size: 13px;">browser.runtime.sendMessage</tt> does not seem to have a callback parameter.</p>

<p>See <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/sendMessage" class="remarkup-link" target="_blank" rel="noreferrer">https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/sendMessage</a>:</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>if two arguments are given:<br />
 the arguments are interpreted as (message, options), and the message is sent internally, if the second argument is any of the following:<br />
 a valid options object (meaning, it is an object which contains only the properties of options that the browser supports)<br />
null<br />
undefined<br />
 otherwise, the arguments are interpreted as (extensionId, message). The message will be sent to the extension identified by extensionId.</p></blockquote></div></div><br /><div><strong>REPOSITORY</strong><div><div>R856 Plasma Browser Integration</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21112">https://phabricator.kde.org/D21112</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, fvogt<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>