Use LinkedDependency.parts to find part URIs during resynthesis.

Use LinkedDependency.parts to find part URIs during resynthesis.

Now we can resynthesize a given library without having to consult any
summaries other than the linked summary for the library itself and the
unlinked summaries for all its compilation units.  All other summaries
are consulted only on demand if the information in them is needed.
Tests are updated to validate this.

R=scheglov@google.com

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

[Paul Berry, 2016-02-01 21:37:50]

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

[scheglov, 2016-02-01 21:45:22]

LGTM

https://codereview.chromium.org/1653163003/diff/1/pkg/analyzer/test/src/summa...
File pkg/analyzer/test/src/summary/resynthesize_test.dart (right):

https://codereview.chromium.org/1653163003/diff/1/pkg/analyzer/test/src/summa...
pkg/analyzer/test/src/summary/resynthesize_test.dart:122: for
(CompilationUnitElement partUnit in library.parts) {
You could use "library.units" and avoid the special case for the defining unit.

[Paul Berry, 2016-02-01 22:32:24]

Description was changed from

==========
Use LinkedDependency.parts to find part URIs during resynthesis.

Now we can resynthesize a given library without having to consult any
summaries other than the linked summary for the library itself and the
unlinked summaries for all its compilation units.  All other summaries
are consulted only on demand if the information in them is needed.
Tests are updated to validate this.
==========

to

==========
Use LinkedDependency.parts to find part URIs during resynthesis.

Now we can resynthesize a given library without having to consult any
summaries other than the linked summary for the library itself and the
unlinked summaries for all its compilation units.  All other summaries
are consulted only on demand if the information in them is needed.
Tests are updated to validate this.

R=scheglov@google.com

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

[Paul Berry, 2016-02-01 22:32:25]

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

[Paul Berry, 2016-02-01 22:32:32]

https://codereview.chromium.org/1653163003/diff/1/pkg/analyzer/test/src/summa...
File pkg/analyzer/test/src/summary/resynthesize_test.dart (right):

https://codereview.chromium.org/1653163003/diff/1/pkg/analyzer/test/src/summa...
pkg/analyzer/test/src/summary/resynthesize_test.dart:122: for
(CompilationUnitElement partUnit in library.parts) {
On 2016/02/01 21:45:22, scheglov wrote:
> You could use "library.units" and avoid the special case for the defining
unit.

Done.