Back out Hugues Fournier's patch from bug 422172 as a possible cause of the crashes.

This commit is contained in:
reed%reedloden.com 2008-05-07 11:29:08 +00:00
Родитель 8eb061334c
Коммит c1251c7f9a
2 изменённых файлов: 6 добавлений и 7 удалений

Просмотреть файл

@ -498,7 +498,7 @@ nsProtocolProxyService::PrefsChanged(nsIPrefBranch *prefBranch,
mSystemProxySettings->GetPACURI(tempString);
}
if (!tempString.IsEmpty())
ConfigureFromPAC(tempString, PR_FALSE);
ConfigureFromPAC(tempString);
}
}
@ -758,8 +758,7 @@ nsProtocolProxyService::IsProxyDisabled(nsProxyInfo *pi)
}
nsresult
nsProtocolProxyService::ConfigureFromPAC(const nsCString &spec,
PRBool forceReload)
nsProtocolProxyService::ConfigureFromPAC(const nsCString &spec)
{
if (!mPACMan) {
mPACMan = new nsPACMan();
@ -772,7 +771,7 @@ nsProtocolProxyService::ConfigureFromPAC(const nsCString &spec,
if (NS_FAILED(rv))
return rv;
if (mPACMan->IsPACURI(pacURI) && !forceReload)
if (mPACMan->IsPACURI(pacURI))
return NS_OK;
mFailedProxies.Clear();
@ -827,7 +826,7 @@ nsProtocolProxyService::ReloadPAC()
pacSpec.AssignLiteral(WPAD_URL);
if (!pacSpec.IsEmpty())
ConfigureFromPAC(pacSpec, PR_TRUE);
ConfigureFromPAC(pacSpec);
return NS_OK;
}
@ -1257,7 +1256,7 @@ nsProtocolProxyService::Resolve_Internal(nsIURI *uri,
!PACURI.IsEmpty()) {
// Switch to new PAC file if that setting has changed. If the setting
// hasn't changed, ConfigureFromPAC will exit early.
nsresult rv = ConfigureFromPAC(PACURI, PR_FALSE);
nsresult rv = ConfigureFromPAC(PACURI);
if (NS_FAILED(rv))
return rv;
} else {

Просмотреть файл

@ -114,7 +114,7 @@ protected:
* @param pacURI
* The URI spec of the PAC file to load.
*/
NS_HIDDEN_(nsresult) ConfigureFromPAC(const nsCString &pacURI, PRBool forceReload);
NS_HIDDEN_(nsresult) ConfigureFromPAC(const nsCString &pacURI);
/**
* This method builds a list of nsProxyInfo objects from the given PAC-