DeterministicSocketDataProvider should not be RefCounted

DeterministicSocketDataProvider should not be RefCounted

BUG=123295
TEST=net_unittests


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=145845

[Ryan Sleevi, 2012-07-09 04:04:26]

Ryan: Almost all of these changes are in SPDY code and... since we talked about
it at lunch...

[Ryan Hamilton, 2012-07-09 17:35:47]

LGTM (assuming my comment does not actually point to a memory leak)

Thanks for doing this!

http://codereview.chromium.org/10690104/diff/1/net/http/http_pipelined_networ...
File net/http/http_pipelined_network_transaction_unittest.cc (right):

http://codereview.chromium.org/10690104/diff/1/net/http/http_pipelined_networ...
net/http/http_pipelined_network_transaction_unittest.cc:850:
DeterministicSocketData* data =
Is this a memory leak?  (Or does AddSocketDataProvider transfer ownership?  If
so, I wonder if scoped_ptr<> and then .release() would be more clear.  Your
call.)

[commit-bot: I haz the power, 2012-07-10 04:34:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsleevi@chromium.org/10690104/6017

[commit-bot: I haz the power, 2012-07-10 06:24:52]

Change committed as 145845