Changeset 15089

Timestamp:

03/10/10 17:32:28 (5 months ago)

Author:

sys

Message:

desk.openRefUri : sécurisation en cas d'appel par d'autres frames et GC intempestifs.

Location:

trunk/Xul_Wsp/content/scenariWsp

Files:

• desks/wspDesk/wspDesk.js (modified) (4 diffs)
• windows/itemSelector/selItem.js (modified) (3 diffs)

trunk/Xul_Wsp/content/scenariWsp/desks/wspDesk/wspDesk.js

@@ -118 +118 @@
                                         Components.classes["@scenari.com/sharedres;1"].getService(Components.interfaces.scISharedRes).reset();
                                         if(system.isMac){
-                                                /* Horrid hack to work around xulrunner bug under MacOSX: The menubar will crash xulrunner after a window.location.reload() */
+                                                // Horrid hack to work around xulrunner bug under MacOSX: The menubar will crash xulrunner after a window.location.reload()
                                                 system.appRestart(true);
                                         } else {
@@ -325 +325 @@
         
         /** 
-         * @Api Desks avec contenus. Enregistre tout ce qui est dirty concernant le wsp pCdWsp ou
+         * @Api Desks avec ContentViewController. Enregistre tout ce qui est dirty concernant le wsp pCdWsp ou
          * tout wsp confondu si pCdWsp n'est pas renseigné.
          */
@@ -333 +333 @@
         
         /**
-         * @Api Desks avec contenus. Permet d'accéder à une uri quelconque.
+         * @Api Desks avec ContentViewController. Permet d'accéder à une uri quelconque.
          */
         openRefUri : function(pRefUri, pOptions) {
-                try {
-                        if(!pRefUri) return;
-                        if(! src.isSrcId(pRefUri)) desk.getController("ContentViewController").openSrcUri(pRefUri, pOptions);
-                        else {
-                                function cbGetSrcUri(pShortDesc, pCdError) {
-                                        if(pShortDesc) desk.getController("ContentViewController").openSrcUri(pShortDesc.srcUri, pOptions);
+                if(!pRefUri) return;
+                //Si l'appel vient d'une autre frame (mainview), les appels asynchrones qui seront effectués pour 
+                //effectuer cet openRefUri peuvent échouer alléatoirement dû à des GC effectués dans l'autre frame.
+                //Le passage par un timer permet de réassocier les objets qui seront créés à cette frame.
+                //Bug peut-etre lié à notre utilisation de l'ancien mode de sécurité (xpcnativewrappers=no dans
+                //les manifests chrome). A réévaluer quand on aurra purgé xpcnativewrappers=no.
+                system.setTimer({notify:function(pTimer){
+                        try {
+                                if(! src.isSrcId(pRefUri)) desk.getController("ContentViewController").openSrcUri(pRefUri, pOptions);
+                                else {
+                                        function cbGetSrcUri(pShortDesc, pCdError) {
+                                                if(pShortDesc) desk.getController("ContentViewController").openSrcUri(pShortDesc.srcUri, pOptions);
+                                        }
+                                        var vUriObjectWsp = new UriObjectWsp(this.getWsp().getCode(), this.getWsp().getDataProvider());
+                                        vUriObjectWsp.loadShortDesc(pRefUri, cbGetSrcUri);
                                 }
-                                var vUriObjectWsp = new UriObjectWsp(this.getWsp().getCode(), this.getWsp().getDataProvider());
-                                vUriObjectWsp.loadShortDesc(pRefUri, cbGetSrcUri);
-                        }
-                }catch(e){log.debug("desk.openRefUri failed for '"+pRefUri+"': "+e);}
+                        }catch(e){log.debug("desk.openRefUri failed for '"+pRefUri+"': "+e);}
+                }}, 0);
         },
         
@@ -403 +410 @@
         
         /** 
-         * Transfert une requete vers la ContentView courante.
+         * @Api Desks avec ContentViewController. Transfert une requete vers la ContentView courante.
          * 
          * @param pKeepIfViewUnvailable Si true et si la view n'est pas encore chargée, la requête sera placée en attente.

trunk/Xul_Wsp/content/scenariWsp/windows/itemSelector/selItem.js

@@ -208 +208 @@
         fController : {},
         
-        /** @Api Desk avec contenus. Enregistre tout ce qui est dirty concernant le wsp pCdWsp ou
+        /** @Api Desks avec ContentViewController. Enregistre tout ce qui est dirty concernant le wsp pCdWsp ou
          * tout wsp confondu si pCdWsp n'est pas renseigné.
          */
@@ -216 +216 @@
         
         /**
-         * @Api Desks avec contenus. Permet d'accéder à une uri quelconque.
+         * @Api Desks avec ContentViewController. Permet d'accéder à une uri quelconque.
          */
         openRefUri : function(pRefUri, pOptions) {
-                try {
-                        if(!pRefUri) return;
-                        if(! src.isSrcId(pRefUri)) desk.getController("ContentViewController").openSrcUri(pRefUri, pOptions);
-                        else {
-                                function cbGetSrcUri(pShortDesc, pCdError) {
-                                        if(pShortDesc) desk.getController("ContentViewController").openSrcUri(pShortDesc.srcUri, pOptions);
+                if(!pRefUri) return;
+                //Si l'appel vient d'une autre frame (mainview), les appels asynchrones qui seront effectués pour 
+                //effectuer cet openRefUri peuvent échouer alléatoirement dû à des GC effectués dans l'autre frame.
+                //Le passage par un timer permet de réassocier les objets qui seront créés à cette frame.
+                //Bug peut-etre lié à notre utilisation de l'ancien mode de sécurité (xpcnativewrappers=no dans
+                //les manifests chrome). A réévaluer quand on aurra purgé xpcnativewrappers=no.
+                system.setTimer({notify:function(pTimer){
+                        try {
+                                if(! src.isSrcId(pRefUri)) desk.getController("ContentViewController").openSrcUri(pRefUri, pOptions);
+                                else {
+                                        function cbGetSrcUri(pShortDesc, pCdError) {
+                                                if(pShortDesc) desk.getController("ContentViewController").openSrcUri(pShortDesc.srcUri, pOptions);
+                                        }
+                                        var vUriObjectWsp = new UriObjectWsp(this.getWsp().getCode(), this.getWsp().getDataProvider());
+                                        vUriObjectWsp.loadShortDesc(pRefUri, cbGetSrcUri);
                                 }
-                                var vUriObjectWsp = new UriObjectWsp(this.getWsp().getCode(), this.getWsp().getDataProvider());
-                                vUriObjectWsp.loadShortDesc(pRefUri, cbGetSrcUri);
-                        }
-                }catch(e){log.debug("desk.openRefUri failed for '"+pRefUri+"': "+e);}
+                        }catch(e){log.debug("desk.openRefUri failed for '"+pRefUri+"': "+e);}
+                }}, 0);
         },
 
@@ -286 +293 @@
         
         /** 
-         * Transfert une requete vers la ContentView courante.
+         * @Api Desks avec ContentViewController. Transfert une requete vers la ContentView courante.
          * 
          * @param pKeepIfViewUnvailable Si true et si la view n'est pas encore chargée, la requête sera placée en attente.