wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 918695 - Only instanciate one singleton target per app r=paul

Browse Source
This commit is contained in:
Alexandre Poirot 2013-10-21 10:35:00 +01:00
parent 2fd5d5a66a
commit 3eb564654a
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
toolkit/devtools/apps/app-actor-front.js
View File

@ -15,6 +15,8 @@ const PR_TRUNCATE = 0x20;
const CHUNK_SIZE = 10000; const CHUNK_SIZE = 10000;
const appTargets = new Map();
function addDirToZip(writer, dir, basePath) { function addDirToZip(writer, dir, basePath) {
let files = dir.directoryEntries; let files = dir.directoryEntries;
@ -212,6 +214,13 @@ function installHosted(client, webappsActor, appId, metadata, manifest) {
exports.installHosted = installHosted; exports.installHosted = installHosted;
function getTargetForApp(client, webappsActor, manifestURL) { function getTargetForApp(client, webappsActor, manifestURL) {
// Ensure always returning the exact same JS object for a target
// of the same app in order to show only one toolbox per app and
// avoid re-creating lot of objects twice.
let existingTarget = appTargets.get(manifestURL);
if (existingTarget)
return promise.resolve(existingTarget);
let deferred = promise.defer(); let deferred = promise.defer();
let request = { let request = {
to: webappsActor, to: webappsActor,
@ -230,6 +239,10 @@ function getTargetForApp(client, webappsActor, manifestURL) {
devtools.TargetFactory.forRemoteTab(options).then((target) => { devtools.TargetFactory.forRemoteTab(options).then((target) => {
target.isApp = true; target.isApp = true;
appTargets.set(manifestURL, target);
target.on("close", () => {
appTargets.delete(manifestURL);
});
deferred.resolve(target) deferred.resolve(target)
}, (error) => { }, (error) => {
deferred.reject(error); deferred.reject(error);