dart2js: Do not use 'super' as property name in object literals

[DartBot]

This issue was originally filed by @bgourlie

---

'super' is a keyword reserved for future use per https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Reserved_Words

See https://groups.google.com/a/dartlang.org/forum/?fromgroups#!topic/misc/wVQ6Mi1dWtU%5B1-25%5D for discussion.

[rakudrama]

At the bottom of the linked page it says:

"Reserved word usage

Reserved Words actually only apply to Identifiers (vs. IdentifierNames) . As described in es5.github.com/#A.1, these are all IdentifierNames which do not exclude ReservedWords.

a.import
a["import"]
a = { import: "test" }.

"

The use of 'super' as a PropertyName is permitted, just like 'import' in the last example above.

I think it is only IE8 and earlier that has trouble with 'super' in our generated code. There are many other problems with IE8, which is why we looking at IE9 and forwards.

Given that there are many other reasons the program is not going to work on IE8, it makes sense to leave this as is.
Everyone gets the same symptom of trying to run on IE8.
If we 'fix' this programs running on IE8 will exhibit a much wider range of issues, some of them very time consuming to analyse.

It is possible that the user is using IE9 but in some back-compatability mode. In that case, the html should explicity specify the documenat compatibility mode ( http://msdn.microsoft.com/en-us/library/ie/cc288325(v=vs.85).aspx )

---

Added Area-Dart2JS, WontFix labels.

[floitschG]

Marking as duplicate of issue #4759.
I'm about to put quotes around the 'super' identifier so it works on Android. Stephen is right that this will reveal other bugs in IE8 that we might not be willing to fix.

---

Added Duplicate label.
Marked as being merged into #4759.