Remove the RequestHandler interface from the HTTP library

Remove the RequestHandler interface from the HTTP library

Having the RequestHandler interface did no add anything, so it has
been removed. The HTTP server function addRequestHandler and setter
defaultRequestHandler now just takes function.

To use an instance of a class with a request handler member function
the Dart auto closurerization works fine.

R=ajohnsen@google.com, ager@google.com

BUG=none
TEST=tests/standalone/src/io/HttpServerHandlerTest.dart

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

[Anders Johnsen, 2012-04-25 08:46:23]

LGTM. I'm not sure on how exactly auto closurerization works in Dart, can we
fast a class directly to a Function?

https://chromiumcodereview.appspot.com/10226002/diff/1/runtime/bin/http_impl....
File runtime/bin/http_impl.dart (right):

https://chromiumcodereview.appspot.com/10226002/diff/1/runtime/bin/http_impl....
runtime/bin/http_impl.dart:871: } catch (var e) {
We should consider doing something with the Stacktrace here.

https://chromiumcodereview.appspot.com/10226002/diff/1/runtime/bin/http_impl....
runtime/bin/http_impl.dart:880: if (_defaultHandler != null) {
!==, as you wrote, we support taking classes using operator(), so != might be
overloaded.

[Søren Gjesse, 2012-04-25 08:54:41]

https://chromiumcodereview.appspot.com/10226002/diff/1/runtime/bin/http_impl....
File runtime/bin/http_impl.dart (right):

https://chromiumcodereview.appspot.com/10226002/diff/1/runtime/bin/http_impl....
runtime/bin/http_impl.dart:871: } catch (var e) {
On 2012/04/25 08:46:23, ajohnsen wrote:
> We should consider doing something with the Stacktrace here.

Yes, that is a general issue with error callbacks in dart:io.

https://chromiumcodereview.appspot.com/10226002/diff/1/runtime/bin/http_impl....
runtime/bin/http_impl.dart:880: if (_defaultHandler != null) {
On 2012/04/25 08:46:23, ajohnsen wrote:
> !==, as you wrote, we support taking classes using operator(), so != might be
> overloaded.

We only support functions now.

[Mads Ager (google), 2012-04-25 09:57:31]

lgtm