701540 - stop test_multi_select.xul hanging on Windows

Status: RESOLVED FIXED

(SeaMonkey :: Download & File Handling, defect)

Description

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch [Checkin: comment 2]

While running the Download Manager chrome mochitests for bug 701051 I found that test_multi_select.xul reproducibly hung for me on Windows (Win7 x64, if that matters), even with the patch from bug 701051 applied.

I compared our version with the one from Toolkit and only ported what was needed to fix it for me. With my patch, all DM chrome tests pass for me on both Windows and Linux.

The -w version comes down to:

   var testObs = {
     observe: function(aSubject, aTopic, aData) {
       if (aTopic != DLMGR_UI_DONE)
         return;

-      var win = aSubject;
+      SimpleTest.waitForFocus(function () { continueTest(aSubject) }, aSubject);
+    }
+  };
+
+  function continueTest(win) {
       var downloadView = win.document.getElementById("downloadTree").view;

       // Default test/check for invocations
       var invokeCount = 0;
       var counter = function() invokeCount++;

       // Accessors for returning a value for various properties
       var getItems = function() downloadView.rowCount;
@@ -217,17 +221,16 @@ function test()
         }
       }

       // We're done!
       win.close();
       obs.removeObserver(testObs, DLMGR_UI_DONE);
       SimpleTest.finish();
     }
-  };
   obs.addObserver(testObs, DLMGR_UI_DONE, false);

Comment 1

[[e-mail]]

Comment on attachment 573648 [details] [diff] [review]
patch [Checkin: comment 2]

Seems reasonable based on -w diff.

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 573648 [details] [diff] [review]
patch [Checkin: comment 2]

http://hg.mozilla.org/comm-central/rev/64edaf61ed76