Changeset 183

Timestamp:

05/20/05 06:11:44 (8 years ago)

Author:

gogo

Message:

Fix for #178 dialog.js tried to set events on frames with src outside
the domain causing security exceptions.

Files:

• trunk/dialog.js (modified) (2 diffs)

trunk/dialog.js

@@ -59 +59 @@
         };
         capwin(window);
-        // capture other frames
-        for (var i = 0; i < window.frames.length; capwin(window.frames[i++]));
+        // capture other frames, note the exception trapping, this is because
+  // we are not permitted to add events to frames outside of the current
+  // window's domain.
+        for (var i = 0; i < window.frames.length; i++) {try { capwin(window.frames[i]); } catch(e) { } };
         // make up a function to be called when the Dialog ends.
         Dialog._return = function (val) {
@@ -68 +70 @@
                 relwin(window);
                 // capture other frames
-                for (var i = 0; i < window.frames.length; relwin(window.frames[i++]));
+                for (var i = 0; i < window.frames.length; i++) { try { relwin(window.frames[i]); } catch(e) { } };
                 Dialog._modal = null;
         };