Load extraRequiredLibraries from the target passed in the options

Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want these implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in http://dartbug.com/30491

R=scheglov@google.com

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

[Siggi Cherem (dart-lang), 2017-08-23 23:19:48]

Description was changed from

==========
Load extraLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want this implicit imports. I sent an email to
brainstorm a solution for that.
==========

to

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want this implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in dartbug.com/30491
==========

[Siggi Cherem (dart-lang), 2017-08-23 23:22:52]

Patchset #1 (id:1) has been deleted

[Siggi Cherem (dart-lang), 2017-08-23 23:23:32]

sigmund@google.com changed reviewers:
+ paulberry@google.com, scheglov@google.com

[Siggi Cherem (dart-lang), 2017-08-23 23:23:33]

only need a review from one of you.

[Siggi Cherem (dart-lang), 2017-08-23 23:27:08]

Description was changed from

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want this implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in dartbug.com/30491
==========

to

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want these implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in dartbug.com/30491
==========

[Siggi Cherem (dart-lang), 2017-08-23 23:27:26]

Description was changed from

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want these implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in dartbug.com/30491
==========

to

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want these implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in http://dartbug.com/30491
==========

[scheglov, 2017-08-24 00:25:30]

LGTM

[Siggi Cherem (dart-lang), 2017-08-24 03:11:57]

Description was changed from

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want these implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in http://dartbug.com/30491
==========

to

==========
Load extraRequiredLibraries from the target passed in the options

This makes the behavior match what fasta does.

Long term I agree we don't want these implicit imports. I sent an email to
brainstorm a solution for that.

This change will address one of the issues reported in http://dartbug.com/30491

R=scheglov@google.com

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

[Siggi Cherem (dart-lang), 2017-08-24 03:11:58]

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