views: Add OnEnabledChanged() method to View class.

views/view.h

 // 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.
 
 #ifndef VIEWS_VIEW_H_
 #define VIEWS_VIEW_H_
 #pragma once
 
 #include <algorithm>
 #include <map>
@@ skipping 299 matching lines @@
   // Return whether a view is visible
   virtual bool IsVisible() const;
 
   // Return whether a view and its ancestors are visible. Returns true if the
   // path from this view to the root view is visible.
   virtual bool IsVisibleInRootView() const;
 
   // Set whether this view is enabled. A disabled view does not receive keyboard
   // or mouse inputs. If flag differs from the current value, SchedulePaint is
   // invoked.
-  virtual void SetEnabled(bool flag);
+  void SetEnabled(bool enabled);
 
   // Returns whether the view is enabled.
   virtual bool IsEnabled() const;
 
+  virtual void OnEnabledChanged(bool enabled);

[Ben Goodger (Google), 2011/05/27 16:01:21]

I would say don't pass the parameter here. This method should only be called if
the state changes. The caller can call enabled() to read the current state.

[tfarina, 2011/05/27 16:25:04]

Done.

+
   // Transformations -----------------------------------------------------------
 
   // Methods for setting transformations for a view (e.g. rotation, scaling).
 
   const ui::Transform& GetTransform() const;
 
   // Clipping parameters. Clipping happens from the right and/or bottom. The
   // clipping amount is in parent's coordinate system, as in, if the view is
   // rotated, then the clipping will be applied after the rotation (and other
   // transformations, if any).
@@ skipping 1094 matching lines @@
   // The Windows-specific accessibility implementation for this View.
   scoped_refptr<NativeViewAccessibilityWin> native_view_accessibility_win_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(View);
 };
 
 }  // namespace views
 
 #endif  // VIEWS_VIEW_H_