The default bug view has changed. See this FAQ.

Status

Core Graveyard
XForms
RESOLVED FIXED
9 years ago
8 months ago

People

(Reporter: Doron Rosenberg (IBM), Assigned: Doron Rosenberg (IBM))

Tracking

({fixed1.8.1.12})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

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

Code was removed
Attachment #295829 - Flags: review?(aaronr)

Comment 1

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

Updated

9 years ago
Attachment #295829 - Flags: review?(aaronr) → review+
(Assignee)

Comment 2

9 years ago
and more importantly, that the code was there and then removed by a mega patch somehow
(Assignee)

Updated

9 years ago
Attachment #295829 - Flags: review?(Olli.Pettay)

Comment 3

9 years ago
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.
Attachment #295829 - Flags: review?(Olli.Pettay) → review+

Comment 4

9 years ago
fixed indentation and checked into trunk for Doron.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch

Comment 5

9 years ago
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.
Keywords: fixed1.8.1.12
Whiteboard: xf-to-branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.