[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-07-09 Thread codesite-noreply
Updates: Cc: a...@chromium.org erik...@chromium.org Comment #22 on issue 13981 by willc...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Issue 15791 has been merged into this issue. -- You received this messag

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-29 Thread codesite-noreply
Comment #21 on issue 13981 by willc...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Issue 15593 has been merged into this issue. -- You received this message because you are listed in the owner or CC fields of this i

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-29 Thread codesite-noreply
Comment #20 on issue 13981 by willc...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Issue 15599 has been merged into this issue. -- You received this message because you are listed in the owner or CC fields of this i

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-29 Thread codesite-noreply
Comment #19 on issue 13981 by willc...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 The only thing different about this one is the frame at the top which looks bogus anyway. It should be fixed by wtc's changelist a

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-28 Thread codesite-noreply
Comment #18 on issue 13981 by lafo...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Seeing a new variant in 3.0.190.4 of the net::CertVerifier::Request::DoCallback(), it's the 3rd top crasher. Since other crashes

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-26 Thread codesite-noreply
Comment #17 on issue 13981 by willc...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 I see. You're right. But that's work we should do anyway, right? Perhaps we should file a bug to verify that all ClientSocket su

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-26 Thread codesite-noreply
Comment #16 on issue 13981 by w...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Also, we'll need to check the IsConnectedAndIdle() method of all ClientSocket subclasses to make sure it has a short-circuit like the s

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-26 Thread codesite-noreply
Comment #15 on issue 13981 by w...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Yes, my point is that we need to verify that IsConnectedAndIdle() is cheap for a disconnected socket. If we have to call recv(), it's a

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-26 Thread codesite-noreply
Updates: Status: Fixed Comment #14 on issue 13981 by willc...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Marked as fixed. I'm not sure I understand the point about IsConnectedAndIdle() being expensive. W

[chromium-bugs] Issue 13981 in chromium: Crash - net::SSLClientSocketWin::OnIOComplete(int)

2009-06-26 Thread codesite-noreply
Comment #13 on issue 13981 by w...@chromium.org: Crash - net::SSLClientSocketWin::OnIOComplete(int) http://code.google.com/p/chromium/issues/detail?id=13981 Will, you can mark this bug Fixed now. I agree with your comment 6 that we should just get rid of is_initialized() and check for the ex