Bug 463088 - Tab tear off cursor touchup. sr=roc

content/events/src/nsDOMDataTransfer.cpp

@@ -301,10 +301,11 @@ nsDOMDataTransfer::GetMozItemCount(PRUint32* aCount)
 NS_IMETHODIMP
 nsDOMDataTransfer::GetMozCursor(nsAString& aCursorState)
 {
-  if (mCursorState)
-    aCursorState.AssignASCII("default");
-  else
-    aCursorState.AssignASCII("auto");
+  if (mCursorState) {
+    aCursorState.AssignLiteral("default");
+  } else {
+    aCursorState.AssignLiteral("auto");
+  }
   return NS_OK;
 }
 
@@ -312,7 +313,7 @@ NS_IMETHODIMP
 nsDOMDataTransfer::SetMozCursor(const nsAString& aCursorState)
 {
   // Lock the cursor to an arrow during the drag.
-  mCursorState = aCursorState.EqualsASCII("default");
+  mCursorState = aCursorState.EqualsLiteral("default");
 
   return NS_OK;
 }

dom/interfaces/events/nsIDOMDataTransfer.idl

@@ -182,11 +182,15 @@ interface nsIDOMNSDataTransfer : nsISupports
 
   /**
    * Sets the drag cursor state. Primarily used to control the cursor during
-   * tab drags, but could be expanded to other uses.
+   * tab drags, but could be expanded to other uses. XXX Currently implemented
+   * on Win32 only.
    *
    * Possible values:
    *  auto - use default system behavior.
    *  default - set the cursor to an arrow during the drag operation.
+   *
+   * Values other than 'default' are indentical to setting mozCursor to
+   * 'auto'.
    */
   attribute DOMString mozCursor;
 

widget/src/windows/nsNativeDragSource.cpp

@@ -130,7 +130,7 @@ nsNativeDragSource::GiveFeedback(DWORD dwEffect)
   if (mDataTransfer) {
     nsAutoString cursor;
     mDataTransfer->GetMozCursor(cursor);
-    if (cursor.EqualsASCII("default")) {
+    if (cursor.EqualsLiteral("default")) {
       m_hCursor = ::LoadCursor(0, IDC_ARROW);
     } else {
       m_hCursor =  nsnull;