chromium/src
0
Fork 0
Code Activity

Revert 106069 - Fix for Memory Leak 72698_b

Browse Source 
BUG=72698
TEST=


Review URL: http://codereview.chromium.org/8330018

TBR=nrqv63@motorola.com
Review URL: http://codereview.chromium.org/8349007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106112 0039d316-1c4b-4281-b951-d872f2087c98
This commit is contained in:
thestig@chromium.org
2011-10-18 20:20:03 +00:00
parent 258a3313f4
commit 7f39bae056
4 changed files with 8 additions and 4 deletions
AUTHORS
chrome/browser/profiles
profile_impl_io_data.ccprofile_impl_io_data.h
tools/valgrind/memcheck
suppressions.txt

1
AUTHORS

@ -136,4 +136,3 @@ Nayan Kumar K <qtc746@motorola.com>
ShankarGanesh K <blr.bmlab@gmail.com>
Goutham Jagannatha <wrm364@motorola.com>
Rosen Dash <rosen.dash@gmail.com>
Parag Radke <nrqv63@motorola.com>

3
chrome/browser/profiles/profile_impl_io_data.cc

@ -393,9 +393,8 @@ void ProfileImplIOData::LazyInitializeInternal(
main_context->set_http_transaction_factory(main_cache);
media_request_context_->set_http_transaction_factory(media_cache);
ftp_factory_.reset(
main_context->set_ftp_transaction_factory(
new net::FtpNetworkLayer(io_thread_globals->host_resolver.get()));
main_context->set_ftp_transaction_factory(ftp_factory_.get());
main_context->set_chrome_url_data_manager_backend(
chrome_url_data_manager_backend());

1
chrome/browser/profiles/profile_impl_io_data.h

@ -139,7 +139,6 @@ class ProfileImplIOData : public ProfileIOData {
mutable scoped_ptr<net::HttpTransactionFactory> main_http_factory_;
mutable scoped_ptr<net::HttpTransactionFactory> media_http_factory_;
mutable scoped_ptr<net::FtpTransactionFactory> ftp_factory_;
mutable scoped_ptr<chrome_browser_net::Predictor> predictor_;
mutable scoped_ptr<chrome_browser_net::HttpServerPropertiesManager>

7
tools/valgrind/memcheck/suppressions.txt

@ -3126,6 +3126,13 @@
...
fun:_ZN13ProfileIOData20InitializeOnUIThreadEP7Profile
}
{
bug_72698_b
Memcheck:Leak
fun:_Znw*
...
fun:_ZNK13ProfileIOData14LazyInitializeEv
}
{
bug_73132
Memcheck:Leak