Default to "127.0.0.1" in pub serve.

Default to "127.0.0.1" in pub serve.

BUG=https://code.google.com/p/dart/issues/detail?id=14677
R=nweiz@google.com

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

[nweiz, 2013-11-08 00:57:57]

lgtm

[Bob Nystrom, 2013-11-08 18:25:21]

Thanks to Søren, I've got a better patch for this. PTAL.

[nweiz, 2013-11-08 19:35:14]

lgtm

https://codereview.chromium.org/65663002/diff/50001/sdk/lib/_internal/pub/lib...
File sdk/lib/_internal/pub/lib/src/barback/load_transformers.dart (right):

https://codereview.chromium.org/65663002/diff/50001/sdk/lib/_internal/pub/lib...
sdk/lib/_internal/pub/lib/src/barback/load_transformers.dart:458: hostAndPort +=
':${server.port}';
Maybe encapsulate this in a utils method that takes an address and a port and
returns a string.

[Bob Nystrom, 2013-11-08 22:35:42]

Thanks!

https://codereview.chromium.org/65663002/diff/50001/sdk/lib/_internal/pub/lib...
File sdk/lib/_internal/pub/lib/src/barback/load_transformers.dart (right):

https://codereview.chromium.org/65663002/diff/50001/sdk/lib/_internal/pub/lib...
sdk/lib/_internal/pub/lib/src/barback/load_transformers.dart:458: hostAndPort +=
':${server.port}';
On 2013/11/08 19:35:14, nweiz wrote:
> Maybe encapsulate this in a utils method that takes an address and a port and
> returns a string.

Done.

[Bob Nystrom, 2013-11-08 22:46:49]

Committed patchset #3 manually as r30128 (presubmit successful).