Changeset 14511

Timestamp:

07/18/08 13:42:36 (6 months ago)

Author:

BradNeuberg

Message:

Fixes #7236

Files:

• dojox/trunk/flash/_base.js (modified) (8 diffs)

dojox/trunk/flash/_base.js

@@ -102 +102 @@
         
         setSwf: function(/* String */ url, /* boolean? */ visible){
+                //console.debug("setSwf, url="+url);
                 // summary: Sets the SWF files and versions we are using.
                 // url: String
@@ -394 +395 @@
                 //      information. Optional value; defaults to false.
                 
-                // determine our container div's styling
-                var containerStyle = "";
-                containerStyle += ("width: " + this.width + "px; ");
-                containerStyle += ("height: " + this.height + "px; ");
-                if(!this._visible){
-                        containerStyle += "position: absolute; z-index: 10000; top: -1000px; left: -1000px; "; //FIXME: avoid horizontal positioning off-page for BiDi
-                }
-
                 // figure out the SWF file to get and how to write out the correct HTML
                 // for this Flash version
@@ -479 +472 @@
                         var div = document.createElement("div");
                         div.id = this.id + "Container";
-                        div.style = containerStyle;
+                        
+                        div.style.width = this.width + "px";
+                        div.style.height = this.height + "px";
+                        if(!this._visible){
+                                div.style.position = "absolute";
+                                div.style.zIndex = "10000";
+                                div.style.top = "-1000px";
+                                div.style.left = "-1000px";
+                                //FIXME: avoid horizontal positioning off-page for BiDi
+                        }
+
                         div.innerHTML = objectHTML;
-        
+
                         var body = document.getElementsByTagName("body");
                         if(!body || !body.length){
@@ -508 +511 @@
         
         setVisible: function(/* Boolean */ visible){
-          //console.debug("setVisible, visible="+visible);
+                //console.debug("setVisible, visible="+visible);
                 
                 // summary: Sets the visibility of this Flash object.           
@@ -556 +559 @@
         // JavaScript, using Flash 8's ExternalInterface. 
         _addExternalInterfaceCallback: function(methodName){
+                //console.debug("addExternalInterfaceCallback, methodName="+methodName);
                 var wrapperCall = dojo.hitch(this, function(){
                         // some browsers don't like us changing values in the 'arguments' array, so
@@ -576 +580 @@
         // present even in Flash 9.
         _encodeData: function(data){
+                //console.debug("encodeData, data=", data);
                 if(!data || typeof data != "string"){
                         return data;
@@ -602 +607 @@
         // present even in Flash 9.
         _decodeData: function(data){
+                //console.debug("decodeData, data=", data);
                 // wierdly enough, Flash sometimes returns the result as an
                 // 'object' that is actually an array, rather than as a String;
@@ -634 +640 @@
         // create on dojox.flash.comm.
         _execFlash: function(methodName, methodArgs){
+                //console.debug("execFlash, methodName="+methodName+", methodArgs=", methodArgs);
                 var plugin = dojox.flash.obj.get();
                 methodArgs = (methodArgs) ? methodArgs : [];