[ash-md] Disables the Wi-Fi and Cellular toggles until the state changes

ash/common/system/chromeos/network/network_list_md.cc

Index: ash/common/system/chromeos/network/network_list_md.cc
diff --git a/ash/common/system/chromeos/network/network_list_md.cc b/ash/common/system/chromeos/network/network_list_md.cc
index b91d6eacff82af46f21fafee6422c4b42749d584..e7e7ca74850d1aaa09ecc46d76108c3b8c015562 100644
--- a/ash/common/system/chromeos/network/network_list_md.cc
+++ b/ash/common/system/chromeos/network/network_list_md.cc
@@ -94,7 +94,10 @@ class NetworkListViewMd::SectionHeaderRowView : public views::View,
     AddToggleButton(enabled);
   }
 
-  virtual void SetEnabled(bool enabled) { toggle_->SetIsOn(enabled, true); }
+  virtual void SetIsOn(bool enabled) {
+    toggle_->SetEnabled(true);
+    toggle_->SetIsOn(enabled, true);

[tdanderson, 2017/02/28 16:30:03]

For Bluetooth you pass in false for the animation parameter here just after
re-enabling the toggle - is that inconsistency intentional?

[varkha, 2017/03/08 01:45:35]

Done (no it was not intentional although I don't know how you can execute
enabling BT without also hiding the detailed system menu view at the same time).

+  }
 
  protected:
   // This is called before the toggle button is added to give subclasses an
@@ -117,6 +120,9 @@ class NetworkListViewMd::SectionHeaderRowView : public views::View,
   // views::ButtonListener:
   void ButtonPressed(views::Button* sender, const ui::Event& event) override {
     DCHECK_EQ(toggle_, sender);
+    // Disable the toggle to prevent bouncing. It will get re-enabled when
+    // NetworkListViewMd::Update() gets called.
+    toggle_->SetEnabled(false);
     OnToggleToggled(toggle_->is_on());
   }
 
@@ -191,9 +197,9 @@ class WifiHeaderRowView : public NetworkListViewMd::SectionHeaderRowView {
 
   ~WifiHeaderRowView() override {}
 
-  void SetEnabled(bool enabled) override {
+  void SetIsOn(bool enabled) override {
     join_->SetEnabled(enabled);
-    SectionHeaderRowView::SetEnabled(enabled);
+    SectionHeaderRowView::SetIsOn(enabled);
   }
 
   const char* GetClassName() const override { return "WifiHeaderRowView"; }
@@ -593,7 +599,7 @@ int NetworkListViewMd::UpdateSectionHeaderRow(
     *separator_view = nullptr;
   }
 
-  (*view)->SetEnabled(enabled);
+  (*view)->SetIsOn(enabled);
   PlaceViewAtIndex(*view, child_index++);
   return child_index;
 }