diff options
| author | tim@chromium.org <tim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-23 20:06:28 +0000 |
|---|---|---|
| committer | tim@chromium.org <tim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-23 20:06:28 +0000 |
| commit | 69094b326c659f5d6d3d20bad0815af9efff3950 (patch) | |
| tree | 3987799a014bcb1670c5651e1c190c276301d90e /net/socket/socket_test_util.h | |
| parent | 129661ead16de9acb631ded92b65ca80ecd8b8f9 (diff) | |
| download | chromium_src-69094b326c659f5d6d3d20bad0815af9efff3950.zip chromium_src-69094b326c659f5d6d3d20bad0815af9efff3950.tar.gz chromium_src-69094b326c659f5d6d3d20bad0815af9efff3950.tar.bz2 | |
Believe this causes the valgrind linux test failures. Since it is a small change I decided to revert
instead of disable the test.
Revert 32807 - The async notification via OnReadComplete was still posting
a callback through this message loop. This makes it difficult
to write tests, as callbacks already scheduled will go in
front of the OnReadComplete(). Treat the notifications through
OnReadComplete as synchronous always.
This doesn't effect existing tests, but makes my new tests
possible.
TBR=mbelshe
BUG=none
TEST=this is the test!
Review URL: http://codereview.chromium.org/414073
TBR=mbelshe@google.com
Review URL: http://codereview.chromium.org/431016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32838 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/socket/socket_test_util.h')
| -rw-r--r-- | net/socket/socket_test_util.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/socket/socket_test_util.h b/net/socket/socket_test_util.h index 96c5ea7..d16dcb4 100644 --- a/net/socket/socket_test_util.h +++ b/net/socket/socket_test_util.h @@ -287,8 +287,6 @@ class MockClientSocket : public net::SSLClientSocket { // If an async IO is pending because the SocketDataProvider returned // ERR_IO_PENDING, then the MockClientSocket waits until this OnReadComplete // is called to complete the asynchronous read operation. - // data.async is ignored, and this read is completed synchronously as - // part of this call. virtual void OnReadComplete(const MockRead& data) = 0; protected: |