[Patch - CLA] File transfers (iframe.js) fail during second file upload

[guest]

When uploading a file using iframeIO (iframe.js) and Dojo 1.0.1 the first submission succeeds but the second submission fails.

This is recreatable quite easily using embedded IE7 but I have seen reports by phiggins and others that they have experienced it as well.

Excerpt of dojo.:

	var bindArgs = {
			url: myUrl,
			form: dojo.byId(formId),
			method: "post",
			handleAs: 'json',
			preventCache: true,			
			content: {
				increment: ctr++,
				name: strFormFileSource,
				filename: myFileName
			}
		}
  var request = dojo.io.iframe.send(bindArgs);

The second request results in an error. Debugging (VS) shows that in iframe.doc() the passed iframeNode does not have access to it's contentWindow:

iframeNode.contentWindow
Object doesn't support this property or method

However, accessing this node via a manual fetch to the iframe does give us access to the iframe.contentWindow:

(document.getElementsByTagName("iframe")[iframeNode.name].contentWindow.document)

The patch which addresses this is attached (and below). It has been used for 2 months and no problems in IE reported by those using it.

CLA on file for "Matthew Knapp"

Patch Generated by SVN:

Index: iframe.js
===================================================================
--- iframe.js	(revision 11817)
+++ iframe.js	(working copy)
@@ -99,7 +99,8 @@
 		//summary: Returns the document object associated with the iframe DOM Node argument.
 		var doc = iframeNode.contentDocument || // W3
 			(
-				(iframeNode.contentWindow)&&(iframeNode.contentWindow.document)
+				((iframeNode.name)&&(iframeNode.document)&&(document.getElementsByTagName("iframe")[iframeNode.name].contentWindow)
+				&&(document.getElementsByTagName("iframe")[iframeNode.name].contentWindow.document))
 			) ||  // IE
 			(
 				(iframeNode.name)&&(document.frames[iframeNode.name])&&

[guest]

Patch for iframe.js

[alex]

(In [12513]) applying patch from Mike Knapp for a really janktastic IE issue when using MSHTML in embedded environments. Fixes #5904. !strict