Add HttpsServer class and test.

sdk/lib/io/http.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * HTTP status codes.
  */
 abstract class HttpStatus {
   static const int CONTINUE = 100;
   static const int SWITCHING_PROTOCOLS = 101;
@@ skipping 44 matching lines @@
 /**
  * HTTP server.
  */
 abstract class HttpServer {
   factory HttpServer() => new _HttpServer();
 
   /**
    * Start listening for HTTP requests on the specified [host] and
    * [port]. If a [port] of 0 is specified the server will choose an
    * ephemeral port. The optional argument [backlog] can be used to
-   * specify the listen backlog for the underlying OS listen
-   * setup. See [addRequestHandler] and [defaultRequestHandler] for
+   * specify the listen backlog for the underlying OS listen.
+   * The optional argument [certificate_name] is used by the HttpsServer
+   * class, which shares the same interface.
+   * See [addRequestHandler] and [defaultRequestHandler] for
    * information on how incoming HTTP requests are handled.
    */
-  void listen(String host, int port, {int backlog: 128});
+  void listen(String host,

[Mads Ager (google), 2012/11/29 09:45:24]

When we are splitting out the abstract classes I don't think it makes sense to
have a certificate_name in this interface.

Can we split this out so that there is an HttpServerBase that has everything
that is shared. Then HttpServer implements HttpServerBase and has the listen
method without the certificate_name and HttpsServer has the listen method with
the certificate_name (as a required parameter?). 

+              int port,
+              {int backlog: 128,
+              String certificate_name});

[Mads Ager (google), 2012/11/29 09:45:24]

I would indent this line by one more char, but that is personal style:

{int backlog: 128
 String certificate_name});

 
   /**
    * Attach the HTTP server to an existing [:ServerSocket:]. If the
    * [HttpServer] is closed, the [HttpServer] will just detach itself,
    * and not close [serverSocket].
    */
   void listenOn(ServerSocket serverSocket);
 
   /**
    * Adds a request handler to the list of request handlers. The
@@ skipping 40 matching lines @@
 
   /**
    * Returns a [:HttpConnectionsInfo:] object with an overview of the
    * current connection handled by the server.
    */
   HttpConnectionsInfo connectionsInfo();
 }
 
 
 /**
+ * HTTPS server.
+ */
+abstract class HttpsServer implements HttpServer {
+  factory HttpsServer() => new _HttpServer.httpsServer();
+}
+
+
+/**
  * Overview information of the [:HttpServer:] socket connections.
  */
 class HttpConnectionsInfo {
   /**
    * Total number of socket connections.
    */
   int total = 0;
 
   /**
    * Number of active connections where actual request/response
@@ skipping 953 matching lines @@
 class RedirectLimitExceededException extends RedirectException {
   const RedirectLimitExceededException(List<RedirectInfo> redirects)
       : super("Redirect limit exceeded", redirects);
 }
 
 
 class RedirectLoopException extends RedirectException {
   const RedirectLoopException(List<RedirectInfo> redirects)
       : super("Redirect loop detected", redirects);
 }