Overhaul the semantics of Request.handlerPath and Request.url.

Overhaul the semantics of Request.handlerPath and Request.url.

The new semantics are easier to understand and make it easier to write handlers.
For full details, see the changelog.

Request.handlerPath used to be named Request.scriptName; the new name reflects
its semantics much better.

R=kevmoo@google.com

Committed: https://github.com/dart-lang/shelf/commit/287d1485534b52a25c4405af53cf9a87f2785f97

[kevmoo, 2015-03-04 21:45:19]

On 2015/03/04 21:28:07, nweiz wrote:

Have you rebased on the latest master? It looks like you might be missing a few
things...

[nweiz, 2015-03-04 21:58:05]

On 2015/03/04 21:45:19, kevmoo wrote:
> On 2015/03/04 21:28:07, nweiz wrote:
> 
> Have you rebased on the latest master? It looks like you might be missing a
few
> things...

Merged, PTAL

[kevmoo, 2015-03-04 22:29:35]

LGTM - see long line and check to see why log_middleware_test.dart is failing
with this change

There are also 2 tests with the same name in log_middleware_test

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

https://codereview.chromium.org/966063003/diff/1/lib/src/request.dart#newcode50
lib/src/request.dart:50: /// [handlerPath] is always a root-relative URL path;
that is, it always starts
long line

https://codereview.chromium.org/966063003/diff/40001/lib/src/request.dart
File lib/src/request.dart (right):

https://codereview.chromium.org/966063003/diff/40001/lib/src/request.dart#new...
lib/src/request.dart:51: /// [handlerPath] is always a root-relative URL path;
that is, it always starts
long line

[nweiz, 2015-03-04 23:46:31]

Code review changes

[nweiz, 2015-03-04 23:46:53]

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

https://codereview.chromium.org/966063003/diff/1/lib/src/request.dart#newcode50
lib/src/request.dart:50: /// [handlerPath] is always a root-relative URL path;
that is, it always starts
On 2015/03/04 22:29:35, kevmoo wrote:
> long line

Done.

[nweiz, 2015-03-04 23:47:07]

Committed patchset #4 (id:60001) manually as
287d1485534b52a25c4405af53cf9a87f2785f97 (presubmit successful).