Fix residue left over from find bar when using accelerated compositing

chrome/browser/renderer_host/render_widget_host_view_win.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 CHROME_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_WIN_H_
 #define CHROME_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_WIN_H_
 #pragma once
 
 #include <atlbase.h>
 #include <atlapp.h>
@@ skipping 57 matching lines @@
       public RenderWidgetHostView,
       public NotificationObserver,
       public BrowserAccessibilityDelegate {
  public:
   // The view will associate itself with the given widget.
   explicit RenderWidgetHostViewWin(RenderWidgetHost* widget);
   virtual ~RenderWidgetHostViewWin();
 
   void CreateWnd(HWND parent);
 
+  // This method schedules and waits for compositing unless it was already done
+  // during this windows event handling loop.
+  // render_widget_host_->ScheduleComposite() blocks waiting for rendering
+  // results. ScheduleCompositeIfNeeded can be called in response to a WM_PAINT
+  // message to the compositor window or an OnPaint message to the parent
+  // window, or both, and we only want to wait once.
+  void ScheduleCompositeIfNeeded();
+
   DECLARE_WND_CLASS_EX(kRenderWidgetHostHWNDClass, CS_DBLCLKS, 0);
 
   BEGIN_MSG_MAP(RenderWidgetHostHWND)
     MSG_WM_CREATE(OnCreate)
     MSG_WM_ACTIVATE(OnActivate)
     MSG_WM_DESTROY(OnDestroy)
     MSG_WM_PAINT(OnPaint)
     MSG_WM_NCPAINT(OnNCPaint)
     MSG_WM_ERASEBKGND(OnEraseBkgnd)
     MSG_WM_SETCURSOR(OnSetCursor)
@@ skipping 173 matching lines @@
   // will be tiled such that it lines up with existing tiles starting from the
   // origin of |dc|.
   void DrawBackground(const RECT& rect, CPaintDC* dc);
 
   // Create an intermediate window between the given HWND and its parent.
   HWND ReparentWindow(HWND window);
 
   // Clean up the compositor window, if needed.
   void CleanupCompositorWindow();
 
+  // Reset was_compositing_just_scheduled_ to false.
+  void ResetWasCompositingJustScheduled();
+
   // Whether the window should be activated.
   bool IsActivatable() const;
 
   // The associated Model.
   RenderWidgetHost* render_widget_host_;
 
   // When we are doing accelerated compositing
   HWND compositor_host_window_;
 
+  // This bool is set to true on the first call to ScheduleComposite, and an

[darin (slow to review), 2011/05/26 23:03:06]

ScheduleComposite -> ScheduleCompositeIfNeeded, right?

+  // async task is posted to reset it to false.
+  // This is to ensure that the blocking call to
+  // render_widget_host_->ScheduleComposite() only happens once for the same
+  // set of windows WM_PAINT/OnPaint events.
+  bool was_compositing_just_scheduled_;
+
   // true if the compositor host window must be hidden after the
   // software renderered view is updated.
   bool hide_compositor_window_at_next_paint_;
 
   // The cursor for the page. This is passed up from the renderer.
   WebCursor current_cursor_;
 
   // Indicates if the page is loading.
   bool is_loading_;
 
@@ skipping 34 matching lines @@
   // The tooltip control hwnd
   HWND tooltip_hwnd_;
   // Whether or not a tooltip is currently visible. We use this to track
   // whether or not we want to force-close the tooltip when we receive mouse
   // move notifications from the renderer. See comment in OnMsgSetTooltipText.
   bool tooltip_showing_;
 
   // Factory used to safely scope delayed calls to ShutdownHost().
   ScopedRunnableMethodFactory<RenderWidgetHostViewWin> shutdown_factory_;
 
+  // Factory used to safely scope calls to RenderWidgetHostViewWin methods.
+  ScopedRunnableMethodFactory<RenderWidgetHostViewWin> method_factory_;
+
   // Our parent HWND.  We keep a reference to it as we SetParent(NULL) when
   // hidden to prevent getting messages (Paint, Resize...), and we reattach
   // when shown again.
   HWND parent_hwnd_;
 
   // Instance of accessibility information for the root of the MSAA
   // tree representation of the WebKit render tree.
   scoped_ptr<BrowserAccessibilityManager> browser_accessibility_manager_;
 
   // The time at which this view started displaying white pixels as a result of
@@ skipping 16 matching lines @@
   WebKit::WebTextInputType text_input_type_;
 
   ScopedVector<ui::ViewProp> props_;
 
   scoped_ptr<ui::ViewProp> accessibility_prop_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewWin);
 };
 
 #endif  // CHROME_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_WIN_H_