Prevent sort on immutable lists.

Prevent sort on immutable lists.

Committed: https://code.google.com/p/dart/source/detail?r=5828

[ahe, 2012-03-25 17:38:56]

@lrn: Let's see if I can confuse you again ;-)

[Lasse Reichstein Nielsen, 2012-03-25 18:09:28]

LGTM

[ngeoffray, 2012-03-25 18:14:10]

LGTM

[ahe, 2012-03-26 12:24:08]

I added a temporary interceptor for copyFrom. Please let me know if you have any
concerns over this method.

[ngeoffray, 2012-03-26 12:30:34]

I very much prefer not adding the interceptor. It's not in the List interface,
and I don't think we should add it just because there is a test for it.

Pushing the others to acknowledge and act on the problem like you do on the
other CL is the right thing to do, that's why I would wait for that CL to be
resolved.

[ahe, 2012-03-26 13:03:19]

On 2012/03/26 12:30:34, ngeoffray wrote:
> I very much prefer not adding the interceptor. It's not in the List interface,
> and I don't think we should add it just because there is a test for it.
> 
> Pushing the others to acknowledge and act on the problem like you do on the
> other CL is the right thing to do, that's why I would wait for that CL to be
> resolved.

I have submitted the change but I'll send out a new CL to make a tweak to the
test that should be less controversial than CL 9839101.