Changeset 15089 for trunk/Xul_Wsp/content/scenariWsp/windows/itemSelector/selItem.js

Timestamp:

03/10/10 17:32:28 (2 years ago)

Author:

sys

Message:

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

File:

• trunk/Xul_Wsp/content/scenariWsp/windows/itemSelector/selItem.js (modified) (3 diffs)

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.