From 86e9afbc069f51297a5028f9cd1c2cf72ec0ed1a Mon Sep 17 00:00:00 2001
From: Markus Stange <mstange@themasta.com>
Date: Thu, 5 Mar 2009 16:44:24 +0100
Subject: [PATCH] Fixing test_bug478438; only print exceptions when something
 fails so that the Tinderbox error parser doesn't get confused

---
 js/src/xpconnect/tests/mochitest/test_bug478438.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/js/src/xpconnect/tests/mochitest/test_bug478438.html b/js/src/xpconnect/tests/mochitest/test_bug478438.html
index 85797329bf9..0fcf0b935ec 100644
--- a/js/src/xpconnect/tests/mochitest/test_bug478438.html
+++ b/js/src/xpconnect/tests/mochitest/test_bug478438.html
@@ -9,8 +9,8 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=478438
   <script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
   <script type="application/javascript;version=1.8">
-    function fail(s) { ok(false, s) }
-    function pass(s) { ok(true, s) }
+    function fail(s, e) { ok(false, s + e) }
+    function pass(s, e) { ok(true, s) }
     (pass.opposite = fail).opposite = pass;
 
     function test() {
@@ -21,8 +21,8 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=478438
       var iwin = (new XPCNativeWrapper(document)).getElementById("f").contentWindow;
 
       function testOne(fn, onAllow, infinitive) {
-        try { fn(); onAllow("able " + infinitive) }
-        catch (e) { onAllow.opposite("unable " + infinitive + ": " + e) }
+        try { fn(); onAllow("able " + infinitive, "") }
+        catch (e) { onAllow.opposite("unable " + infinitive, ": " + e) }
       }
 
       testOne(function() iwin.focus, pass,