Class.js tweak

[tschaub]

The OpenLayers.Class function refers to arguments[i] a number of times. We can create a variable referring to this argument for clarity. As a bonus, we may also treat an obscure and very incompletely described issue using the OpenLayers.Class function in IE8 (we know this works in general, but an IRC report from dheinbach says that to support his use, he had to explicitly add parenthesis when constructing an instance of the parent).

I'm not in favor of voodoo coding in general. But I think this change adds clarity anyway.

[tschaub]

fewer references to arguments[i]

[tschaub]

Tests pass. I think this nearly qualifies for our "trivial change" exception, but I'll await a review.

[elemoine]

It looks good. Please commit.

I'd also love to have tests showing in what cases the problem triggers.

[tschaub]

(In [10642]) Creating a Type variable that refers to arguments[i]. Including parentheses when calling the constructor. r=elemoine (closes #2791)