Removed an unnecessary future when it came to constructing the request iteratee. In addition the RequestBodyHandler was re-factored to a) handle this future-less iteratee and b) to factor out the concern of replacing the channel pipeline i.e. keep the removal the caller's concern as per the insertion of the handler.

Finally the above re-factoring potentially fixes a memory leak given that new default upstream handlers were created on the completion of chunked requests.

Formatting
Added some documentation, streamlined some code by being able to remove the "redeemed" field, and incorporated Rich's suggested way of migrating from Play's Promise.
Updated test to reflect that the body parser is now run in the Netty IO thread given that we removed an unnecessary future from an earlier commit.
Merge pull request #1110 from huntc/request-performance

Streamline upstream handler performance around Iteratees