Remove "listening on port $port" message.

Remove "listening on port $port" message.

This prevents it from interfering with stdio-based communication
between the editor and analysis server.

R=brianwilkerson@google.com, jwren@google.com

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

[Brian Wilkerson, 2014-04-15 23:19:34]

LGTM

https://codereview.chromium.org/236063019/diff/1/pkg/analysis_server/lib/http...
File pkg/analysis_server/lib/http_server.dart (right):

https://codereview.chromium.org/236063019/diff/1/pkg/analysis_server/lib/http...
pkg/analysis_server/lib/http_server.dart:88: stderr.writeln('Listening on port
$port');
Somehow, writing non-error messages to stderr seems wrong. It might be better to
simply remove it. I think it was originally added primarily as a user-level
confirmation that the server was running, and it isn't clear to me that we need
it at all. That said, I'll leave it to you to decide.

[Paul Berry, 2014-04-15 23:51:47]

On 2014/04/15 23:19:34, Brian Wilkerson wrote:
> LGTM
> 
>
https://codereview.chromium.org/236063019/diff/1/pkg/analysis_server/lib/http...
> File pkg/analysis_server/lib/http_server.dart (right):
> 
>
https://codereview.chromium.org/236063019/diff/1/pkg/analysis_server/lib/http...
> pkg/analysis_server/lib/http_server.dart:88: stderr.writeln('Listening on port
> $port');
> Somehow, writing non-error messages to stderr seems wrong. It might be better
to
> simply remove it. I think it was originally added primarily as a user-level
> confirmation that the server was running, and it isn't clear to me that we
need
> it at all. That said, I'll leave it to you to decide.

Good point.  I'll go ahead and completely remove it.

[jwren, 2014-04-16 00:04:01]

On 2014/04/15 23:51:47, Paul Berry wrote:
> On 2014/04/15 23:19:34, Brian Wilkerson wrote:
> > LGTM
> > 
> >
>
https://codereview.chromium.org/236063019/diff/1/pkg/analysis_server/lib/http...
> > File pkg/analysis_server/lib/http_server.dart (right):
> > 
> >
>
https://codereview.chromium.org/236063019/diff/1/pkg/analysis_server/lib/http...
> > pkg/analysis_server/lib/http_server.dart:88: stderr.writeln('Listening on
port
> > $port');
> > Somehow, writing non-error messages to stderr seems wrong. It might be
better
> to
> > simply remove it. I think it was originally added primarily as a user-level
> > confirmation that the server was running, and it isn't clear to me that we
> need
> > it at all. That said, I'll leave it to you to decide.
> 
> Good point.  I'll go ahead and completely remove it.

LGTM

[Paul Berry, 2014-04-16 14:55:12]

Committed patchset #1 manually as r35107 (presubmit successful).