netwerk/socket/nsSOCKSIOLayer.cpp: 219 PRStatus 220 nsSOCKSSocketInfo::ConnectToProxy(PRFileDesc *fd) [...] 234 rv = dns->Resolve(mProxyHost, 0, getter_AddRefs(mDnsRec)); We probably shouldn't ever make synchronous DNS resolution calls, and we definitely shouldn't do it on the main thread. Removing this stuff is a high priority.
Created attachment 636862 [details] [diff] [review] patch 0 while I do some more testing lets call this WIP
Comment on attachment 636862 [details] [diff] [review] patch 0 confirmed behavior before/after patch matches for: * normal dns based proxy * ip based proxy * dns resolution that takes a long time * invalid host name * valid hostname, but invalid proxy Christian, do you have time to look at this?
Comment on attachment 636862 [details] [diff] [review] patch 0 nice :)