Changeset 91040 in webkit

Timestamp:

Jul 14, 2011 6:51:45 PM (13 years ago)

Author:

Darin Adler

Message:

[Mac] Use [NSCursor operationNotAllowedCursor] for CSS no-drop as well as CSS not-allowed
​https://bugs.webkit.org/show_bug.cgi?id=64570

Reviewed by Alexey Proskuryakov.

Mac OS X documentation and usage makes it clear that this cursor is usually used for no-drop,
and that it's also reasonable to use it for not-allowed.

• platform/mac/CursorMac.mm:

(WebCore::Cursor::ensurePlatformCursor): Use [NSCursor operationNotAllowedCursor] for
Cursor::NoDrop.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/mac/CursorMac.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-07-14  Darin Adler  <darin@apple.com>
+
+        [Mac] Use [NSCursor operationNotAllowedCursor] for CSS no-drop as well as CSS not-allowed
+        https://bugs.webkit.org/show_bug.cgi?id=64570
+
+        Reviewed by Alexey Proskuryakov.
+
+        Mac OS X documentation and usage makes it clear that this cursor is usually used for no-drop,
+        and that it's also reasonable to use it for not-allowed.
+
+        * platform/mac/CursorMac.mm:
+        (WebCore::Cursor::ensurePlatformCursor): Use [NSCursor operationNotAllowedCursor] for
+        Cursor::NoDrop.
+
 2011-07-14  Luke Macpherson   <macpherson@chromium.org>
 

trunk/Source/WebCore/platform/mac/CursorMac.mm

@@ -239 +239 @@
         break;
     case Cursor::NoDrop:
+#if !defined(BUILDING_ON_LEOPARD)
+        m_platformCursor = [NSCursor operationNotAllowedCursor];
+#else
         m_platformCursor = createNamedCursor("noDropCursor", 3, 1);
+#endif
         break;
     case Cursor::Copy: