411194 - XForms testsuite fails

Status: RESOLVED FIXED

Description

[Doron Rosenberg (IBM)]

Created attachment 295829 [details] [diff] [review]
the patch to fix the issue

Code was removed

Comment 1

[aaronr]

to be clear, this patch fixes the fact that some of the mozilla/extensions/schema-validation/tests/schema-complex-test.html tests are failing.

Comment 2

[Doron Rosenberg (IBM)]

and more importantly, that the code was there and then removed by a mega patch somehow

Comment 3

[Olli Pettay [:smaug] (vacation-ish until July 30)]

Comment on attachment 295829 [details] [diff] [review]
the patch to fix the issue

>@@ -4761,6 +4770,7 @@
>     rv = elm->GetAttribute(aAttrName, attrValue);
>     NS_ENSURE_SUCCESS(rv, rv);
>   }
>+  }
I assume this and the relevant block get aligned properly when checked in.

Comment 4

[aaronr]

fixed indentation and checked into trunk for Doron.

Comment 5

[aaronr]

checked into 1.8 branch via bug 410239.  Also fixed the schema testcases on 1.8 branch since they still had doron's machine hardcoded in them.