Sync with trunk (r48545)
[reactos.git] / subsystems / win32 / win32k / ntuser / cursoricon.c
index f0c4de4..4b0fa99 100644 (file)
@@ -38,7 +38,7 @@
 
 #include <win32k.h>
 
-#define NDEBUG
+//#define NDEBUG
 #include <debug.h>
 
 static PAGED_LOOKASIDE_LIST gProcessLookasideList;
@@ -175,12 +175,10 @@ UserSetCursor(
     return hOldCursor;
 }
 
-BOOL UserSetCursorPos( INT x, INT y, BOOL CallHooks)
+BOOL UserSetCursorPos( INT x, INT y, BOOL SendMouseMoveMsg)
 {
     PWINDOW_OBJECT DesktopWindow;
     PSYSTEM_CURSORINFO CurInfo;
-    LARGE_INTEGER LargeTickCount;
-    MSLLHOOKSTRUCT MouseHookData;
     HDC hDC;
     MSG Msg;
 
@@ -223,50 +221,19 @@ BOOL UserSetCursorPos( INT x, INT y, BOOL CallHooks)
     gpsi->ptCursor.x = x;
     gpsi->ptCursor.y = y;
 
-    KeQueryTickCount(&LargeTickCount);
-    Msg.time = MsqCalculateMessageTime(&LargeTickCount);
-
     //Move the mouse pointer
     GreMovePointer(hDC, x, y);
 
+    if (!SendMouseMoveMsg)
+       return TRUE;
+
     //Generate a mouse move message
     Msg.message = WM_MOUSEMOVE;
     Msg.wParam = CurInfo->ButtonsDown;
     Msg.lParam = MAKELPARAM(x, y);
     Msg.pt = gpsi->ptCursor;
-
-    MouseHookData.pt.x = LOWORD(Msg.lParam);
-    MouseHookData.pt.y = HIWORD(Msg.lParam);
-    switch(Msg.message)
-    {
-        case WM_MOUSEWHEEL:
-            MouseHookData.mouseData = MAKELONG(0, GET_WHEEL_DELTA_WPARAM(Msg.wParam));
-            break;
-        case WM_XBUTTONDOWN:
-        case WM_XBUTTONUP:
-        case WM_XBUTTONDBLCLK:
-        case WM_NCXBUTTONDOWN:
-        case WM_NCXBUTTONUP:
-        case WM_NCXBUTTONDBLCLK:
-             MouseHookData.mouseData = MAKELONG(0, HIWORD(Msg.wParam));
-             break;
-        default:
-             MouseHookData.mouseData = 0;
-             break;
-     }
-
-    MouseHookData.flags = 0;
-    MouseHookData.time = Msg.time;
-    MouseHookData.dwExtraInfo = 0;
-
-    if (CallHooks)
-    {
-      /* If the hook procedure returned non zero, dont send the message */
-      if (co_HOOK_CallHooks(WH_MOUSE_LL, HC_ACTION, Msg.message, (LPARAM) &MouseHookData))
-        return FALSE;
-    }
-
     MsqInsertSystemMessage(&Msg);
+
     return TRUE;
 }
 
@@ -524,92 +491,6 @@ IntCleanupCurIcons(struct _EPROCESS *Process, PPROCESSINFO Win32Process)
 
 }
 
