Extract ClosedWorldImpl from WorldImpl.

Extract ClosedWorldImpl from WorldImpl.

R=sigmund@google.com

Committed: https://github.com/dart-lang/sdk/commit/3f977cb5d18b7cbedd91b16a2176e396fa96487a

[Johnni Winther, 2016-12-19 14:31:45]

Description was changed from

==========
dartfmt


Extract ClosedWorldImpl from WorldImpl.

BUG=
==========

to

==========
Extract ClosedWorldImpl from WorldImpl.
==========

[Johnni Winther, 2016-12-19 14:32:09]

johnniwinther@google.com changed reviewers:
+ sigmund@google.com

[Johnni Winther, 2016-12-20 08:36:06]

ping

[Siggi Cherem (dart-lang), 2016-12-20 15:36:36]

lgtm, nice to see this split!

https://codereview.chromium.org/2580403002/diff/1/pkg/compiler/lib/src/world....
File pkg/compiler/lib/src/world.dart (right):

https://codereview.chromium.org/2580403002/diff/1/pkg/compiler/lib/src/world....
pkg/compiler/lib/src/world.dart:494: reporter.internalError(cls, 'Class
"${cls.name}" is not resolved.');
related to the discussion in the other CL - this would be a case where we could
simply do an invariant assert and remove the reporter argument, correct?

[Johnni Winther, 2016-12-21 08:41:25]

https://codereview.chromium.org/2580403002/diff/1/pkg/compiler/lib/src/world....
File pkg/compiler/lib/src/world.dart (right):

https://codereview.chromium.org/2580403002/diff/1/pkg/compiler/lib/src/world....
pkg/compiler/lib/src/world.dart:494: reporter.internalError(cls, 'Class
"${cls.name}" is not resolved.');
On 2016/12/20 15:36:36, Siggi Cherem (dart-lang) wrote:
> related to the discussion in the other CL - this would be a case where we
could
> simply do an invariant assert and remove the reporter argument, correct?

Yes. Will do later.

[Johnni Winther, 2016-12-21 08:47:10]

Description was changed from

==========
Extract ClosedWorldImpl from WorldImpl.
==========

to

==========
Extract ClosedWorldImpl from WorldImpl.

R=sigmund@google.com

Committed:
https://github.com/dart-lang/sdk/commit/3f977cb5d18b7cbedd91b16a2176e396fa96487a
==========

[Johnni Winther, 2016-12-21 08:47:12]

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