Revert 181390 "SPDY - implement greedy approach to read all the ..."

Revert 181390 "SPDY - implement greedy approach to read all the ..."
Revert 181637 "SPDY - Small bug fix for user after free. Will ..."
Revert 182012 "SPDY - Added unit tests for use after free of ..."

> SPDY - implement greedy approach to read all the data and process it
> from the ClientSocket until we block.
> 
> This is more of a first cut at the greedy approach to see if it
> improves the performance on the mobile.
> 
> spdy_session_test_util.* files have the common code between
> SpdySessionSpd2Test and SpdySessionSpd3Test. Will move other common
> code in a separate CL. Created this file to reduce errors and
> to avoid duplicating of code.
> 
> R=willchan
> BUG=166958
> TEST=network unittests
> 
> Review URL: https://chromiumcodereview.appspot.com/11644088

> SPDY - Small bug fix for user after free. Will revert the
> change asap.
> 
> The change adds a scoped_refptr to SpdySession in DoLoop
> to clean up the state and to finish pending reads, in case
>  SpdyFramer receives GOAway and closes all streams.
> 
> DoReadComplete was holding the last reference before and
> DoLoop was accessing the object after DoReadComplete deleted
> the object at the end of the function.
> 
> R=willchan@chromium.org
> BUG=175069
> TEST=network unit tests
> Review URL: https://codereview.chromium.org/12212102


> SPDY - Added unit tests for use after free of SpdySession
> and unit test to test yielding + async during Read.
> 
> A test that crashes when scoped_refptr to SpdySession deleted
> from DoLoop.
> 
> Added the following test per rch to test interactions of
> yielding + async during Read.
> 
> Do the following MockReads and verify all the data is consumed
> and SpdySession has yielded during Read.
>   SYNCHRONOUS 8K
>   SYNCHRONOUS 8K
>   SYNCHRONOUS 8K
>   SYNCHRONOUS 2K
>   ASYNC 8K
>   SYNCHRONOUS 8K
>   SYNCHRONOUS 8K
>   SYNCHRONOUS 8K
>   SYNCHRONOUS 2K
> 
> R=rch@chromium.org
> BUG=175574, 166958
> TEST=net unit tests
> 
> Review URL: https://chromiumcodereview.appspot.com/12207122


TBR=rtenneti@chromium.org, rch@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192313

[commit-bot: I haz the power, 2013-04-04 01:45:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/13608003/1

[ramant (doing other things), 2013-04-04 01:53:31]

Hi Ryan,
  Have reverted the following three patches (all related to greedy changes) to
see if this change fixes issue 224701.

  Will land these changes back with the right fix.

[commit-bot: I haz the power, 2013-04-04 01:54:22]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-04-04 02:22:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/13608003/1

[commit-bot: I haz the power, 2013-04-04 12:12:39]

Change committed as 192313