-/*
- * @implemented
- */
-HANDLE
-APIENTRY
-NtUserCreateCursorIconHandle(PICONINFO IconInfo OPTIONAL, BOOL Indirect)
-{
-    PCURICON_OBJECT CurIcon;
-    PSURFACE psurfBmp;
-    NTSTATUS Status;
-    HANDLE Ret;
-    DECLARE_RETURN(HANDLE);
-
-    DPRINT("Enter NtUserCreateCursorIconHandle\n");
-    UserEnterExclusive();
-
-    if (!(CurIcon = IntCreateCurIconHandle()))
-    {
-        SetLastWin32Error(ERROR_NOT_ENOUGH_MEMORY);
-        RETURN((HANDLE)0);
-    }
-
-    Ret = CurIcon->Self;
-
-    if (IconInfo)
-    {
-        Status = MmCopyFromCaller(&CurIcon->IconInfo, IconInfo, sizeof(ICONINFO));
-        if (NT_SUCCESS(Status))
-        {
-            /* Copy bitmaps and size info */
-            if (Indirect)
-            {
-                // FIXME: WTF?
-                CurIcon->IconInfo.hbmMask = BITMAP_CopyBitmap(CurIcon->IconInfo.hbmMask);
-                GDIOBJ_SetOwnership(CurIcon->IconInfo.hbmMask, NULL);
-                if(CurIcon->IconInfo.hbmColor)
-                {
-                    CurIcon->IconInfo.hbmColor = BITMAP_CopyBitmap(CurIcon->IconInfo.hbmColor);
-                    GDIOBJ_SetOwnership(CurIcon->IconInfo.hbmColor, NULL);
-                }
-            }
-            else
-            {
-                if (CurIcon->IconInfo.hbmColor &&
-                        (psurfBmp = SURFACE_LockSurface(CurIcon->IconInfo.hbmColor)))
-                {
-                    CurIcon->Size.cx = psurfBmp->SurfObj.sizlBitmap.cx;
-                    CurIcon->Size.cy = psurfBmp->SurfObj.sizlBitmap.cy;
-                    SURFACE_UnlockSurface(psurfBmp);
-                    GDIOBJ_SetOwnership(CurIcon->IconInfo.hbmColor, NULL);
-                }
-                if (CurIcon->IconInfo.hbmMask &&
-                        (psurfBmp = SURFACE_LockSurface(CurIcon->IconInfo.hbmMask)))
-                {
-                    if(!CurIcon->IconInfo.hbmColor)
-                    {
-                        CurIcon->Size.cx = psurfBmp->SurfObj.sizlBitmap.cx;
-                        CurIcon->Size.cy = psurfBmp->SurfObj.sizlBitmap.cy*2;
-                    }
-                    SURFACE_UnlockSurface(psurfBmp);
-                    GDIOBJ_SetOwnership(CurIcon->IconInfo.hbmMask, NULL);
-                }
-            }
-
-            /* Calculate icon hotspot */
-            if (CurIcon->IconInfo.fIcon == TRUE)
-            {
-                CurIcon->IconInfo.xHotspot = CurIcon->Size.cx >> 1;
-                CurIcon->IconInfo.yHotspot = CurIcon->Size.cy >> 1;
-            }
-        }
-        else
-        {
-            SetLastNtError(Status);
-            /* FIXME - Don't exit here */
-        }
-    }
-
-    UserDereferenceObject(CurIcon);
-    RETURN(Ret);
-
-CLEANUP:
-    DPRINT("Leave NtUserCreateCursorIconHandle, ret=%i\n",_ret_);
-    UserLeave();
-    END_CLEANUP;
-}
 
 /*
  * @implemented
@@ -1093,11 +974,13 @@ NtUserSetCursorContents(
     }
 
     /* Delete old bitmaps */
-    if (CurIcon->IconInfo.hbmColor != IconInfo.hbmColor)
+    if ((CurIcon->IconInfo.hbmColor)
+                       && (CurIcon->IconInfo.hbmColor != IconInfo.hbmColor))
     {
         GreDeleteObject(CurIcon->IconInfo.hbmColor);
     }
-    if (CurIcon->IconInfo.hbmMask != IconInfo.hbmMask)
+    if ((CurIcon->IconInfo.hbmMask)
+                       && CurIcon->IconInfo.hbmMask != IconInfo.hbmMask)
     {
         GreDeleteObject(CurIcon->IconInfo.hbmMask);
     }
@@ -1289,6 +1172,15 @@ NtUserSetCursorIconData(
     }
 
 done:
