summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorwillchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-09 00:37:04 +0000
committerwillchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-09 00:37:04 +0000
commitcf4375afc8ad2f2566af2665330d60d0a52beda3 (patch)
treec66bdeccc0b1b7f27c58d8480b83d4760f1fb633 /net
parent06e7d7fd35cbd00b8c0bc771a654d0d56d859ac4 (diff)
downloadchromium_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.cc14
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() {}