Properly serve descriptors containing URI-sensitive characters in pub.

Properly serve descriptors containing URI-sensitive characters in pub.

R=rnystrom@google.com

Committed: https://code.google.com/p/dart/source/detail?r=39570

[Bob Nystrom, 2014-08-26 21:31:53]

One suggestion, but LGTM.

https://codereview.chromium.org/507823002/diff/1/sdk/lib/_internal/pub/test/t...
File sdk/lib/_internal/pub/test/test_pub.dart (right):

https://codereview.chromium.org/507823002/diff/1/sdk/lib/_internal/pub/test/t...
sdk/lib/_internal/pub/test/test_pub.dart:238: '..', '..', '..', '..', 'pkg'));
While you're here, what do you think about changing this to "../../../../pkg"?

[nweiz, 2014-08-26 22:09:47]

Code review changes

[nweiz, 2014-08-26 22:09:52]

https://codereview.chromium.org/507823002/diff/1/sdk/lib/_internal/pub/test/t...
File sdk/lib/_internal/pub/test/test_pub.dart (right):

https://codereview.chromium.org/507823002/diff/1/sdk/lib/_internal/pub/test/t...
sdk/lib/_internal/pub/test/test_pub.dart:238: '..', '..', '..', '..', 'pkg'));
On 2014/08/26 21:31:53, Bob Nystrom wrote:
> While you're here, what do you think about changing this to "../../../../pkg"?

Done.

[nweiz, 2014-08-26 22:29:30]

Committed patchset #2 manually as 39570 (presubmit successful).