+       if(Ret)
+       {
+               /* This icon is shared now */
+               GDIOBJ_SetOwnership(CurIcon->IconInfo.hbmMask, NULL);
+               if(CurIcon->IconInfo.hbmColor)
+               {
+                       GDIOBJ_SetOwnership(CurIcon->IconInfo.hbmColor, NULL);
+               }
+       }
     UserDereferenceObject(CurIcon);
     RETURN(Ret);
 
@@ -1325,271 +1217,325 @@ UserDrawIconEx(
     HBRUSH hbrFlickerFreeDraw,
     UINT diFlags)
 {
-    BOOL Ret = FALSE;
-    HBITMAP hbmMask, hbmColor;
-    BITMAP bmpColor, bm;
+    PSURFACE psurfColor = NULL, psurfMask, psurfDst = NULL;
+    HGDIOBJ hObjs[3];
+    PGDIOBJ pObjs[3];
     BOOL DoFlickerFree;
-    SIZE IconSize;
-    INT iOldBkColor = 0, iOldTxtColor = 0;
-
-    HDC hMemDC, hOffDC = NULL;
-    HGDIOBJ hOldOffBrush = 0;
-    HGDIOBJ hOldOffBmp = 0;
-    HBITMAP hTmpBmp = 0, hOffBmp = 0;
-    BOOL bAlpha = FALSE;
-
-    hbmMask = pIcon->IconInfo.hbmMask;
-    hbmColor = pIcon->IconInfo.hbmColor;
+    PDC pdc;
+    HSURF hsurfDst = NULL;
+    RECTL rcSrc, rcDst;
+    CLIPOBJ clo, *pclo;
+    EXLATEOBJ exlo;
+    BOOL bAlpha = FALSE, Ret = FALSE, bStretch;
 
     if (istepIfAniCur)
         DPRINT1("NtUserDrawIconEx: istepIfAniCur is not supported!\n");
 
-    if (!hbmMask || !IntGdiGetObject(hbmMask, sizeof(BITMAP), (PVOID)&bm))
-    {
-        return FALSE;
-    }
+    DPRINT("Flags : 0x%08x\n", diFlags);
+
+    hObjs[0] = pIcon->IconInfo.hbmMask;
+    hObjs[1] = pIcon->IconInfo.hbmColor;
+    hObjs[2] = hDc;
+    GDIOBJ_LockMultipleObjs(3, hObjs, pObjs);
+    psurfMask = pObjs[0];
+    psurfColor = pObjs[1];
+    pdc = pObjs[2];
 
-    if (hbmColor && !IntGdiGetObject(hbmColor, sizeof(BITMAP), (PVOID)&bmpColor))
+    if (!pIcon->IconInfo.hbmMask
+         || !psurfMask)
     {
+        DPRINT1("No hbmMask?!\n");
+        if(pdc) DC_UnlockDc(pdc);
+        if(psurfColor) SURFACE_UnlockSurface(psurfColor);
         return FALSE;
     }
 
-    if(!(hMemDC = NtGdiCreateCompatibleDC(hDc)))
+    if (pIcon->IconInfo.hbmColor
+         && !psurfColor)
     {
-        DPRINT1("NtGdiCreateCompatibleDC failed!\n");
+        DPRINT1("Unable to lock the color Bitmap?!\n");
+        SURFACE_UnlockSurface(psurfMask);
+        if(pdc) DC_UnlockDc(pdc);
         return FALSE;
     }
 
-    if (hbmColor)
+    if(!psurfColor)
     {
-        IconSize.cx = bmpColor.bmWidth;
-        IconSize.cy = bmpColor.bmHeight;
+        DPRINT("Monochrome Icon\n");
+        psurfColor = psurfMask;
+        RECTL_vSetRect(&rcSrc, 0, pIcon->Size.cy, pIcon->Size.cx, 2*pIcon->Size.cy);
     }
-    else /* take it from mask */
+    else
     {
-        IconSize.cx = bm.bmWidth;
-        IconSize.cy = bm.bmHeight/2;
+        DPRINT("Color Icon\n");
+        RECTL_vSetRect(&rcSrc, 0, 0, pIcon->Size.cx, pIcon->Size.cy);
     }
 
-    if (!diFlags)
-        diFlags = DI_NORMAL;
+    if(!pdc)
+    {
+        SetLastWin32Error(ERROR_INVALID_PARAMETER);
+        SURFACE_UnlockSurface(psurfMask);
+        if(psurfColor != psurfMask) SURFACE_UnlockSurface(psurfColor);
+        DPRINT1("Invalid DC!\n");
+        return FALSE;
+    }
 
-    /* NtGdiCreateCompatibleBitmap will create a monochrome bitmap
-       when cxWidth or cyHeight is 0 */
-    if (hbmColor
-            && (bmpColor.bmBitsPixel == 32)
-            && (cxWidth != 0)
-            && (cyHeight != 0)
+    /* Check for alpha */
+    if ((BitsPerFormat(psurfColor->SurfObj.iBitmapFormat) == 32)
             && (diFlags & DI_IMAGE))
     {
-        SURFACE *psurfOff = NULL;
         PFN_DIB_GetPixel fnSource_GetPixel = NULL;
-        INT x, y;
+        INT i, j;
 
         /* In order to correctly display 32 bit icons Windows first scans the image,
            because information about transparency is not stored in any image's headers */
-        psurfOff = SURFACE_LockSurface(hbmColor);
-        if (psurfOff)
+        fnSource_GetPixel = DibFunctionsForBitmapFormat[psurfColor->SurfObj.iBitmapFormat].DIB_GetPixel;
+        if (fnSource_GetPixel)
         {
-            fnSource_GetPixel = DibFunctionsForBitmapFormat[psurfOff->SurfObj.iBitmapFormat].DIB_GetPixel;
-            if (fnSource_GetPixel)
+            for (i = 0; i < psurfColor->SurfObj.sizlBitmap.cx; i++)
             {
-                for (x = 0; x < psurfOff->SurfObj.sizlBitmap.cx; x++)
+                for (j = 0; j < psurfColor->SurfObj.sizlBitmap.cy; j++)
                 {
-                    for (y = 0; y < psurfOff->SurfObj.sizlBitmap.cy; y++)
-                    {
-                        bAlpha = ((BYTE)(fnSource_GetPixel(&psurfOff->SurfObj, x, y) >> 24) & 0xff);
-                        if (bAlpha)
-                            break;
-                    }
+                    bAlpha = ((BYTE)(fnSource_GetPixel(&psurfColor->SurfObj, i, j) >> 24) & 0xff);
                     if (bAlpha)
                         break;
                 }
+                if (bAlpha)
+                    break;
             }
-            SURFACE_UnlockSurface(psurfOff);
         }
     }
 
     if (!cxWidth)
         cxWidth = ((diFlags & DI_DEFAULTSIZE) ?
-                   UserGetSystemMetrics(SM_CXICON) : IconSize.cx);
+                   UserGetSystemMetrics(SM_CXICON) : pIcon->Size.cx);
 
     if (!cyHeight)
         cyHeight = ((diFlags & DI_DEFAULTSIZE) ?
-                    UserGetSystemMetrics(SM_CYICON) : IconSize.cy);
+                    UserGetSystemMetrics(SM_CYICON) : pIcon->Size.cy);
+
+    /* Check stretching */
+    bStretch = (pIcon->Size.cx != cxWidth) || (pIcon->Size.cy != cyHeight);
 
     DoFlickerFree = (hbrFlickerFreeDraw &&
                      (GDI_HANDLE_GET_TYPE(hbrFlickerFreeDraw) == GDI_OBJECT_TYPE_BRUSH));
 
     if (DoFlickerFree)
     {
-        hOffDC = NtGdiCreateCompatibleDC(hDc);
-        if(!hOffDC)
+        EBRUSHOBJ ebo;
+        PBRUSH pBrush ;
+        POINTL ptBrushOrig;
+
+        pBrush = BRUSH_LockBrush(hbrFlickerFreeDraw);
+        if(!pBrush)
         {
-            DPRINT1("NtGdiCreateCompatibleBitmap failed!\n");
-            Ret = FALSE;
-            goto Cleanup ;
+            SetLastWin32Error(ERROR_INVALID_PARAMETER);
+            DPRINT1("Invalid brush!\n");
+            goto cleanup;
         }
-        hOffBmp = NtGdiCreateCompatibleBitmap(hDc, cxWidth, cyHeight);
-        if(!hOffBmp)
+
+        hsurfDst = IntCreateCompatibleBitmap(pdc, cxWidth, cyHeight);
+        if(!hsurfDst)
         {
-            DPRINT1("NtGdiCreateCompatibleBitmap failed!\n");
-            goto Cleanup ;
+            DPRINT1("Error : Failed to allocate the offscreen surface\n");
+            goto cleanup;
         }
-        hOldOffBmp = NtGdiSelectBitmap(hOffDC, hOffBmp);
-        hOldOffBrush = NtGdiSelectBrush(hOffDC, hbrFlickerFreeDraw);
-        NtGdiPatBlt(hOffDC, 0, 0, cxWidth, cyHeight, PATCOPY);
-        NtGdiSelectBrush(hOffDC, hOldOffBrush);
+        psurfDst = SURFACE_LockSurface(hsurfDst);
+        if(!psurfDst)
+        {
+            DPRINT1("Error : Failed to lock the offScreen bitmap\n");
+            goto cleanup;
+        }
+        RECTL_vSetRect(&rcDst, 0, 0, cxWidth, cyHeight);
+
+        ptBrushOrig.x = pBrush->ptOrigin.x;
+        ptBrushOrig.y = pBrush->ptOrigin.y;
+
+        EBRUSHOBJ_vInit(&ebo, pBrush, pdc);
+
+        clo.iDComplexity =  DC_TRIVIAL;
+        pclo = &clo;
+
+        IntEngBitBlt(&psurfDst->SurfObj, NULL, NULL, pclo, NULL, &rcDst, NULL,
+                     NULL, &ebo.BrushObject, &ptBrushOrig, ROP3_TO_ROP4(PATCOPY));
+
+        EBRUSHOBJ_vCleanup(&ebo);
+        BRUSH_UnlockBrush(pBrush);
     }
     else
     {
-        /* Set Background/foreground colors */
-        iOldTxtColor = IntGdiSetTextColor(hDc, 0); //black
-        iOldBkColor = IntGdiSetBkColor(hDc, 0x00FFFFFF); //white
-    }
+        RECT rcBmp;
+        RECTL_vSetRect(&rcDst, xLeft, yTop, xLeft + cxWidth, yTop + cyHeight);
+        IntLPtoDP(pdc, (LPPOINT)&rcDst, 2);
+        RECTL_vOffsetRect(&rcDst, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y);
 
