Comment 3 for bug 588842

The fix as committed might not be 100% correct (although it does fix the bug). In my quick patch I just assumed the goal of the name=XXX ord=XXX pairs was to add metadata to the option tag to use in JavaScript. If, in fact, the goal is only to provide some extra information to developer in "view source" then perhaps name=XXX and ord=XXX should remain as they were and the only change should be it -> it.name() for name=it and value=it. See my previous comment also regarding message string/lookup. I think it would be nicer behavior for that line also to change to it.name().