Revert 75306 - Force native frames to repaint when OnNCActivate is called....

Revert 75306 - Force native frames to repaint when OnNCActivate is called. With a profile menu button added to the GlassBrowserFrameView, failure to trigger the frame repaint means that the button will remain in the active state even when the frame has been deactivated.

DesktopNotification tests failing on win full - speculative revert since this is the only CL in the range that seems possibly related.

BUG=none
TEST=frames behave normally in Aero mode, when activated and deactivated.

Review URL: http://codereview.chromium.org/6507044

TBR=mirandac@chromium.org
Review URL: http://codereview.chromium.org/6546006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75347 0039d316-1c4b-4281-b951-d872f2087c98

views/window/window_win.cc

@@ -854,10 +854,10 @@ void WindowWin::OnMouseLeave() {
 LRESULT WindowWin::OnNCActivate(BOOL active) {
   is_active_ = !!active;
 
-  // We need to force a synchronous repaint, otherwise we'll be left in the
-  // wrong activation state until something else causes a repaint later.
-  // Both the native and non-native frames may render activation-state
-  // dependent UI.
+  // If we're not using the native frame, we need to force a synchronous repaint
+  // otherwise we'll be left in the wrong activation state until something else
+  // causes a repaint later.
+  if (!non_client_view_->UseNativeFrame()) {
     // We can get WM_NCACTIVATE before we're actually visible. If we're not
     // visible, no need to paint.
     if (IsWindowVisible(GetNativeView())) {
@@ -866,6 +866,7 @@ LRESULT WindowWin::OnNCActivate(BOOL active) {
       // painting operations while the move is in progress.
       PaintNow(root_view_->GetScheduledPaintRect());
     }
+  }
 
   // If we're active again, we should be allowed to render as inactive, so
   // tell the non-client view. This must be done independently of the check for