views: Add OnEnabledChanged() method to View class.

views/controls/button/text_button.cc

Index: views/controls/button/text_button.cc
diff --git a/views/controls/button/text_button.cc b/views/controls/button/text_button.cc
index e72deb3b328d881fcfdd6a4b5f44377f3fc95c1a..d9c017ae942d4bc1752d4f533880bb358984a344 100644
--- a/views/controls/button/text_button.cc
+++ b/views/controls/button/text_button.cc
@@ -560,10 +560,8 @@ gfx::Size TextButtonBase::GetMinimumSize() {
   return max_text_size_;
 }
 
-void TextButtonBase::SetEnabled(bool enabled) {
-  if (enabled != IsEnabled()) {
-    CustomButton::SetEnabled(enabled);
-  }
+void TextButtonBase::OnEnabledChanged() {
+  CustomButton::OnEnabledChanged();

[tfarina, 2011/05/27 18:14:44]

It seems this call can be moved in the place of the SchedulePaint() call below.

[Ben Goodger (Google), 2011/05/27 18:19:06]

Yes you can just get rid of the SchedulePaint().

[tfarina, 2011/05/27 18:22:26]

Done.

   // We should always call UpdateColor() since the state of the button might be
   // changed by other functions like CustomButton::SetState().
   UpdateColor();