chromeos: Split NetworkChangeNotifierChromeos initialization.

chrome/browser/chromeos/net/network_change_notifier_chromeos.cc

Index: chrome/browser/chromeos/net/network_change_notifier_chromeos.cc
diff --git a/chrome/browser/chromeos/net/network_change_notifier_chromeos.cc b/chrome/browser/chromeos/net/network_change_notifier_chromeos.cc
index eb6503b38ea5abcb8bba24cb3877a98db9afb179..9cb9725ff789a07bcc53066fbd0c43863b79f448 100644
--- a/chrome/browser/chromeos/net/network_change_notifier_chromeos.cc
+++ b/chrome/browser/chromeos/net/network_change_notifier_chromeos.cc
@@ -65,15 +65,6 @@ NetworkChangeNotifierChromeos::NetworkChangeNotifierChromeos()
       connection_state_(chromeos::STATE_UNKNOWN),
       online_notification_task_(NULL) {
 
-  chromeos::NetworkLibrary* net =
-      chromeos::CrosLibrary::Get()->GetNetworkLibrary();
-  net->AddNetworkManagerObserver(this);
-
-  chromeos::PowerLibrary* power =
-      chromeos::CrosLibrary::Get()->GetPowerLibrary();
-  power->AddObserver(this);
-
-  UpdateNetworkState(net);

[satorux1, 2011/11/03 20:13:54]

So I thought the removal of the line 76 was harmless as net->active_network() is
NULL by the time this is called, which is true in production.

However, for tests on Linux desktop, net->active_network() is non-NULL here, and
removing the line breaks many tests in browser_tests.

Now thinking about how to fix this...

   BrowserThread::PostDelayedTask(
          BrowserThread::UI, FROM_HERE,
          base::Bind(
@@ -257,4 +248,4 @@ void NetworkChangeNotifierChromeos::UpdateInitialState(
   self->UpdateNetworkState(net);
 }
 
-}  // namespace net
+}  // namespace chromeos