Coverage: 94.885% (-0.02%) from 94.901% when pulling db156c8 on chunweii:lazy-reading into 1d92ad2 on gavv:master.

Thanks for update! I'll review the code tomorrow, but here is one preliminary comment: we should wrap body into newBodyWrapper in response constructor, not in getContent. Because it's not guaranteed that getContent will be ever called. And adding newBodyWrapper is needed to ensure that if getContent is not called, the finalizer will close the body. (bodyWrapper has finalizer inside it).

Sorry for the test case duplication. My code is ready for review again, @gavv

Thanks for update! I've added a few small fixes:

• replaced contentReceived boolean with contentState enum; in addition to two existing states (not read; successfully read) I added a third state indicating that a read was attempted but failed; in this case we also don't want to read body again

• covered this addition with tests

• improved mock body a bit for better checks in tests of what was called; now wee test how many times read and close were called