mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-12-03 02:25:34 +00:00
bug 904104 fix window.close usage from chat and social panels, r=felipc
This commit is contained in:
parent
1962adcdc7
commit
fd238172c4
@ -66,6 +66,11 @@ function injectController(doc, topic, data) {
|
||||
return;
|
||||
}
|
||||
|
||||
// we always handle window.close on social content, even if they are not
|
||||
// "enabled". "enabled" is about the worker state and a provider may
|
||||
// still be in e.g. the share panel without having their worker enabled.
|
||||
handleWindowClose(window);
|
||||
|
||||
SocialService.getProvider(doc.nodePrincipal.origin, function(provider) {
|
||||
if (provider && provider.enabled) {
|
||||
attachToWindow(provider, window);
|
||||
@ -214,7 +219,9 @@ function attachToWindow(provider, targetWindow) {
|
||||
schedule(function () { port.close(); });
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
function handleWindowClose(targetWindow) {
|
||||
// We allow window.close() to close the panel, so add an event handler for
|
||||
// this, then cancel the event (so the window itself doesn't die) and
|
||||
// close the panel instead.
|
||||
|
Loading…
Reference in New Issue
Block a user