views: Add OnEnabledChanged() method to View class.

views/controls/button/text_button.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "views/controls/button/text_button.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/utf_string_conversions.h"
@@ skipping 542 matching lines @@
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // TextButtonBase, View overrides:
 
 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();
   SchedulePaint();
 }
 
 std::string TextButtonBase::GetClassName() const {
   return kViewClassName;
 }
 
@@ skipping 159 matching lines @@
       bounds.Inset(0, 0, extra_width, 0);
     }
   }
 
   return bounds;
 }
 
 }  // namespace views