views: Add OnEnabledChanged() method to View class.

views/controls/native_control.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/native_control.h"
 
 #include <atlbase.h>
 #include <atlapp.h>
 #include <atlcrack.h>
 #include <atlframe.h>
 #include <atlmisc.h>
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "ui/base/accessibility/accessibility_types.h"
+#include "ui/base/keycodes/keyboard_code_conversion_win.h"
 #include "ui/base/keycodes/keyboard_codes.h"
-#include "ui/base/keycodes/keyboard_code_conversion_win.h"
 #include "ui/base/l10n/l10n_util_win.h"
 #include "ui/base/view_prop.h"
 #include "ui/base/win/hwnd_util.h"
 #include "views/background.h"
 #include "views/border.h"
 #include "views/controls/native/native_view_host.h"
 #include "views/focus/focus_manager.h"
 #include "views/widget/widget.h"
 
 using ui::ViewProp;
@@ skipping 143 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(NativeControlContainer);
 };
 
 NativeControl::NativeControl() : hwnd_view_(NULL),
                                  container_(NULL),
                                  fixed_width_(-1),
                                  horizontal_alignment_(CENTER),
                                  fixed_height_(-1),
                                  vertical_alignment_(CENTER) {
   enabled_ = true;

[tfarina, 2011/05/27 17:10:48]

I've removed this. View already initialize this to true.

   focusable_ = true;
 }
 
 NativeControl::~NativeControl() {
   if (container_) {
     container_->ResetParent();
     ::DestroyWindow(*container_);
   }
 }
 
@@ skipping 113 matching lines @@
   if (f != IsVisible()) {
     View::SetVisible(f);
     if (!f && container_) {
       ::DestroyWindow(*container_);
     } else if (f && !container_) {
       ValidateNativeControl();
     }
   }
 }
 
-void NativeControl::SetEnabled(bool enabled) {
-  if (enabled_ != enabled) {
-    View::SetEnabled(enabled);
-    if (GetNativeControlHWND()) {
-      EnableWindow(GetNativeControlHWND(), enabled_);
-    }
-  }
+void NativeControl::OnEnabledChanged() {
+  View::OnEnabledChanged();
+  if (GetNativeControlHWND())
+    EnableWindow(GetNativeControlHWND(), View::IsEnabled());
 }
 
 void NativeControl::OnPaint(gfx::Canvas* canvas) {
 }
 
 void NativeControl::VisibilityChanged(View* starting_from, bool is_visible) {
   SetVisible(is_visible);
 }
 
 void NativeControl::SetFixedWidth(int width, Alignment alignment) {
@@ skipping 55 matching lines @@
   } else if (message == WM_DESTROY) {
     ui::SetWindowProc(window, reinterpret_cast<WNDPROC>(original_handler));
     native_control->container_->prop_.reset();
   }
 
   return CallWindowProc(reinterpret_cast<WNDPROC>(original_handler), window,
                         message, w_param, l_param);
 }
 
 }  // namespace views