+        DC_vPrepareDCsForBlit(pdc, rcDst, NULL, rcDst );
 
-    if (hbmMask && (diFlags & DI_MASK) && !bAlpha)
-    {
-        hTmpBmp = NtGdiSelectBitmap(hMemDC, hbmMask);
-        NtGdiStretchBlt(hOffDC ? hOffDC : hDc,
-                        hOffDC ? 0 : xLeft,
-                        hOffDC ? 0 : yTop,
-                        cxWidth,
-                        cyHeight,
-                        hMemDC,
-                        0,
-                        0,
-                        IconSize.cx,
-                        IconSize.cy,
-                        SRCAND,
-                        0);
-        NtGdiSelectBitmap(hMemDC, hTmpBmp);
-    }
+        if (pdc->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY))
+            DC_vUpdateFillBrush(pdc);
 
-    if(diFlags & DI_IMAGE)
-    {
-        if (bAlpha)
+        psurfDst = pdc->dclevel.pSurface;
+        pclo = pdc->rosdc.CombinedClip;
+        RECTL_vSetRect(&rcBmp, 0, 0, psurfDst->SurfObj.sizlBitmap.cx, psurfDst->SurfObj.sizlBitmap.cy);
+        if(!RECTL_bIntersectRect(&rcDst, &rcDst, &rcBmp))
         {
-            BLENDFUNCTION pixelblend = { AC_SRC_OVER, 0, 255, AC_SRC_ALPHA };
-            DWORD Pixel;
-            BYTE Red, Green, Blue, Alpha;
-            DWORD Count = 0;
-            INT i, j;
-            PSURFACE psurf;
-            PBYTE pBits ;
-            HBITMAP hMemBmp = NULL;
-
-            pBits = ExAllocatePoolWithTag(PagedPool,
-                                          bmpColor.bmWidthBytes * abs(bmpColor.bmHeight),
-                                          TAG_BITMAP);
-            if (pBits == NULL)
-            {
-                Ret = FALSE;
-                goto CleanupAlpha;
-            }
-
-            hMemBmp = BITMAP_CopyBitmap(hbmColor);
-            if(!hMemBmp)
-            {
-                DPRINT1("BITMAP_CopyBitmap failed!");
-                goto CleanupAlpha;
-            }
+            Ret = FALSE;
+            goto done;
+        }
+    }
 
