diff options
| author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-12 00:46:55 +0000 |
|---|---|---|
| committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-12 00:46:55 +0000 |
| commit | b495e4fa3f1fcaaef798e62846b90a995c1950a8 (patch) | |
| tree | 68e41be7e68d94f7380bf2674d2a48aabab24ed6 /chrome_frame | |
| parent | f9fe00070f95ca9a53fde2ac48c4daae7f5390c2 (diff) | |
| download | chromium_src-b495e4fa3f1fcaaef798e62846b90a995c1950a8.zip chromium_src-b495e4fa3f1fcaaef798e62846b90a995c1950a8.tar.gz chromium_src-b495e4fa3f1fcaaef798e62846b90a995c1950a8.tar.bz2 | |
Attempt to fix a chrome frame tests crash consistently seen on the IE9 builder. I was able to
reproduce it once and it appears to be occuring when the ListenSocket attempts to pass an
OnObjectSignaled notification to a delegate which is the web server which has already been destroyed
Attempted fix is to destroy the ListenSocket in the destructor of the web server.
BUG=none
TEST=Existing chrome frame tests.
TBR=amit
Review URL: http://codereview.chromium.org/6155009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71125 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame')
| -rw-r--r-- | chrome_frame/test/test_server.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/chrome_frame/test/test_server.cc b/chrome_frame/test/test_server.cc index 8bc6689..2bddc96 100644 --- a/chrome_frame/test/test_server.cc +++ b/chrome_frame/test/test_server.cc @@ -237,6 +237,8 @@ HTTPTestServer::HTTPTestServer(int port, const std::wstring& address, } HTTPTestServer::~HTTPTestServer() { + LOG(INFO) << __FUNCTION__; + server_ = NULL; } std::list<scoped_refptr<ConfigurableConnection>>::iterator |