Encode URIs stored in unlinked summaries to make them Uri.parse() safe.

Encode URIs stored in unlinked summaries to make them Uri.parse() safe.

R=brianwilkerson@google.com, paulberry@google.com
BUG=

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

[Brian Wilkerson, 2017-02-01 17:28:24]

lgtm

https://codereview.chromium.org/2665213003/diff/1/pkg/analyzer/lib/src/summar...
File pkg/analyzer/lib/src/summary/public_namespace_computer.dart (right):

https://codereview.chromium.org/2665213003/diff/1/pkg/analyzer/lib/src/summar...
pkg/analyzer/lib/src/summary/public_namespace_computer.dart:150: String uriStr =
Uri.encodeFull(node.uri.stringValue ?? '');
Consider a getter (such as 'encodedUri') on UriBasedDirective.

[Paul Berry, 2017-02-01 17:32:56]

lgtm

[scheglov, 2017-02-01 17:36:52]

Description was changed from

==========
Encode URIs stored in unlinked summaries to make them Uri.parse() safe.

R=brianwilkerson@google.com, paulberry@google.com
BUG=
==========

to

==========
Encode URIs stored in unlinked summaries to make them Uri.parse() safe.

R=brianwilkerson@google.com, paulberry@google.com
BUG=

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

[scheglov, 2017-02-01 17:36:54]

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