-            psurf = SURFACE_LockSurface(hMemBmp);
-            if(!psurf)
-            {
-                DPRINT1("SURFACE_LockSurface failed!\n");
-                goto CleanupAlpha;
-            }
-            /* get color bits */
-            IntGetBitmapBits(psurf,
-                             bmpColor.bmWidthBytes * abs(bmpColor.bmHeight),
-                             pBits);
+    /* Optimization : use directly the palette of the DC,
+     * so we XLATE only once, and then we directly copy bits */
+    EXLATEOBJ_vInitialize(&exlo, psurfColor->ppal, pdc->dclevel.pSurface->ppal, 0, 0, 0xFFFFFFFF);
 
-            /* premultiply with the alpha channel value */
-            for (i = 0; i < cyHeight; i++)
-            {
-                for (j = 0; j < cxWidth; j++)
-                {
-                    Pixel = *(DWORD *)(pBits + Count);
+       if(bAlpha && (diFlags & DI_IMAGE))
+       {
+               BLENDFUNCTION pixelblend = { AC_SRC_OVER, 0, 255, AC_SRC_ALPHA };
+        BYTE Alpha;
+        INT i, j;
+        PSURFACE psurf = NULL;
+        PBYTE ptr ;
+        HBITMAP hMemBmp = NULL;
 
-                    Alpha = ((BYTE)(Pixel >> 24) & 0xff);
+        hMemBmp = BITMAP_CopyBitmap(pIcon->IconInfo.hbmColor);
+        if(!hMemBmp)
+        {
+            DPRINT1("BITMAP_CopyBitmap failed!");
+            goto CleanupAlpha;
+        }
 
-                    Red   = (((BYTE)(Pixel >>  0)) * Alpha) / 0xff;
-                    Green = (((BYTE)(Pixel >>  8)) * Alpha) / 0xff;
-                    Blue  = (((BYTE)(Pixel >> 16)) * Alpha) / 0xff;
+        psurf = SURFACE_LockSurface(hMemBmp);
+        if(!psurf)
+        {
+            DPRINT1("SURFACE_LockSurface failed!\n");
+            goto CleanupAlpha;
+        }
 
-                    *(DWORD *)(pBits + Count) = (DWORD)(Red | (Green << 8) | (Blue << 16) | (Alpha << 24));
+        /* premultiply with the alpha channel value */
+        for (i = 0; i < psurf->SurfObj.sizlBitmap.cy; i++)
+        {
+                       ptr = (PBYTE)psurf->SurfObj.pvScan0 + i*psurf->SurfObj.lDelta;
+            for (j = 0; j < psurf->SurfObj.sizlBitmap.cx; j++)
+            {
+                Alpha = ptr[3];
+                ptr[0] = (ptr[0] * Alpha) / 0xff;
+                ptr[1] = (ptr[1] * Alpha) / 0xff;
+                ptr[2] = (ptr[2] * Alpha) / 0xff;
 
-                    Count += sizeof(DWORD);
-                }
+                               ptr += 4;
             }
+        }
 
-            /* set mem bits */
-            IntSetBitmapBits(psurf,
-                             bmpColor.bmWidthBytes * abs(bmpColor.bmHeight),
-                             pBits);
-            SURFACE_UnlockSurface(psurf);
-
-            hTmpBmp = NtGdiSelectBitmap(hMemDC, hMemBmp);
-
-            NtGdiAlphaBlend(hOffDC ? hOffDC : hDc,
-                            hOffDC ? 0 : xLeft,
-                            hOffDC ? 0 : yTop,
-                            cxWidth,
-                            cyHeight,
-                            hMemDC,
-                            0,
-                            0,
-                            IconSize.cx,
-                            IconSize.cy,
-                            pixelblend,
-                            NULL);
-            NtGdiSelectBitmap(hMemDC, hTmpBmp);
-        CleanupAlpha:
-            if(pBits) ExFreePoolWithTag(pBits, TAG_BITMAP);
-            if(hMemBmp) NtGdiDeleteObjectApp(hMemBmp);
+        DPRINT("Performing alpha blending\n");
+        Ret = IntEngAlphaBlend(&psurfDst->SurfObj,
+                               &psurf->SurfObj,
+                               pclo,
+                               &exlo.xlo,
+                               &rcDst,
+                               &rcSrc,
+                               (BLENDOBJ*)&pixelblend);
+
+    CleanupAlpha:
+        if(psurf) SURFACE_UnlockSurface(psurf);
+        if(hMemBmp) NtGdiDeleteObjectApp(hMemBmp);
+               if(Ret) goto done;
+        else DPRINT1("IntEngAlphaBlend failed!\n");
+    }
+
+    if (diFlags & DI_IMAGE)
+    {
+        POINTL ptMaskOrig = {0,0};
+        if(bStretch)
+        {
+            DPRINT("Stretching\n");
+            Ret = IntEngStretchBlt(&psurfDst->SurfObj,
+                                   &psurfColor->SurfObj,
+                                   (diFlags & DI_MASK) ? &psurfMask->SurfObj : NULL,
+                                   pclo,
+                                   &exlo.xlo,
+                                   &rcDst,
+                                   &rcSrc,
+                                   (diFlags & DI_MASK) ? &ptMaskOrig : NULL,
+                                   NULL,
+                                   NULL,
+                                   (diFlags & DI_MASK) ? R4_MASK : ROP3_TO_ROP4(SRCCOPY));
+            if(!Ret) DPRINT1("IntEngStretchBlt Failed\n");
         }
-        else if (hbmColor)
+        else
         {
-            DWORD rop = (diFlags & DI_MASK) ? SRCINVERT : SRCCOPY ;
-            hTmpBmp = NtGdiSelectBitmap(hMemDC, hbmColor);
-            NtGdiStretchBlt(hOffDC ? hOffDC : hDc,
-                            hOffDC ? 0 : xLeft,
-                            hOffDC ? 0 : yTop,
-                            cxWidth,
-                            cyHeight,
-                            hMemDC,
-                            0,
-                            0,
-                            IconSize.cx,
-                            IconSize.cy,
-                            rop,
-                            0);
-            NtGdiSelectBitmap(hMemDC, hTmpBmp);
+            DPRINT("Blting\n");
+            Ret = IntEngBitBlt(&psurfDst->SurfObj,
+                               &psurfColor->SurfObj,
+                               (diFlags & DI_MASK) ? &psurfMask->SurfObj : NULL,
+                               pclo,
+                               &exlo.xlo,
+                               &rcDst,
+                               (PPOINTL)&rcSrc,
+                               (diFlags & DI_MASK) ? &ptMaskOrig : NULL,
+                               NULL,
+                               NULL,
+                               (diFlags & DI_MASK) ? R4_MASK : ROP3_TO_ROP4(SRCCOPY));
+            if(!Ret) DPRINT1("IntEngBitBlt Failed\n");
         }
     }
-
-    if(hOffDC)
+    else
     {
-        NtGdiBitBlt(hDc, xLeft, yTop, cxWidth, cyHeight, hOffDC, 0, 0, SRCCOPY, 0, 0);
+        DPRINT1("Uh? Calling DrawIcon without anything to draw? diFlags %d\n", diFlags);
     }
-    else
+
+done:
+    if(DoFlickerFree && Ret)
     {
-        IntGdiSetBkColor(hDc, iOldBkColor);
-        IntGdiSetTextColor(hDc, iOldTxtColor);
+        POINTL ptSrc = {0,0};
+        RECTL rcBmp;
+
+        RECTL_vSetRect(&rcDst, xLeft, yTop, xLeft + cxWidth, yTop + cyHeight);
+
+        IntLPtoDP(pdc, (LPPOINT)&rcDst, 2);
+
+        RECTL_vOffsetRect(&rcDst, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y);
+        DC_vPrepareDCsForBlit(pdc, rcDst, NULL, rcDst );
+
+        if (pdc->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY))
+            DC_vUpdateFillBrush(pdc);
+
+        RECTL_vSetRect(&rcBmp, 0, 0,
+                       pdc->dclevel.pSurface->SurfObj.sizlBitmap.cx,
+                       pdc->dclevel.pSurface->SurfObj.sizlBitmap.cy);
+
+        if(RECTL_bIntersectRect(&rcDst, &rcDst, &rcBmp))
+        {
+            /* Copy everything */
+            DPRINT("Copying bits from offscreen buffer\n");
+            Ret = IntEngCopyBits(&pdc->dclevel.pSurface->SurfObj,
+                                 &psurfDst->SurfObj,
+                                 pdc->rosdc.CombinedClip,
+                                 gpxloTrivial,
+                                 &rcDst,
+                                 &ptSrc);
+            if(!Ret) DPRINT1("IntEngCopyBits Failed\n");
+        }
+
+        DC_vFinishBlit(pdc, NULL);
     }
 
-    Ret = TRUE ;
+    if(!DoFlickerFree) DC_vFinishBlit(pdc, NULL);
+    EXLATEOBJ_vCleanup(&exlo);
+
+cleanup:
+    if(psurfColor != psurfMask)
+        SURFACE_UnlockSurface(psurfColor);
+    SURFACE_UnlockSurface(psurfMask);
+    if(hsurfDst)
+    {
+        if(psurfDst) SURFACE_UnlockSurface(psurfDst);
+        GreDeleteObject(hsurfDst);
+    }
+    DC_UnlockDc(pdc);
 
-Cleanup:
-    NtGdiDeleteObjectApp(hMemDC);
-    if(hOldOffBmp) NtGdiSelectBitmap(hOffDC, hOldOffBmp);
-    if(hOffDC) NtGdiDeleteObjectApp(hOffDC);
-    if(hOffBmp) NtGdiDeleteObjectApp(hOffBmp);
+    DPRINT("return %s\n", Ret ? "TRUE" : "FALSE") ;
 
-    return Ret;
+    return Ret ;
 }
 
 /*