Add support for custom cursors set by windowless plugins. Windowless plugins...

chrome/browser/render_widget_host_view_win.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/render_widget_host_view_win.h"
 
 #include "base/command_line.h"
 #include "base/gfx/gdi_util.h"
 #include "base/gfx/rect.h"
 #include "base/histogram.h"
@@ skipping 49 matching lines @@
 RenderWidgetHostView* RenderWidgetHostView::CreateViewForWidget(
     RenderWidgetHost* widget) {
   return new RenderWidgetHostViewWin(widget);
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // RenderWidgetHostViewWin, public:
 
 RenderWidgetHostViewWin::RenderWidgetHostViewWin(RenderWidgetHost* widget)
     : render_widget_host_(widget),
-      cursor_(LoadCursor(NULL, IDC_ARROW)),
-      cursor_is_custom_(false),
       track_mouse_leave_(false),
       ime_notification_(false),
       is_hidden_(false),
       close_on_deactivate_(false),
       tooltip_hwnd_(NULL),
       tooltip_showing_(false),
       shutdown_factory_(this),
       parent_hwnd_(NULL),
       is_loading_(false),
       focus_on_show_(true) {
   render_widget_host_->set_view(this);
   renderer_accessible_ =
       CommandLine().HasSwitch(switches::kEnableRendererAccessibility);
 }
 
 RenderWidgetHostViewWin::~RenderWidgetHostViewWin() {
-  if (cursor_is_custom_)
-    DestroyIcon(cursor_);
   ResetTooltip();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // RenderWidgetHostViewWin, RenderWidgetHostView implementation:
 
 RenderWidgetHost* RenderWidgetHostViewWin::GetRenderWidgetHost() const {
   return render_widget_host_;
 }
 
@@ skipping 111 matching lines @@
   WasHidden();
 }
 
 gfx::Rect RenderWidgetHostViewWin::GetViewBounds() const {
   CRect window_rect;
   GetWindowRect(&window_rect);
   return gfx::Rect(window_rect);
 }
 
 void RenderWidgetHostViewWin::UpdateCursor(const WebCursor& cursor) {
-  static HINSTANCE module_handle =
-      GetModuleHandle(chrome::kBrowserResourcesDll);
-
-  // If the last active cursor was a custom cursor, we need to destroy
-  // it before setting the new one.
-  if (cursor_is_custom_)
-    DestroyIcon(cursor_);
-
-  cursor_is_custom_ = cursor.IsCustom();
-  if (cursor_is_custom_) {
-    cursor_ = cursor.GetCustomCursor();
-  } else {
-    // We cannot pass in NULL as the module handle as this would only
-    // work for standard win32 cursors. We can also receive cursor
-    // types which are defined as webkit resources. We need to specify
-    // the module handle of chrome.dll while loading these cursors.
-    cursor_ = cursor.GetCursor(module_handle);
-  }
-
+  current_cursor_ = cursor;
   UpdateCursorIfOverSelf();
 }
 
 void RenderWidgetHostViewWin::UpdateCursorIfOverSelf() {
   static HCURSOR kCursorArrow = LoadCursor(NULL, IDC_ARROW);
   static HCURSOR kCursorAppStarting = LoadCursor(NULL, IDC_APPSTARTING);
+  static HINSTANCE module_handle =
+      GetModuleHandle(chrome::kBrowserResourcesDll);
 
-  HCURSOR display_cursor = cursor_;
+  // We cannot pass in NULL as the module handle as this would only work for
+  // standard win32 cursors. We can also receive cursor types which are defined
+  // as webkit resources. We need to specify the module handle of chrome.dll
+  // while loading these cursors.
+  HCURSOR display_cursor = current_cursor_.GetCursor(module_handle);
+
   // If a page is in the loading state, we want to show the Arrow+Hourglass
   // cursor only when the current cursor is the ARROW cursor. In all other
   // cases we should continue to display the current cursor.
   if (is_loading_ && display_cursor == kCursorArrow)
     display_cursor = kCursorAppStarting;
 
   // If the mouse is over our HWND, then update the cursor state immediately.
   CPoint pt;
   GetCursorPos(&pt);
   if (WindowFromPoint(pt) == m_hWnd)
@@ skipping 515 matching lines @@
   if (!handled_by_webcontents) {
     render_widget_host_->ForwardWheelEvent(
         WebMouseWheelEvent(m_hWnd, message, wparam, lparam));
   }
   handled = TRUE;
   return 0;
 }
 
 LRESULT RenderWidgetHostViewWin::OnMouseActivate(UINT, WPARAM, LPARAM,
                                                  BOOL& handled) {
+  if (!focus_on_show_)
+    return MA_NOACTIVATE;
+
   HWND focus_window = GetFocus();
   if (!::IsWindow(focus_window) || !IsChild(focus_window)) {
     // We handle WM_MOUSEACTIVATE to set focus to the underlying plugin
     // child window. This is to ensure that keyboard events are received
     // by the plugin. The correct way to fix this would be send over
     // an event to the renderer which would then eventually send over
     // a setFocus call to the plugin widget. This would ensure that
     // the renderer (webkit) knows about the plugin widget receiving
     // focus.
     // TODO(iyengar) Do the right thing as per the above comment.
@@ skipping 116 matching lines @@
   if (::IsWindow(tooltip_hwnd_))
     ::DestroyWindow(tooltip_hwnd_);
   tooltip_hwnd_ = NULL;
 }
 
 void RenderWidgetHostViewWin::ShutdownHost() {
   shutdown_factory_.RevokeAll();
   render_widget_host_->Shutdown();
   // Do not touch any members at this point, |this| has been deleted.
 }
-