Add HttpRequest::requestedUri.

Add HttpRequest::requestedUri.

This makes it easier for the user to generate redirect-urls.

BUG=
R=sgjesse@google.com

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

[Søren Gjesse, 2014-01-07 11:13:42]

https://codereview.chromium.org/125743002/diff/40001/sdk/lib/io/http.dart
File sdk/lib/io/http.dart (right):

https://codereview.chromium.org/125743002/diff/40001/sdk/lib/io/http.dart#new...
sdk/lib/io/http.dart:793: * The absolute URI for the request (read-only).
I like the idea of having this, but we should be sure to make it work in most
settings.

The initial comment should start with something like this: "Reconstructs the URL
used by the client to make this request."

We should briefly explain how this reconstruction works, and that when reverse
proxies are used some configuration is required.

Also mention that in most cases the [uri] property should be used, and this is
mainly for making redirect responses.

Regarding the original protocol there is the 'X-Forwareded-Proto' header, see
http://stackoverflow.com/questions/13111080/what-is-a-full-specification-of-x....

Likewise the 'X-Forwarded-Host' is sometimes used for the original host.

Another option could be to wait with adding it to dart:io and instead add it as
a utility to the http_server package.

[Anders Johnsen, 2014-01-07 13:59:09]

PTAL

https://codereview.chromium.org/125743002/diff/40001/sdk/lib/io/http.dart
File sdk/lib/io/http.dart (right):

https://codereview.chromium.org/125743002/diff/40001/sdk/lib/io/http.dart#new...
sdk/lib/io/http.dart:793: * The absolute URI for the request (read-only).
On 2014/01/07 11:13:43, Søren Gjesse wrote:
> I like the idea of having this, but we should be sure to make it work in most
> settings.
> 
> The initial comment should start with something like this: "Reconstructs the
URL
> used by the client to make this request."
> 
> We should briefly explain how this reconstruction works, and that when reverse
> proxies are used some configuration is required.
> 
> Also mention that in most cases the [uri] property should be used, and this is
> mainly for making redirect responses.
> 
> Regarding the original protocol there is the 'X-Forwareded-Proto' header, see
>
http://stackoverflow.com/questions/13111080/what-is-a-full-specification-of-x....
> 
> Likewise the 'X-Forwarded-Host' is sometimes used for the original host.
> 
> Another option could be to wait with adding it to dart:io and instead add it
as
> a utility to the http_server package.

Done.

[Søren Gjesse, 2014-01-07 16:40:46]

lgtm

[Søren Gjesse, 2014-01-07 16:41:19]

On 2014/01/07 16:40:46, Søren Gjesse wrote:
> lgtm

But update the description to reflect the new name.

[Anders Johnsen, 2014-01-09 06:43:51]

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