Remove an unused field from one of the CPS translation classes.

Remove an unused field from one of the CPS translation classes.

BUG=
R=asgerf@google.com

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

[Kevin Millikin (Google), 2015-04-15 14:53:25]

kmillikin@google.com changed reviewers:
+ asgerf@google.com

[asgerf, 2015-04-15 15:01:36]

lgtm

[Kevin Millikin (Google), 2015-04-15 15:13:27]

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

[Kevin Millikin (Google), 2015-04-15 15:14:58]

https://codereview.chromium.org/1087243002/diff/1/pkg/compiler/lib/src/cps_ir...
File pkg/compiler/lib/src/cps_ir/cps_ir_builder.dart (right):

https://codereview.chromium.org/1087243002/diff/1/pkg/compiler/lib/src/cps_ir...
pkg/compiler/lib/src/cps_ir/cps_ir_builder.dart:479: // TODO(johnniwinther):
Make these field final and remove the default values
I guess this comment was referring to the field I just removed.  I'll ask Johnni
to be sure and remove the comment if so.

[Paul Berry, 2015-04-15 16:14:58]

On 2015/04/15 15:14:58, kmillikin wrote:
>
https://codereview.chromium.org/1087243002/diff/1/pkg/compiler/lib/src/cps_ir...
> File pkg/compiler/lib/src/cps_ir/cps_ir_builder.dart (right):
> 
>
https://codereview.chromium.org/1087243002/diff/1/pkg/compiler/lib/src/cps_ir...
> pkg/compiler/lib/src/cps_ir/cps_ir_builder.dart:479: // TODO(johnniwinther):
> Make these field final and remove the default values
> I guess this comment was referring to the field I just removed.  I'll ask
Johnni
> to be sure and remove the comment if so.

This CL is causing some analyzer tests to fail, e.g.: python tools/test.py
--checked -mrelease -cdart2analyzer -rnone -t240 pkg/docgen/test/typedef_test

The analyzer output is:
WARNING|STATIC_TYPE_WARNING|INVALID_ASSIGNMENT|/Users/paulberry/dart/dart-test/dart/pkg/compiler/lib/src/cps_ir/cps_ir_builder_task.dart|2300|41|104|A
value of type 'void' cannot be assigned to a variable of type 'List<Parameter>'

Looking at line 2300 of cps_ir_builder_task.dart, it looks like the return value
of buildFunctionHeader() is still needed.  Would you mind having a look?