Remove CompositeResultDescriptor

Remove CompositeResultDescriptor

R=scheglov@google.com

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

[Brian Wilkerson, 2015-05-07 19:28:10]

brianwilkerson@google.com changed reviewers:
+ scheglov@google.com

[scheglov, 2015-05-07 19:33:50]

LGTM

https://codereview.chromium.org/1136603002/diff/1/pkg/analyzer/lib/src/task/m...
File pkg/analyzer/lib/src/task/model.dart (right):

https://codereview.chromium.org/1136603002/diff/1/pkg/analyzer/lib/src/task/m...
pkg/analyzer/lib/src/task/model.dart:60: ResultDescriptorImpl(this.name,
this.defaultValue,
Can we make this a constant constructor now?

[Brian Wilkerson, 2015-05-07 20:00:30]

https://codereview.chromium.org/1136603002/diff/1/pkg/analyzer/lib/src/task/m...
File pkg/analyzer/lib/src/task/model.dart (right):

https://codereview.chromium.org/1136603002/diff/1/pkg/analyzer/lib/src/task/m...
pkg/analyzer/lib/src/task/model.dart:60: ResultDescriptorImpl(this.name,
this.defaultValue,
> Can we make this a constant constructor now?

I'm not sure it can remain a const constructor. We can make it const later
without it being a breaking change, but it's a breaking change to make it
non-const.

And if two plugins accidentally each create a result with the same name and
value, it's important that those not be the same result.

[Brian Wilkerson, 2015-05-07 20:01:00]

Committed patchset #1 (id:1) manually as 45613 (presubmit successful).