diff options
author | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-09 00:37:04 +0000 |
---|---|---|
committer | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-09 00:37:04 +0000 |
commit | cf4375afc8ad2f2566af2665330d60d0a52beda3 (patch) | |
tree | c66bdeccc0b1b7f27c58d8480b83d4760f1fb633 /net | |
parent | 06e7d7fd35cbd00b8c0bc771a654d0d56d859ac4 (diff) | |
download | chromium_src-cf4375afc8ad2f2566af2665330d60d0a52beda3.zip chromium_src-cf4375afc8ad2f2566af2665330d60d0a52beda3.tar.gz chromium_src-cf4375afc8ad2f2566af2665330d60d0a52beda3.tar.bz2 |
Disable NetworkChangeNotifierMac's helper thread.
This is so I can determine if r38052 caused the mac startup test regression.
This will disable the Mac implementation of NetworkChangeNotifier, which was implemented in January and is used by the ClientSocketPool to flush idle sockets on a network change.
r38052 added it to the HostResolver to flush the HostCache on a network change, which required instantiating NetworkChangeNotifierMac earlier in startup.
This change will prevent the initialization of the helper thread, which will turn off network change notifications on OS X. This is a relatively new feature, so it's fine to turn this off. I'll let this change sit overnight and check on the startup graphs tomorrow.
BUG=34926
Review URL: http://codereview.chromium.org/583015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38418 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
-rw-r--r-- | net/base/network_change_notifier_mac.cc | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/net/base/network_change_notifier_mac.cc b/net/base/network_change_notifier_mac.cc index 5606210..f0583db 100644 --- a/net/base/network_change_notifier_mac.cc +++ b/net/base/network_change_notifier_mac.cc @@ -218,12 +218,14 @@ void NetworkChangeNotifierThread::Init() { } // namespace NetworkChangeNotifierMac::NetworkChangeNotifierMac() - : notifier_thread_( - new NetworkChangeNotifierThread(MessageLoop::current(), this)) { - base::Thread::Options thread_options; - thread_options.message_loop_type = MessageLoop::TYPE_UI; - notifier_thread_->StartWithOptions(thread_options); -} +// http://crbug.com/34926: This might be slowing down Mac startup. Disable the +// helper thread to see if that improves mac startup times. +// : notifier_thread_( +// new NetworkChangeNotifierThread(MessageLoop::current(), this)) { +// base::Thread::Options thread_options; +// thread_options.message_loop_type = MessageLoop::TYPE_UI; +// notifier_thread_->StartWithOptions(thread_options); + : notifier_thread_(NULL) {} NetworkChangeNotifierMac::~NetworkChangeNotifierMac() {} |