Convert all connect code to use NetworkHandler instead of NetworkLibrary

chrome/browser/chromeos/options/passphrase_textfield.cc

Index: chrome/browser/chromeos/options/passphrase_textfield.cc
diff --git a/chrome/browser/chromeos/options/passphrase_textfield.cc b/chrome/browser/chromeos/options/passphrase_textfield.cc
index 598d1924f16b9aefe6e44300bfa5952bf4a93b82..9cc9bd33ca91a1e1e0eb6932dfa4e74d904286a1 100644
--- a/chrome/browser/chromeos/options/passphrase_textfield.cc
+++ b/chrome/browser/chromeos/options/passphrase_textfield.cc
@@ -8,12 +8,18 @@
 
 namespace chromeos {
 
-PassphraseTextfield::PassphraseTextfield(bool show_fake)
+PassphraseTextfield::PassphraseTextfield()
     : Textfield(views::Textfield::STYLE_OBSCURED),
-      show_fake_(show_fake),
+      show_fake_(false),

[pneubeck (no reviews), 2013/08/06 15:45:19]

I'm not happy about this change.
If this should use a default value at all, then it should be the secure one
namely show_fake_=true.

Also would it require a comment at the constructor, that it starts with this
default value.

[stevenjb, 2013/08/06 20:23:55]

We should really deprecate this entirely (it's only used by VpnConfigView, we
should really just use a Textfield with STYLE_OBSCURED), but I didn't want to do
that here. I can push a follow-up though. show_fake doesn't obscure anything, it
just shows ***** instead of blank.

       changed_(true) {
+}
+
+void PassphraseTextfield::SetShowFake(bool show_fake) {
+  show_fake_ = show_fake;
   if (show_fake_)
     SetFakePassphrase();
+  else
+    ClearFakePassphrase();
 }
 
 void PassphraseTextfield::OnFocus() {