Rename ConstantValueComputer_InitializerCloner.

Rename ConstantValueComputer_InitializerCloner.

In order to migrate constant evaluation to the new task model, this
class will need to be used from outside of ConstantValueComputer, and
it will need to be used on AST's for all constant values; not just
those for initializers.  So the old name really doesn't make any
sense.

R=brianwilkerson@google.com

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

[Paul Berry, 2015-04-01 16:36:01]

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

[Brian Wilkerson, 2015-04-01 16:39:21]

LGTM

https://codereview.chromium.org/1055543003/diff/1/pkg/analyzer/lib/src/genera...
File pkg/analyzer/lib/src/generated/constant.dart (right):

https://codereview.chromium.org/1055543003/diff/1/pkg/analyzer/lib/src/genera...
pkg/analyzer/lib/src/generated/constant.dart:151: // TODO(brianwilkerson)
Investigate replacing uses of this class with uses of
I think we can remove the TODO: having one visitor is more efficient that having
two.

[Paul Berry, 2015-04-01 18:12:07]

https://codereview.chromium.org/1055543003/diff/1/pkg/analyzer/lib/src/genera...
File pkg/analyzer/lib/src/generated/constant.dart (right):

https://codereview.chromium.org/1055543003/diff/1/pkg/analyzer/lib/src/genera...
pkg/analyzer/lib/src/generated/constant.dart:151: // TODO(brianwilkerson)
Investigate replacing uses of this class with uses of
On 2015/04/01 16:39:21, Brian Wilkerson wrote:
> I think we can remove the TODO: having one visitor is more efficient that
having
> two.

Done.

[Paul Berry, 2015-04-01 18:13:13]

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