Scrap WNDCLASSEX.hCursor, update GetCursorForPoint, etc.

views/view.h

Index: views/view.h
diff --git a/views/view.h b/views/view.h
index 8fe4de8656cf9a39ebd9cbe70b600260254e977f..a41988d4828aa9aa7797a7dedafe057405948747 100644
--- a/views/view.h
+++ b/views/view.h
@@ -583,12 +583,11 @@ class View : public AcceleratorTarget {
   // Returns the deepest visible descendant that contains the specified point.
   virtual View* GetEventHandlerForPoint(const gfx::Point& point);
 
-  // Return the cursor that should be used for this view or NULL if

[Peter Kasting, 2011/04/27 00:22:12]

Nit: This still can return NULL in some cases.  Should this sentence have been
left as-is?

[msw, 2011/04/27 01:58:03]

With native cursor types, NULL should be used explicitly to mean:
WIN: "the cursor is removed from the screen"
http://msdn.microsoft.com/en-us/library/ms648393(v=vs.85).aspx
GDK: "use the cursor of its parent window"
http://developer.gnome.org/gdk/stable/gdk-Windows.html#gdk-window-set-cursor

I nixed the behavior that replaced NULL with default/|previous_cursor_| from
WidgetWin, and WidgetGtk seemingly already uses the native behavior.

We ought to eventually create a cross platform cursor type/enum.

-  // the default cursor should be used. The provided point is in the
-  // receiver's coordinate system. The caller is responsible for managing the
-  // lifetime of the returned object, though that lifetime may vary from
-  // platform to platform. On Windows, the cursor is a shared resource but in
-  // Gtk, the framework destroys the returned cursor after setting it.
+  // Return the cursor that should be used for this view or the default cursor.
+  // The provided point is in the receiver's coordinate system. The caller is
+  // responsible for managing the lifetime of the returned object, though that
+  // lifetime may vary from platform to platform. On Windows, the cursor is a
+  //shared resource but Gtk destroys the returned cursor after setting it.

[Peter Kasting, 2011/04/27 00:22:12]

Nit: Initial space; comma after "resource"

[msw, 2011/04/27 01:58:03]

Done.

   virtual gfx::NativeCursor GetCursorForPoint(ui::EventType event_type,
                                               const gfx::Point& p);