Allow the SDK to be set via a summary file

Allow the SDK to be set via a summary file

The generated summary file matches the generated DDC sdk - i.e., it
does not include dart:io.  I modified the codegen_tests to the summary
and had to prune the parts of unittest that relied on dart:io.

Paul / Konstantin: can one of you please look at the summary part?

Bob: can you please look at the unittest changes?

R=rnystrom@google.com, scheglov@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/d785be9ae644bf68c359eb8a7a89756dd73196d1

[vsm, 2016-05-16 23:41:58]

Description was changed from

==========
Allow the SDK to be set via a summary file

The generated summary file matches the generated DDC sdk - i.e., it
does not include dart:io.  I modified the codegen_tests to the summary
and had to prune the parts of unittest that relied on dart:io.

Paul: can you please look at the summary part?

Bob: can you please look at the unittest changes?
==========

to

==========
Allow the SDK to be set via a summary file

The generated summary file matches the generated DDC sdk - i.e., it
does not include dart:io.  I modified the codegen_tests to the summary
and had to prune the parts of unittest that relied on dart:io.

Paul / Konstantin: can one of you please look at the summary part?

Bob: can you please look at the unittest changes?
==========

[vsm, 2016-05-16 23:41:58]

vsm@google.com changed reviewers:
+ paulberry@google.com, rnystrom@google.com, scheglov@google.com

[vsm, 2016-05-16 23:43:21]

On 2016/05/16 23:42:11, vsm wrote:

Hmm, perhaps I should delete dart_sdk .sum from here.  The bots should regen it
as needed on a presubmit.

[scheglov, 2016-05-16 23:49:43]

LGTM

[Bob Nystrom, 2016-05-16 23:59:40]

Couple of nits then unittest stuff LGTM!

https://codereview.chromium.org/1982853002/diff/1/test/codegen_test.dart
File test/codegen_test.dart (right):

https://codereview.chromium.org/1982853002/diff/1/test/codegen_test.dart#newc...
test/codegen_test.dart:222: packageFiles.forEach((f) {
Nit: Use a regular for-in loop here.

https://codereview.chromium.org/1982853002/diff/1/test/codegen_test.dart#newc...
test/codegen_test.dart:232: .where((entry) => entry.path.endsWith('dart'))
Nit: ".dart"

[vsm, 2016-05-17 00:06:38]

https://codereview.chromium.org/1982853002/diff/1/test/codegen_test.dart
File test/codegen_test.dart (right):

https://codereview.chromium.org/1982853002/diff/1/test/codegen_test.dart#newc...
test/codegen_test.dart:222: packageFiles.forEach((f) {
On 2016/05/16 23:59:39, Bob Nystrom wrote:
> Nit: Use a regular for-in loop here.

Done.

https://codereview.chromium.org/1982853002/diff/1/test/codegen_test.dart#newc...
test/codegen_test.dart:232: .where((entry) => entry.path.endsWith('dart'))
On 2016/05/16 23:59:40, Bob Nystrom wrote:
> Nit: ".dart"

Done.

[vsm, 2016-05-17 00:10:16]

Description was changed from

==========
Allow the SDK to be set via a summary file

The generated summary file matches the generated DDC sdk - i.e., it
does not include dart:io.  I modified the codegen_tests to the summary
and had to prune the parts of unittest that relied on dart:io.

Paul / Konstantin: can one of you please look at the summary part?

Bob: can you please look at the unittest changes?
==========

to

==========
Allow the SDK to be set via a summary file

The generated summary file matches the generated DDC sdk - i.e., it
does not include dart:io.  I modified the codegen_tests to the summary
and had to prune the parts of unittest that relied on dart:io.

Paul / Konstantin: can one of you please look at the summary part?

Bob: can you please look at the unittest changes?

R=rnystrom@google.com, scheglov@google.com

Committed:
https://github.com/dart-lang/dev_compiler/commit/d785be9ae644bf68c359eb8a7a89...
==========

[vsm, 2016-05-17 00:10:18]

Committed patchset #2 (id:20001) manually as
d785be9ae644bf68c359eb8a7a89756dd73196d1 (presubmit successful).