pkg/shelf: include the original `onHijack` callback for Request.change

pkg/shelf: include the original `onHijack` callback for Request.change

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

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

[kevmoo, 2014-09-30 02:41:23]

kevmoo@google.com changed reviewers:
+ nweiz@google.com

[kevmoo, 2014-10-08 10:34:59]

On 2014/09/30 02:41:23, kevmoo wrote:

ping?

[nweiz, 2014-10-13 18:43:44]

lgtm

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/CHANGELOG.md
File pkg/shelf/CHANGELOG.md (right):

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/CHANGELOG.md#newcode4
pkg/shelf/CHANGELOG.md:4: exists.
Nit: +2 spaces.

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/lib/src/request.dart
File pkg/shelf/lib/src/request.dart (right):

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/lib/src/request.da...
pkg/shelf/lib/src/request.dart:119: Request(method, Uri requestedUri, {String
protocolVersion,
Add a type annotation for [method].

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/lib/src/request.da...
pkg/shelf/lib/src/request.dart:128: Request._(this.method, Uri requestedUri,
{String protocolVersion,
Explain how this constructor is different than [new Request].

[kevmoo, 2014-10-14 16:42:38]

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/CHANGELOG.md
File pkg/shelf/CHANGELOG.md (right):

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/CHANGELOG.md#newcode4
pkg/shelf/CHANGELOG.md:4: exists.
On 2014/10/13 18:43:43, nweiz wrote:
> Nit: +2 spaces.

Done.

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/lib/src/request.dart
File pkg/shelf/lib/src/request.dart (right):

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/lib/src/request.da...
pkg/shelf/lib/src/request.dart:119: Request(method, Uri requestedUri, {String
protocolVersion,
On 2014/10/13 18:43:43, nweiz wrote:
> Add a type annotation for [method].

Done.

https://codereview.chromium.org/616463004/diff/1/pkg/shelf/lib/src/request.da...
pkg/shelf/lib/src/request.dart:128: Request._(this.method, Uri requestedUri,
{String protocolVersion,
On 2014/10/13 18:43:43, nweiz wrote:
> Explain how this constructor is different than [new Request].

Done.

[kevmoo, 2014-10-14 16:43:06]

Committed patchset #2 (id:50001) manually as 41102 (presubmit successful).