git.oblomov.eu Git - wine/blob - dlls/user32/tests/menu.c

   1 /*
   2  * Unit tests for menus
   3  *
   4  * Copyright 2005 Robert Shearman
   5  * Copyright 2007 Dmitry Timoshkov
   6  *
   7  * This library is free software; you can redistribute it and/or
   8  * modify it under the terms of the GNU Lesser General Public
   9  * License as published by the Free Software Foundation; either
  10  * version 2.1 of the License, or (at your option) any later version.
  11  *
  12  * This library is distributed in the hope that it will be useful,
  13  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  14  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  15  * Lesser General Public License for more details.
  16  *
  17  * You should have received a copy of the GNU Lesser General Public
  18  * License along with this library; if not, write to the Free Software
  19  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
  20  */
  21
  22 #define _WIN32_WINNT 0x0501
  23
  24 #include <stdarg.h>
  25 #include <stdio.h>
  26 #include <string.h>
  27 #include <assert.h>
  28
  29 #define OEMRESOURCE         /* For OBM_MNARROW */
  30
  31 #include "windef.h"
  32 #include "winbase.h"
  33 #include "wingdi.h"
  34 #include "winuser.h"
  35
  36 #include "wine/test.h"
  37
  38 static ATOM atomMenuCheckClass;
  39
  40 static BOOL (WINAPI *pGetMenuInfo)(HMENU,LPCMENUINFO);
  41 static UINT (WINAPI *pSendInput)(UINT, INPUT*, size_t);
  42 static BOOL (WINAPI *pSetMenuInfo)(HMENU,LPCMENUINFO);
  43 static BOOL (WINAPI *pEndMenu) (void);
  44
  45 static void init_function_pointers(void)
  46 {
  47     HMODULE hdll = GetModuleHandleA("user32");
  48
  49 #define GET_PROC(func) \
  50     p ## func = (void*)GetProcAddress(hdll, #func); \
  51     if(!p ## func) \
  52       trace("GetProcAddress(%s) failed\n", #func);
  53
  54     GET_PROC(GetMenuInfo)
  55     GET_PROC(SendInput)
  56     GET_PROC(SetMenuInfo)
  57     GET_PROC(EndMenu)
  58
  59 #undef GET_PROC
  60 }
  61
  62 static BOOL correct_behavior(void)
  63 {
  64     HMENU hmenu;
  65     MENUITEMINFO info;
  66     BOOL rc;
  67
  68     hmenu = CreateMenu();
  69
  70     memset(&info, 0, sizeof(MENUITEMINFO));
  71     info.cbSize= sizeof(MENUITEMINFO);
  72     SetLastError(0xdeadbeef);
  73     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info);
  74     /* Win9x  : 0xdeadbeef
  75      * NT4    : ERROR_INVALID_PARAMETER
  76      * >= W2K : ERROR_MENU_ITEM_NOT_FOUND
  77      */
  78     if (!rc && GetLastError() != ERROR_MENU_ITEM_NOT_FOUND)
  79     {
  80         win_skip("NT4 and below can't handle a bigger MENUITEMINFO struct\n");
  81         DestroyMenu(hmenu);
  82         return FALSE;
  83     }
  84
  85     DestroyMenu(hmenu);
  86     return TRUE;
  87 }
  88
  89 static LRESULT WINAPI menu_check_wnd_proc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
  90 {
  91     switch (msg)
  92     {
  93     case WM_ENTERMENULOOP:
  94         /* mark window as having entered menu loop */
  95         SetWindowLongPtr(hwnd, GWLP_USERDATA, TRUE);
  96         /* exit menu modal loop
  97          * ( A SendMessage does not work on NT3.51 here ) */
  98         return PostMessage(hwnd, WM_CANCELMODE, 0, 0);
  99     }
 100     return DefWindowProc(hwnd, msg, wparam, lparam);
 101 }
 102
 103 /* The MSVC headers ignore our NONAMELESSUNION requests so we have to define
 104  * our own type */
 105 typedef struct
 106 {
 107     DWORD type;
 108     union
 109     {
 110         MOUSEINPUT      mi;
 111         KEYBDINPUT      ki;
 112         HARDWAREINPUT   hi;
 113     } u;
 114 } TEST_INPUT;
 115
 116 /* globals to communicate between test and wndproc */
 117
 118 static BOOL bMenuVisible;
 119 static BOOL got_input;
 120 static HMENU hMenus[4];
 121
 122 #define MOD_SIZE 10
 123 #define MOD_NRMENUS 8
 124
 125  /* menu texts with their sizes */
 126 static struct {
 127     LPCSTR text;
 128     SIZE size; /* size of text up to any \t */
 129     SIZE sc_size; /* size of the short-cut */
 130 } MOD_txtsizes[] = {
 131         { "Pinot &Noir" },
 132         { "&Merlot\bF4" },
 133         { "Shira&z\tAlt+S" },
 134         { "" },
 135         { NULL }
 136 };
 137
 138 static unsigned int MOD_maxid;
 139 static RECT MOD_rc[MOD_NRMENUS];
 140 static int MOD_avec, MOD_hic;
 141 static int MOD_odheight;
 142 static SIZE MODsizes[MOD_NRMENUS]= { {MOD_SIZE, MOD_SIZE},{MOD_SIZE, MOD_SIZE},
 143     {MOD_SIZE, MOD_SIZE},{MOD_SIZE, MOD_SIZE}};
 144 static int MOD_GotDrawItemMsg = FALSE;
 145 static int  gflag_initmenupopup,
 146             gflag_entermenuloop,
 147             gflag_initmenu;
 148
 149 /* wndproc used by test_menu_ownerdraw() */
 150 static LRESULT WINAPI menu_ownerdraw_wnd_proc(HWND hwnd, UINT msg,
 151         WPARAM wparam, LPARAM lparam)
 152 {
 153     static HMENU hmenupopup;
 154     switch (msg)
 155     {
 156         case WM_INITMENUPOPUP:
 157             gflag_initmenupopup++;
 158             hmenupopup = (HMENU) wparam;
 159             break;
 160         case WM_ENTERMENULOOP:
 161             gflag_entermenuloop++;
 162             break;
 163         case WM_INITMENU:
 164             gflag_initmenu++;
 165             break;
 166         case WM_MEASUREITEM:
 167             {
 168                 MEASUREITEMSTRUCT* pmis = (MEASUREITEMSTRUCT*)lparam;
 169                 if( winetest_debug)
 170                     trace("WM_MEASUREITEM received data %lx size %dx%d\n",
 171                             pmis->itemData, pmis->itemWidth, pmis->itemHeight);
 172                 MOD_odheight = pmis->itemHeight;
 173                 pmis->itemWidth = MODsizes[pmis->itemData].cx;
 174                 pmis->itemHeight = MODsizes[pmis->itemData].cy;
 175                 return TRUE;
 176             }
 177         case WM_DRAWITEM:
 178             {
 179                 DRAWITEMSTRUCT * pdis;
 180                 TEXTMETRIC tm;
 181                 HPEN oldpen;
 182                 char chrs[]="abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ";
 183                 SIZE sz;
 184                 int i;
 185                 pdis = (DRAWITEMSTRUCT *) lparam;
 186                 if( winetest_debug) {
 187                     RECT rc;
 188                     GetMenuItemRect( hwnd, (HMENU)pdis->hwndItem, pdis->itemData ,&rc);
 189                     trace("WM_DRAWITEM received hwnd %p hmenu %p itemdata %ld item %d rc %d,%d-%d,%d itemrc:  %d,%d-%d,%d\n",
 190                             hwnd, pdis->hwndItem, pdis->itemData, pdis->itemID,
 191                             pdis->rcItem.left, pdis->rcItem.top,
 192                             pdis->rcItem.right,pdis->rcItem.bottom,
 193                             rc.left,rc.top,rc.right,rc.bottom);
 194                     oldpen=SelectObject( pdis->hDC, GetStockObject(
 195                                 pdis->itemState & ODS_SELECTED ? WHITE_PEN :BLACK_PEN));
 196                     Rectangle( pdis->hDC, pdis->rcItem.left,pdis->rcItem.top,
 197                             pdis->rcItem.right,pdis->rcItem.bottom );
 198                     SelectObject( pdis->hDC, oldpen);
 199                 }
 200                 /* calculate widths of some menu texts */
 201                 if( ! MOD_txtsizes[0].size.cx)
 202                     for(i = 0; MOD_txtsizes[i].text; i++) {
 203                         char buf[100], *p;
 204                         RECT rc={0,0,0,0};
 205                         strcpy( buf, MOD_txtsizes[i].text);
 206                         if( ( p = strchr( buf, '\t'))) {
 207                             *p = '\0';
 208                             DrawText( pdis->hDC, p + 1, -1, &rc,
 209                                     DT_SINGLELINE|DT_CALCRECT);
 210                             MOD_txtsizes[i].sc_size.cx= rc.right - rc.left;
 211                             MOD_txtsizes[i].sc_size.cy= rc.bottom - rc.top;
 212                         }
 213                         DrawText( pdis->hDC, buf, -1, &rc,
 214                                 DT_SINGLELINE|DT_CALCRECT);
 215                         MOD_txtsizes[i].size.cx= rc.right - rc.left;
 216                         MOD_txtsizes[i].size.cy= rc.bottom - rc.top;
 217                     }
 218
 219                 if( pdis->itemData > MOD_maxid) return TRUE;
 220                 /* store the rectangl */
 221                 MOD_rc[pdis->itemData] = pdis->rcItem;
 222                 /* calculate average character width */
 223                 GetTextExtentPoint( pdis->hDC, chrs, 52, &sz );
 224                 MOD_avec = (sz.cx + 26)/52;
 225                 GetTextMetrics( pdis->hDC, &tm);
 226                 MOD_hic = tm.tmHeight;
 227                 MOD_GotDrawItemMsg = TRUE;
 228                 return TRUE;
 229             }
 230         case WM_ENTERIDLE:
 231             {
 232                 ok( lparam || broken(!lparam), /* win9x, nt4 */
 233                     "Menu window handle is NULL!\n");
 234                 if( lparam) {
 235                     HMENU hmenu = (HMENU)SendMessageA( (HWND)lparam, MN_GETHMENU, 0, 0);
 236                     ok( hmenupopup == hmenu, "MN_GETHMENU returns %p expected %p\n",
 237                         hmenu, hmenupopup);
 238                 }
 239                 PostMessage(hwnd, WM_CANCELMODE, 0, 0);
 240                 return TRUE;
 241             }
 242
 243     }
 244     return DefWindowProc(hwnd, msg, wparam, lparam);
 245 }
 246
 247 static void register_menu_check_class(void)
 248 {
 249     WNDCLASS wc =
 250     {
 251         0,
 252         menu_check_wnd_proc,
 253         0,
 254         0,
 255         GetModuleHandle(NULL),
 256         NULL,
 257         LoadCursor(NULL, IDC_ARROW),
 258         (HBRUSH)(COLOR_BTNFACE+1),
 259         NULL,
 260         TEXT("WineMenuCheck"),
 261     };
 262
 263     atomMenuCheckClass = RegisterClass(&wc);
 264 }
 265
 266 /* demonstrates that windows locks the menu object so that it is still valid
 267  * even after a client calls DestroyMenu on it */
 268 static void test_menu_locked_by_window(void)
 269 {
 270     BOOL ret;
 271     HMENU hmenu;
 272     HWND hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
 273                                WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
 274                                NULL, NULL, NULL, NULL);
 275     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
 276     hmenu = CreateMenu();
 277     ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
 278     ret = InsertMenu(hmenu, 0, MF_STRING, 0, TEXT("&Test"));
 279     ok(ret, "InsertMenu failed with error %d\n", GetLastError());
 280     ret = SetMenu(hwnd, hmenu);
 281     ok(ret, "SetMenu failed with error %d\n", GetLastError());
 282     ret = DestroyMenu(hmenu);
 283     ok(ret, "DestroyMenu failed with error %d\n", GetLastError());
 284
 285     ret = DrawMenuBar(hwnd);
 286     todo_wine {
 287     ok(ret, "DrawMenuBar failed with error %d\n", GetLastError());
 288     }
 289     ret = IsMenu(GetMenu(hwnd));
 290     ok(!ret || broken(ret) /* nt4 */, "Menu handle should have been destroyed\n");
 291
 292     SendMessage(hwnd, WM_SYSCOMMAND, SC_KEYMENU, 0);
 293     /* did we process the WM_INITMENU message? */
 294     ret = GetWindowLongPtr(hwnd, GWLP_USERDATA);
 295     todo_wine {
 296     ok(ret, "WM_INITMENU should have been sent\n");
 297     }
 298
 299     DestroyWindow(hwnd);
 300 }
 301
 302 /* demonstrates that subpopup's are locked
 303  * even after a client calls DestroyMenu on it */
 304 static LRESULT WINAPI subpopuplocked_wnd_proc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
 305 {
 306     HWND hwndmenu;
 307     switch (msg)
 308     {
 309     case WM_ENTERIDLE:
 310         hwndmenu = GetCapture();
 311         if( hwndmenu) {
 312             PostMessage( hwndmenu, WM_KEYDOWN, VK_DOWN, 0);
 313             PostMessage( hwndmenu, WM_KEYDOWN, VK_RIGHT, 0);
 314             PostMessage( hwndmenu, WM_KEYDOWN, VK_RETURN, 0);
 315         }
 316     }
 317     return DefWindowProc(hwnd, msg, wparam, lparam);
 318 }
 319
 320 static void test_subpopup_locked_by_menu(void)
 321 {
 322     DWORD gle;
 323     BOOL ret;
 324     HMENU hmenu, hsubmenu;
 325     MENUINFO mi = { sizeof( MENUINFO)};
 326     MENUITEMINFO mii = { sizeof( MENUITEMINFO)};
 327     HWND hwnd;
 328     const int itemid = 0x1234567;
 329     if( !pGetMenuInfo)
 330     {
 331         win_skip("GetMenuInfo is not available\n");
 332         return;
 333     }
 334     /* create window, popupmenu with one subpopup */
 335     hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
 336             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
 337             NULL, NULL, NULL, NULL);
 338     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
 339     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR) subpopuplocked_wnd_proc);
 340     hmenu = CreatePopupMenu();
 341     ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
 342     hsubmenu = CreatePopupMenu();
 343     ok(hsubmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
 344     ret = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hsubmenu,
 345             TEXT("PopUpLockTest"));
 346     ok(ret, "InsertMenu failed with error %d\n", GetLastError());
 347     ret = InsertMenu(hsubmenu, 0, MF_BYPOSITION | MF_STRING, itemid, TEXT("PopUpMenu"));
 348     ok(ret, "InsertMenu failed with error %d\n", GetLastError());
 349     /* first some tests that all this functions properly */
 350     mii.fMask = MIIM_SUBMENU;
 351     ret = GetMenuItemInfo( hmenu, 0, TRUE, &mii);
 352     ok( ret, "GetMenuItemInfo failed error %d\n", GetLastError());
 353     ok( mii.hSubMenu == hsubmenu, "submenu is %p\n", mii.hSubMenu);
 354     mi.fMask |= MIM_STYLE;
 355     ret = pGetMenuInfo( hsubmenu, &mi);
 356     ok( ret , "GetMenuInfo returned 0 with error %d\n", GetLastError());
 357     ret = IsMenu( hsubmenu);
 358     ok( ret , "Menu handle is not valid\n");
 359     SetLastError( 0xdeadbeef);
 360     ret = TrackPopupMenu( hmenu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
 361     if( ret == (itemid & 0xffff)) {
 362         win_skip("not on 16 bit menu subsystem\n");
 363         DestroyMenu( hsubmenu);
 364     } else {
 365         gle = GetLastError();
 366         ok( ret == itemid , "TrackPopupMenu returned %d error is %d\n", ret, gle);
 367         ok( gle == 0 ||
 368                 broken( gle == 0xdeadbeef), /* win2k0 */
 369                 "Last error is %d\n", gle);
 370         /* then destroy the sub-popup */
 371         ret = DestroyMenu( hsubmenu);
 372         ok(ret, "DestroyMenu failed with error %d\n", GetLastError());
 373         /* and repeat the tests */
 374         mii.fMask = MIIM_SUBMENU;
 375         ret = GetMenuItemInfo( hmenu, 0, TRUE, &mii);
 376         ok( ret, "GetMenuItemInfo failed error %d\n", GetLastError());
 377         /* GetMenuInfo fails now */
 378         ok( mii.hSubMenu == hsubmenu, "submenu is %p\n", mii.hSubMenu);
 379         mi.fMask |= MIM_STYLE;
 380         ret = pGetMenuInfo( hsubmenu, &mi);
 381         ok( !ret , "GetMenuInfo should have failed\n");
 382         /* IsMenu says it is not */
 383         ret = IsMenu( hsubmenu);
 384         ok( !ret , "Menu handle should be invalid\n");
 385         /* but TrackPopupMenu still works! */
 386         SetLastError( 0xdeadbeef);
 387         ret = TrackPopupMenu( hmenu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
 388         gle = GetLastError();
 389         todo_wine {
 390             ok( ret == itemid , "TrackPopupMenu returned %d error is %d\n", ret, gle);
 391         }
 392         ok( gle == 0 ||
 393             broken(gle == 0xdeadbeef) || /* wow64 */
 394             broken(gle == ERROR_INVALID_PARAMETER), /* win2k0 */
 395             "Last error is %d\n", gle);
 396     }
 397     /* clean up */
 398     DestroyMenu( hmenu);
 399     DestroyWindow(hwnd);
 400 }
 401
 402 static void test_menu_ownerdraw(void)
 403 {
 404     int i,j,k;
 405     BOOL ret;
 406     HMENU hmenu;
 407     LONG leftcol;
 408     HWND hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
 409                                WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
 410                                NULL, NULL, NULL, NULL);
 411     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
 412     if( !hwnd) return;
 413     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_ownerdraw_wnd_proc);
 414     hmenu = CreatePopupMenu();
 415     ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
 416     if( !hmenu) { DestroyWindow(hwnd);return;}
 417     k=0;
 418     for( j=0;j<2;j++) /* create columns */
 419         for(i=0;i<2;i++) { /* create rows */
 420             ret = AppendMenu( hmenu, MF_OWNERDRAW |
 421                               (i==0 ? MF_MENUBREAK : 0), k, MAKEINTRESOURCE(k));
 422             k++;
 423             ok( ret, "AppendMenu failed for %d\n", k-1);
 424         }
 425     MOD_maxid = k-1;
 426     assert( k <= sizeof(MOD_rc)/sizeof(RECT));
 427     /* display the menu */
 428     ret = TrackPopupMenu( hmenu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
 429
 430     /* columns have a 4 pixel gap between them */
 431     ok( MOD_rc[0].right + 4 ==  MOD_rc[2].left,
 432             "item rectangles are not separated by 4 pixels space\n");
 433     /* height should be what the MEASUREITEM message has returned */
 434     ok( MOD_rc[0].bottom - MOD_rc[0].top == MOD_SIZE,
 435             "menu item has wrong height: %d should be %d\n",
 436             MOD_rc[0].bottom - MOD_rc[0].top, MOD_SIZE);
 437     /* no gaps between the rows */
 438     ok( MOD_rc[0].bottom - MOD_rc[1].top == 0,
 439             "There should not be a space between the rows, gap is %d\n",
 440             MOD_rc[0].bottom - MOD_rc[1].top);
 441     /* test the correct value of the item height that was sent
 442      * by the WM_MEASUREITEM message */
 443     ok( MOD_odheight == HIWORD( GetDialogBaseUnits()) || /* WinNT,2k,XP */
 444             MOD_odheight == MOD_hic,                     /* Win95,98,ME */
 445             "Wrong height field in MEASUREITEMSTRUCT, expected %d or %d actual %d\n",
 446             HIWORD( GetDialogBaseUnits()), MOD_hic, MOD_odheight);
 447     /* test what MF_MENUBREAK did at the first position. Also show
 448      * that an MF_SEPARATOR is ignored in the height calculation. */
 449     leftcol= MOD_rc[0].left;
 450     ModifyMenu( hmenu, 0, MF_BYCOMMAND| MF_OWNERDRAW| MF_SEPARATOR, 0, 0);
 451     /* display the menu */
 452     ret = TrackPopupMenu( hmenu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
 453     /* left should be 4 pixels less now */
 454     ok( leftcol == MOD_rc[0].left + 4,
 455             "columns should be 4 pixels to the left (actual %d).\n",
 456             leftcol - MOD_rc[0].left);
 457     /* test width */
 458     ok( MOD_rc[0].right - MOD_rc[0].left == 2 * MOD_avec + MOD_SIZE,
 459             "width of owner drawn menu item is wrong. Got %d expected %d\n",
 460             MOD_rc[0].right - MOD_rc[0].left , 2*MOD_avec + MOD_SIZE);
 461     /* and height */
 462     ok( MOD_rc[0].bottom - MOD_rc[0].top == MOD_SIZE,
 463             "Height is incorrect. Got %d expected %d\n",
 464             MOD_rc[0].bottom - MOD_rc[0].top, MOD_SIZE);
 465
 466     /* test width/height of an ownerdraw menu bar as well */
 467     ret = DestroyMenu(hmenu);
 468     ok(ret, "DestroyMenu failed with error %d\n", GetLastError());
 469     hmenu = CreateMenu();
 470     ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
 471     if( !hmenu) { DestroyWindow(hwnd);return;}
 472     MOD_maxid=1;
 473     for(i=0;i<2;i++) {
 474         ret = AppendMenu( hmenu, MF_OWNERDRAW , i, 0);
 475         ok( ret, "AppendMenu failed for %d\n", i);
 476     }
 477     ret = SetMenu( hwnd, hmenu);
 478     UpdateWindow( hwnd); /* hack for wine to draw the window + menu */
 479     ok(ret, "SetMenu failed with error %d\n", GetLastError());
 480     /* test width */
 481     ok( MOD_rc[0].right - MOD_rc[0].left == 2 * MOD_avec + MOD_SIZE,
 482             "width of owner drawn menu item is wrong. Got %d expected %d\n",
 483             MOD_rc[0].right - MOD_rc[0].left , 2*MOD_avec + MOD_SIZE);
 484     /* test hight */
 485     ok( MOD_rc[0].bottom - MOD_rc[0].top == GetSystemMetrics( SM_CYMENU) - 1,
 486             "Height of owner drawn menu item is wrong. Got %d expected %d\n",
 487             MOD_rc[0].bottom - MOD_rc[0].top, GetSystemMetrics( SM_CYMENU) - 1);
 488
 489     /* clean up */
 490     ret = DestroyMenu(hmenu);
 491     ok(ret, "DestroyMenu failed with error %d\n", GetLastError());
 492     DestroyWindow(hwnd);
 493 }
 494
 495 /* helper for test_menu_bmp_and_string() */
 496 static void test_mbs_help( int ispop, int hassub, int mnuopt,
 497         HWND hwnd, int arrowwidth, int count, HBITMAP hbmp,
 498         SIZE bmpsize, LPCSTR text, SIZE size, SIZE sc_size)
 499 {
 500     BOOL ret;
 501     HMENU hmenu, submenu;
 502     MENUITEMINFO mii={ sizeof( MENUITEMINFO )};
 503     MENUINFO mi;
 504     RECT rc;
 505     CHAR text_copy[16];
 506     int hastab,  expect;
 507     int failed = 0;
 508
 509     MOD_GotDrawItemMsg = FALSE;
 510     mii.fMask = MIIM_FTYPE | MIIM_DATA | MIIM_STATE;
 511     mii.fType = 0;
 512     /* check the menu item unless MNS_CHECKORBMP is set */
 513     mii.fState = (mnuopt != 2 ? MFS_CHECKED : MFS_UNCHECKED);
 514     mii.dwItemData =0;
 515     MODsizes[0] = bmpsize;
 516     hastab = 0;
 517     if( text ) {
 518         char *p;
 519         mii.fMask |= MIIM_STRING;
 520         strcpy(text_copy, text);
 521         mii.dwTypeData = text_copy; /* structure member declared non-const */
 522         if( ( p = strchr( text, '\t'))) {
 523             hastab = *(p + 1) ? 2 : 1;
 524         }
 525     }
 526     /* tabs don't make sense in menubars */
 527     if(hastab && !ispop) return;
 528     if( hbmp) {
 529         mii.fMask |= MIIM_BITMAP;
 530         mii.hbmpItem = hbmp;
 531     }
 532     submenu = CreateMenu();
 533     ok( submenu != 0, "CreateMenu failed with error %d\n", GetLastError());
 534     if( ispop)
 535         hmenu = CreatePopupMenu();
 536     else
 537         hmenu = CreateMenu();
 538     ok( hmenu != 0, "Create{Popup}Menu failed with error %d\n", GetLastError());
 539     if( hassub) {
 540         mii.fMask |= MIIM_SUBMENU;
 541         mii.hSubMenu = submenu;
 542     }
 543     if( mnuopt) {
 544         mi.cbSize = sizeof(mi);
 545         mi.fMask = MIM_STYLE;
 546         pGetMenuInfo( hmenu, &mi);
 547         if( mnuopt) mi.dwStyle |= mnuopt == 1 ? MNS_NOCHECK : MNS_CHECKORBMP;
 548         ret = pSetMenuInfo( hmenu, &mi);
 549         ok( ret, "SetMenuInfo failed with error %d\n", GetLastError());
 550     }
 551     ret = InsertMenuItem( hmenu, 0, FALSE, &mii);
 552     ok( ret, "InsertMenuItem failed with error %d\n", GetLastError());
 553     failed = !ret;
 554     if( winetest_debug) {
 555         HDC hdc=GetDC(hwnd);
 556         RECT rc = {100, 50, 400, 70};
 557         char buf[100];
 558
 559         sprintf( buf,"%d text \"%s\" mnuopt %d", count, text ? text: "(nil)", mnuopt);
 560         FillRect( hdc, &rc, (HBRUSH) COLOR_WINDOW);
 561         TextOut( hdc, 10, 50, buf, strlen( buf));
 562         ReleaseDC( hwnd, hdc);
 563     }
 564     if(ispop)
 565         ret = TrackPopupMenu( hmenu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
 566     else {
 567         ret = SetMenu( hwnd, hmenu);
 568         ok(ret, "SetMenu failed with error %d\n", GetLastError());
 569         DrawMenuBar( hwnd);
 570     }
 571     ret = GetMenuItemRect( hwnd, hmenu, 0, &rc);
 572     if (0)  /* comment out menu size checks, behavior is different in almost every Windows version */
 573             /* the tests should however succeed on win2000, XP and Wine (at least up to 1.1.15) */
 574             /* with a variety of dpis and desktop font sizes */
 575     {
 576         /* check menu width */
 577         if( ispop)
 578             expect = ( text || hbmp ?
 579                        4 + (mnuopt != 1 ? GetSystemMetrics(SM_CXMENUCHECK) : 0)
 580                        : 0) +
 581                        arrowwidth  + MOD_avec + (hbmp ?
 582                                     ((INT_PTR)hbmp<0||(INT_PTR)hbmp>12 ? bmpsize.cx + 2 : GetSystemMetrics( SM_CXMENUSIZE) + 2)
 583                                     : 0) +
 584                 (text && hastab ? /* TAB space */
 585                  MOD_avec + ( hastab==2 ? sc_size.cx : 0) : 0) +
 586                 (text ?  2 + (text[0] ? size.cx :0): 0) ;
 587         else
 588             expect = !(text || hbmp) ? 0 :
 589                 ( hbmp ? (text ? 2:0) + bmpsize.cx  : 0 ) +
 590                 (text ? 2 * MOD_avec + (text[0] ? size.cx :0): 0) ;
 591         ok( rc.right - rc.left == expect,
 592             "menu width wrong, got %d expected %d\n", rc.right - rc.left, expect);
 593         failed = failed || !(rc.right - rc.left == expect);
 594         /* check menu height */
 595         if( ispop)
 596             expect = max( ( !(text || hbmp) ? GetSystemMetrics( SM_CYMENUSIZE)/2 : 0),
 597                           max( (text ? max( 2 + size.cy, MOD_hic + 4) : 0),
 598                                (hbmp ?
 599                                    ((INT_PTR)hbmp<0||(INT_PTR)hbmp>12 ?
 600                                        bmpsize.cy + 2
 601                                      : GetSystemMetrics( SM_CYMENUSIZE) + 2)
 602                                  : 0)));
 603         else
 604             expect = ( !(text || hbmp) ? GetSystemMetrics( SM_CYMENUSIZE)/2 :
 605                        max( GetSystemMetrics( SM_CYMENU) - 1, (hbmp ? bmpsize.cy : 0)));
 606         ok( rc.bottom - rc.top == expect,
 607             "menu height wrong, got %d expected %d (%d)\n",
 608             rc.bottom - rc.top, expect, GetSystemMetrics( SM_CYMENU));
 609         failed = failed || !(rc.bottom - rc.top == expect);
 610         if( hbmp == HBMMENU_CALLBACK && MOD_GotDrawItemMsg) {
 611             /* check the position of the bitmap */
 612             /* horizontal */
 613             if (!ispop)
 614                 expect = 3;
 615             else if (mnuopt == 0)
 616                 expect = 4 + GetSystemMetrics(SM_CXMENUCHECK);
 617             else if (mnuopt == 1)
 618                 expect = 4;
 619             else /* mnuopt == 2 */
 620                 expect = 2;
 621             ok( expect == MOD_rc[0].left,
 622                 "bitmap left is %d expected %d\n", MOD_rc[0].left, expect);
 623             failed = failed || !(expect == MOD_rc[0].left);
 624             /* vertical */
 625             expect = (rc.bottom - rc.top - MOD_rc[0].bottom + MOD_rc[0].top) / 2;
 626             ok( expect == MOD_rc[0].top,
 627                 "bitmap top is %d expected %d\n", MOD_rc[0].top, expect);
 628             failed = failed || !(expect == MOD_rc[0].top);
 629         }
 630     }
 631     /* if there was a failure, report details */
 632     if( failed) {
 633         trace("*** count %d %s text \"%s\" bitmap %p bmsize %d,%d textsize %d+%d,%d mnuopt %d hastab %d\n",
 634                 count, (ispop? "POPUP": "MENUBAR"),text ? text: "(nil)", hbmp, bmpsize.cx, bmpsize.cy,
 635                 size.cx, size.cy, sc_size.cx, mnuopt, hastab);
 636         trace("    check %d,%d arrow %d avechar %d\n",
 637                 GetSystemMetrics(SM_CXMENUCHECK ),
 638                 GetSystemMetrics(SM_CYMENUCHECK ),arrowwidth, MOD_avec);
 639         if( hbmp == HBMMENU_CALLBACK)
 640             trace( "    rc %d,%d-%d,%d bmp.rc %d,%d-%d,%d\n",
 641                 rc.left, rc.top, rc.top, rc.bottom, MOD_rc[0].left,
 642                 MOD_rc[0].top,MOD_rc[0].right, MOD_rc[0].bottom);
 643     }
 644     /* clean up */
 645     ret = DestroyMenu(submenu);
 646     ok(ret, "DestroyMenu failed with error %d\n", GetLastError());
 647     ret = DestroyMenu(hmenu);
 648     ok(ret, "DestroyMenu failed with error %d\n", GetLastError());
 649 }
 650
 651
 652 static void test_menu_bmp_and_string(void)
 653 {
 654     BYTE bmfill[300];
 655     HBITMAP hbm_arrow;
 656     BITMAP bm;
 657     INT arrowwidth;
 658     HWND hwnd;
 659     HMENU hsysmenu;
 660     MENUINFO mi= {sizeof(MENUINFO)};
 661     MENUITEMINFOA mii= {sizeof(MENUITEMINFOA)};
 662     int count, szidx, txtidx, bmpidx, hassub, mnuopt, ispop;
 663     BOOL got;
 664
 665     if( !pGetMenuInfo)
 666     {
 667         win_skip("GetMenuInfo is not available\n");
 668         return;
 669     }
 670
 671     memset( bmfill, 0xcc, sizeof( bmfill));
 672     hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL, WS_SYSMENU |
 673                           WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
 674                           NULL, NULL, NULL, NULL);
 675     hbm_arrow=LoadBitmap( 0, (CHAR*)OBM_MNARROW);
 676     GetObject( hbm_arrow, sizeof(bm), &bm);
 677     arrowwidth = bm.bmWidth;
 678     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
 679     if( !hwnd) return;
 680     /* test system menu */
 681     hsysmenu = GetSystemMenu( hwnd, FALSE);
 682     ok( hsysmenu != NULL, "GetSystemMenu failed with error %d\n", GetLastError());
 683     mi.fMask = MIM_STYLE;
 684     mi.dwStyle = 0;
 685     got = pGetMenuInfo( hsysmenu, &mi);
 686     ok( got, "GetMenuInfo failed gle=%d\n", GetLastError());
 687     ok( MNS_CHECKORBMP == mi.dwStyle, "System Menu Style is %08x, without the bit %08x\n",
 688         mi.dwStyle, MNS_CHECKORBMP);
 689     mii.fMask = MIIM_BITMAP;
 690     mii.hbmpItem = NULL;
 691     got = GetMenuItemInfoA( hsysmenu, SC_CLOSE, FALSE, &mii);
 692     ok( got, "GetMenuItemInfoA failed gle=%d\n", GetLastError());
 693     ok( HBMMENU_POPUP_CLOSE == mii.hbmpItem, "Item info did not get the right hbitmap: got %p  expected %p\n",
 694         mii.hbmpItem, HBMMENU_POPUP_CLOSE);
 695
 696     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_ownerdraw_wnd_proc);
 697
 698     if( winetest_debug)
 699         trace("    check %d,%d arrow %d avechar %d\n",
 700                 GetSystemMetrics(SM_CXMENUCHECK ),
 701                 GetSystemMetrics(SM_CYMENUCHECK ),arrowwidth, MOD_avec);
 702     count = 0;
 703     MOD_maxid = 0;
 704     for( ispop=1; ispop >= 0; ispop--){
 705         static SIZE bmsizes[]= {
 706             {10,10},{38,38},{1,30},{55,5}};
 707         for( szidx=0; szidx < sizeof( bmsizes) / sizeof( SIZE); szidx++) {
 708             HBITMAP hbm = CreateBitmap( bmsizes[szidx].cx, bmsizes[szidx].cy,1,1,bmfill);
 709             HBITMAP bitmaps[] = { HBMMENU_CALLBACK, hbm, HBMMENU_POPUP_CLOSE, NULL  };
 710             ok( hbm != 0, "CreateBitmap failed err %d\n", GetLastError());
 711             for( txtidx = 0; txtidx < sizeof(MOD_txtsizes)/sizeof(MOD_txtsizes[0]); txtidx++) {
 712                 for( hassub = 0; hassub < 2 ; hassub++) { /* add submenu item */
 713                     for( mnuopt = 0; mnuopt < 3 ; mnuopt++){ /* test MNS_NOCHECK/MNS_CHECKORBMP */
 714                         for( bmpidx = 0; bmpidx <sizeof(bitmaps)/sizeof(HBITMAP); bmpidx++) {
 715                             /* no need to test NULL bitmaps of several sizes */
 716                             if( !bitmaps[bmpidx] && szidx > 0) continue;
 717                             /* the HBMMENU_POPUP not to test for menu bars */
 718                             if( !ispop &&
 719                                 bitmaps[bmpidx] >= HBMMENU_POPUP_CLOSE &&
 720                                 bitmaps[bmpidx] <= HBMMENU_POPUP_MINIMIZE) continue;
 721                             if( !ispop && hassub) continue;
 722                             test_mbs_help( ispop, hassub, mnuopt,
 723                                     hwnd, arrowwidth, ++count,
 724                                     bitmaps[bmpidx],
 725                                     bmsizes[szidx],
 726                                     MOD_txtsizes[txtidx].text,
 727                                     MOD_txtsizes[txtidx].size,
 728                                     MOD_txtsizes[txtidx].sc_size);
 729                         }
 730                     }
 731                 }
 732             }
 733             DeleteObject( hbm);
 734         }
 735     }
 736     /* clean up */
 737     DestroyWindow(hwnd);
 738 }
 739
 740 static void test_menu_add_string( void )
 741 {
 742     HMENU hmenu;
 743     MENUITEMINFO info;
 744     BOOL rc;
 745     int ret;
 746
 747     char string[0x80];
 748     char string2[0x80];
 749
 750     char strback[0x80];
 751     WCHAR strbackW[0x80];
 752     static CHAR blah[] = "blah";
 753     static const WCHAR expectedString[] = {'D','u','m','m','y',' ','s','t','r','i','n','g', 0};
 754
 755     hmenu = CreateMenu();
 756
 757     memset( &info, 0, sizeof info );
 758     info.cbSize = sizeof info;
 759     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_STATE | MIIM_ID;
 760     info.dwTypeData = blah;
 761     info.cch = 6;
 762     info.dwItemData = 0;
 763     info.wID = 1;
 764     info.fState = 0;
 765     InsertMenuItem(hmenu, 0, TRUE, &info );
 766
 767     memset( &info, 0, sizeof info );
 768     info.cbSize = sizeof info;
 769     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_STATE | MIIM_DATA | MIIM_ID;
 770     info.dwTypeData = string;
 771     info.cch = sizeof string;
 772     string[0] = 0;
 773     GetMenuItemInfo( hmenu, 0, TRUE, &info );
 774
 775     ok( !strcmp( string, "blah" ), "menu item name differed\n");
 776
 777     /* Test combination of ownerdraw and strings with GetMenuItemString(A/W) */
 778     strcpy(string, "Dummy string");
 779     memset(&info, 0x00, sizeof(info));
 780     info.cbSize= sizeof(MENUITEMINFO);
 781     info.fMask= MIIM_FTYPE | MIIM_STRING; /* Set OwnerDraw + typeData */
 782     info.fType= MFT_OWNERDRAW;
 783     info.dwTypeData= string;
 784     rc = InsertMenuItem( hmenu, 0, TRUE, &info );
 785     ok (rc, "InsertMenuItem failed\n");
 786
 787     strcpy(string,"Garbage");
 788     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 789     ok (!strcmp( strback, "Dummy string" ), "Menu text from Ansi version incorrect\n");
 790
 791     SetLastError(0xdeadbeef);
 792     ret = GetMenuStringW( hmenu, 0, strbackW, 99, MF_BYPOSITION );
 793     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
 794         win_skip("GetMenuStringW is not implemented\n");
 795     else
 796     {
 797         ok (ret, "GetMenuStringW on ownerdraw entry failed\n");
 798         ok (!lstrcmpW( strbackW, expectedString ), "Menu text from Unicode version incorrect\n");
 799     }
 800
 801     /* Just change ftype to string and see what text is stored */
 802     memset(&info, 0x00, sizeof(info));
 803     info.cbSize= sizeof(MENUITEMINFO);
 804     info.fMask= MIIM_FTYPE; /* Set string type */
 805     info.fType= MFT_STRING;
 806     info.dwTypeData= (char *)0xdeadbeef;
 807     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 808     ok (rc, "SetMenuItemInfo failed\n");
 809
 810     /* Did we keep the old dwTypeData? */
 811     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 812     ok (!strcmp( strback, "Dummy string" ), "Menu text from Ansi version incorrect\n");
 813
 814     /* Ensure change to bitmap type fails */
 815     memset(&info, 0x00, sizeof(info));
 816     info.cbSize= sizeof(MENUITEMINFO);
 817     info.fMask= MIIM_FTYPE; /* Set as bitmap type */
 818     info.fType= MFT_BITMAP;
 819     info.dwTypeData= (char *)0xdeadbee2;
 820     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 821     ok (!rc, "SetMenuItemInfo unexpectedly worked\n");
 822
 823     /* Just change ftype back and ensure data hasn't been freed */
 824     info.fType= MFT_OWNERDRAW; /* Set as ownerdraw type */
 825     info.dwTypeData= (char *)0xdeadbee3;
 826     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 827     ok (rc, "SetMenuItemInfo failed\n");
 828
 829     /* Did we keep the old dwTypeData? */
 830     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 831     ok (!strcmp( strback, "Dummy string" ), "Menu text from Ansi version incorrect\n");
 832
 833     /* Just change string value (not type) */
 834     memset(&info, 0x00, sizeof(info));
 835     info.cbSize= sizeof(MENUITEMINFO);
 836     info.fMask= MIIM_STRING; /* Set typeData */
 837     strcpy(string2, "string2");
 838     info.dwTypeData= string2;
 839     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 840     ok (rc, "SetMenuItemInfo failed\n");
 841
 842     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 843     ok (!strcmp( strback, "string2" ), "Menu text from Ansi version incorrect\n");
 844
 845     /*  crashes with wine 0.9.5 */
 846     memset(&info, 0x00, sizeof(info));
 847     info.cbSize= sizeof(MENUITEMINFO);
 848     info.fMask= MIIM_FTYPE | MIIM_STRING; /* Set OwnerDraw + typeData */
 849     info.fType= MFT_OWNERDRAW;
 850     rc = InsertMenuItem( hmenu, 0, TRUE, &info );
 851     ok (rc, "InsertMenuItem failed\n");
 852     ok (!GetMenuString( hmenu, 0, NULL, 0, MF_BYPOSITION),
 853             "GetMenuString on ownerdraw entry succeeded.\n");
 854     SetLastError(0xdeadbeef);
 855     ret = GetMenuStringW( hmenu, 0, NULL, 0, MF_BYPOSITION);
 856     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
 857         win_skip("GetMenuStringW is not implemented\n");
 858     else
 859         ok (!ret, "GetMenuStringW on ownerdraw entry succeeded.\n");
 860
 861     DestroyMenu( hmenu );
 862 }
 863
 864 /* define building blocks for the menu item info tests */
 865 static int strncmpW( const WCHAR *str1, const WCHAR *str2, int n )
 866 {
 867     if (n <= 0) return 0;
 868     while ((--n > 0) && *str1 && (*str1 == *str2)) { str1++; str2++; }
 869     return *str1 - *str2;
 870 }
 871
 872 static  WCHAR *strcpyW( WCHAR *dst, const WCHAR *src )
 873 {
 874     WCHAR *p = dst;
 875     while ((*p++ = *src++));
 876     return dst;
 877 }
 878
 879 static void insert_menu_item( int line, HMENU hmenu, BOOL ansi, UINT mask, UINT type, UINT state, UINT id,
 880                               HMENU submenu, HBITMAP checked, HBITMAP unchecked, ULONG_PTR data,
 881                               void *type_data, UINT len, HBITMAP item, BOOL expect )
 882 {
 883     MENUITEMINFOA info;
 884     BOOL ret;
 885
 886     /* magic bitmap handle to test smaller cbSize */
 887     if (item == (HBITMAP)(ULONG_PTR)0xdeadbeef)
 888         info.cbSize = FIELD_OFFSET(MENUITEMINFOA,hbmpItem);
 889     else
 890         info.cbSize = sizeof(info);
 891     info.fMask = mask;
 892     info.fType = type;
 893     info.fState = state;
 894     info.wID = id;
 895     info.hSubMenu = submenu;
 896     info.hbmpChecked = checked;
 897     info.hbmpUnchecked = unchecked;
 898     info.dwItemData = data;
 899     info.dwTypeData = type_data;
 900     info.cch = len;
 901     info.hbmpItem = item;
 902     SetLastError( 0xdeadbeef );
 903     if (ansi) ret = InsertMenuItemA( hmenu, 0, TRUE, &info );
 904     else ret = InsertMenuItemW( hmenu, 0, TRUE, (MENUITEMINFOW*)&info );
 905     if (!expect) ok_(__FILE__, line)( !ret, "InsertMenuItem should have failed.\n" );
 906     else ok_(__FILE__, line)( ret, "InsertMenuItem failed, err %u\n", GetLastError());
 907 }
 908
 909 static void check_menu_item_info( int line, HMENU hmenu, BOOL ansi, UINT mask, UINT type, UINT state,
 910                                   UINT id, HMENU submenu, HBITMAP checked, HBITMAP unchecked,
 911                                   ULONG_PTR data, void *type_data, UINT in_len, UINT out_len,
 912                                   HBITMAP item, LPCSTR expname, BOOL expect, BOOL expstring )
 913 {
 914     MENUITEMINFOA info;
 915     BOOL ret;
 916     WCHAR buffer[80];
 917
 918     SetLastError( 0xdeadbeef );
 919     memset( &info, 0xcc, sizeof(info) );
 920     info.cbSize = sizeof(info);
 921     info.fMask = mask;
 922     info.dwTypeData = type_data;
 923     info.cch = in_len;
 924
 925     ret = ansi ? GetMenuItemInfoA( hmenu, 0, TRUE, &info ) :
 926                  GetMenuItemInfoW( hmenu, 0, TRUE, (MENUITEMINFOW *)&info );
 927     if (!expect)
 928     {
 929         ok_(__FILE__, line)( !ret, "GetMenuItemInfo should have failed.\n" );
 930         return;
 931     }
 932     ok_(__FILE__, line)( ret, "GetMenuItemInfo failed, err %u\n", GetLastError());
 933     if (mask & MIIM_TYPE)
 934         ok_(__FILE__, line)( info.fType == type || info.fType == LOWORD(type),
 935                              "wrong type %x/%x\n", info.fType, type );
 936     if (mask & MIIM_STATE)
 937         ok_(__FILE__, line)( info.fState == state || info.fState == LOWORD(state),
 938                              "wrong state %x/%x\n", info.fState, state );
 939     if (mask & MIIM_ID)
 940         ok_(__FILE__, line)( info.wID == id || info.wID == LOWORD(id),
 941                              "wrong id %x/%x\n", info.wID, id );
 942     if (mask & MIIM_SUBMENU)
 943         ok_(__FILE__, line)( info.hSubMenu == submenu || (ULONG_PTR)info.hSubMenu == LOWORD(submenu),
 944                              "wrong submenu %p/%p\n", info.hSubMenu, submenu );
 945     if (mask & MIIM_CHECKMARKS)
 946     {
 947         ok_(__FILE__, line)( info.hbmpChecked == checked || (ULONG_PTR)info.hbmpChecked == LOWORD(checked),
 948                              "wrong bmpchecked %p/%p\n", info.hbmpChecked, checked );
 949         ok_(__FILE__, line)( info.hbmpUnchecked == unchecked || (ULONG_PTR)info.hbmpUnchecked == LOWORD(unchecked),
 950                              "wrong bmpunchecked %p/%p\n", info.hbmpUnchecked, unchecked );
 951     }
 952     if (mask & MIIM_DATA)
 953         ok_(__FILE__, line)( info.dwItemData == data || info.dwItemData == LOWORD(data),
 954                              "wrong item data %lx/%lx\n", info.dwItemData, data );
 955     if (mask & MIIM_BITMAP)
 956         ok_(__FILE__, line)( info.hbmpItem == item || (ULONG_PTR)info.hbmpItem == LOWORD(item),
 957                              "wrong bmpitem %p/%p\n", info.hbmpItem, item );
 958     ok_(__FILE__, line)( info.dwTypeData == type_data || (ULONG_PTR)info.dwTypeData == LOWORD(type_data),
 959                          "wrong type data %p/%p\n", info.dwTypeData, type_data );
 960     ok_(__FILE__, line)( info.cch == out_len ||
 961                          broken(! ansi && info.cch == 2 * out_len) /* East-Asian */,
 962                          "wrong len %x/%x\n", info.cch, out_len );
 963     if (expname)
 964     {
 965         if(ansi)
 966             ok_(__FILE__, line)( !strncmp( expname, info.dwTypeData, out_len ),
 967                                  "menu item name differed from '%s' '%s'\n", expname, info.dwTypeData );
 968         else
 969             ok_(__FILE__, line)( !strncmpW( (WCHAR *)expname, (WCHAR *)info.dwTypeData, out_len ),
 970                                  "menu item name wrong\n" );
 971
 972         SetLastError( 0xdeadbeef );
 973         ret = ansi ? GetMenuStringA( hmenu, 0, (char *)buffer, 80, MF_BYPOSITION ) :
 974             GetMenuStringW( hmenu, 0, buffer, 80, MF_BYPOSITION );
 975         if (expstring)
 976             ok_(__FILE__, line)( ret, "GetMenuString failed, err %u\n", GetLastError());
 977         else
 978             ok_(__FILE__, line)( !ret, "GetMenuString should have failed\n" );
 979     }
 980 }
 981
 982 static void modify_menu( int line, HMENU hmenu, BOOL ansi, UINT flags, UINT_PTR id, void *data )
 983 {
 984     BOOL ret;
 985
 986     SetLastError( 0xdeadbeef );
 987     if (ansi) ret = ModifyMenuA( hmenu, 0, flags, id, data );
 988     else ret = ModifyMenuW( hmenu, 0, flags, id, data );
 989     ok_(__FILE__,line)( ret, "ModifyMenuA failed, err %u\n", GetLastError());
 990 }
 991
 992 static void set_menu_item_info( int line, HMENU hmenu, BOOL ansi, UINT mask, UINT type, UINT state,
 993                                 UINT id, HMENU submenu, HBITMAP checked, HBITMAP unchecked, ULONG_PTR data,
 994                                 void *type_data, UINT len, HBITMAP item )
 995
 996 {
 997     MENUITEMINFOA info;
 998     BOOL ret;
 999
1000     /* magic bitmap handle to test smaller cbSize */
1001     if (item == (HBITMAP)(ULONG_PTR)0xdeadbeef)
1002         info.cbSize = FIELD_OFFSET(MENUITEMINFOA,hbmpItem);
1003     else
1004         info.cbSize = sizeof(info);
1005     info.fMask = mask;
1006     info.fType = type;
1007     info.fState = state;
1008     info.wID = id;
1009     info.hSubMenu = submenu;
1010     info.hbmpChecked = checked;
1011     info.hbmpUnchecked = unchecked;
1012     info.dwItemData = data;
1013     info.dwTypeData = type_data;
1014     info.cch = len;
1015     info.hbmpItem = item;
1016     SetLastError( 0xdeadbeef );
1017     if (ansi) ret = SetMenuItemInfoA( hmenu, 0, TRUE, &info );
1018     else ret = SetMenuItemInfoW( hmenu, 0, TRUE, (MENUITEMINFOW*)&info );
1019     ok_(__FILE__, line)( ret, "SetMenuItemInfo failed, err %u\n", GetLastError());
1020 }
1021
1022 #define TMII_INSMI( c1,d1,e1,f1,g1,h1,i1,j1,k1,l1,m1,eret1 )\
1023     hmenu = CreateMenu();\
1024     submenu = CreateMenu();\
1025     if(ansi)strcpy( string, init );\
1026     else strcpyW( string, init );\
1027     insert_menu_item( __LINE__, hmenu, ansi, c1, d1, e1, f1, g1, h1, i1, j1, k1, l1, m1, eret1 )
1028
1029 /* GetMenuItemInfo + GetMenuString  */
1030 #define TMII_GMII( c2,l2,\
1031     d3,e3,f3,g3,h3,i3,j3,k3,l3,m3,\
1032     expname, eret2, eret3)\
1033     check_menu_item_info( __LINE__, hmenu, ansi, c2, d3, e3, f3, g3, h3, i3, j3, k3, l2, l3, m3, \
1034                           expname, eret2, eret3 )
1035
1036 #define TMII_DONE \
1037     RemoveMenu(hmenu, 0, TRUE );\
1038     DestroyMenu( hmenu );\
1039     DestroyMenu( submenu );
1040
1041 /* modify menu */
1042 #define TMII_MODM( flags, id, data ) \
1043     modify_menu( __LINE__, hmenu, ansi, flags, id, data )
1044
1045 /* SetMenuItemInfo */
1046 #define TMII_SMII( c1,d1,e1,f1,g1,h1,i1,j1,k1,l1,m1 ) \
1047     set_menu_item_info( __LINE__, hmenu, ansi, c1, d1, e1, f1, g1, h1, i1, j1, k1, l1, m1 )
1048
1049
1050 #define OK 1
1051 #define ER 0
1052
1053
1054 static void test_menu_iteminfo( void )
1055 {
1056   int ansi = TRUE;
1057   char txtA[]="wine";
1058   char initA[]="XYZ";
1059   char emptyA[]="";
1060   WCHAR txtW[]={'W','i','n','e',0};
1061   WCHAR initW[]={'X','Y','Z',0};
1062   WCHAR emptyW[]={0};
1063   void *txt, *init, *empty, *string;
1064   HBITMAP hbm = CreateBitmap(1,1,1,1,NULL);
1065   char stringA[0x80];
1066   HMENU hmenu, submenu=CreateMenu();
1067   HBITMAP dummy_hbm = (HBITMAP)(ULONG_PTR)0xdeadbeef;
1068
1069   do {
1070     if( ansi) {txt=txtA;init=initA;empty=emptyA;string=stringA;}
1071     else {txt=txtW;init=initW;empty=emptyW;string=stringA;}
1072     trace( "%s string %p hbm %p txt %p\n", ansi ?  "ANSI tests:   " : "Unicode tests:", string, hbm, txt);
1073     /* test all combinations of MFT_STRING, MFT_OWNERDRAW and MFT_BITMAP */
1074     /* (since MFT_STRING is zero, there are four of them) */
1075     TMII_INSMI( MIIM_TYPE, MFT_STRING, 0, 0, 0, 0, 0, 0, txt, 0, 0, OK );
1076     TMII_GMII ( MIIM_TYPE, 80,
1077         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1078         txt, OK, OK );
1079     TMII_DONE
1080     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1081     TMII_GMII ( MIIM_TYPE, 80,
1082         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1083         NULL, OK, ER );
1084     TMII_DONE
1085     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, 6, 0, OK );
1086     TMII_GMII ( MIIM_TYPE, 80,
1087         MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1088         NULL, OK, ER );
1089     TMII_DONE
1090     TMII_INSMI( MIIM_TYPE, MFT_BITMAP|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, hbm, 6, 0, OK );
1091     TMII_GMII ( MIIM_TYPE, 80,
1092         MFT_BITMAP|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1093         NULL, OK, ER );
1094     TMII_DONE
1095     /* not enough space for name*/
1096     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1097     TMII_GMII ( MIIM_TYPE, 0,
1098         MFT_STRING, 0, 0, 0, 0, 0, 0, NULL, 4, 0,
1099         NULL, OK, OK );
1100     TMII_DONE
1101     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1102     TMII_GMII ( MIIM_TYPE, 5,
1103         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1104         txt, OK, OK );
1105     TMII_DONE
1106     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1107     TMII_GMII ( MIIM_TYPE, 4,
1108         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 3, 0,
1109         txt, OK, OK );
1110     TMII_DONE
1111     TMII_INSMI( MIIM_FTYPE|MIIM_STRING, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1112     TMII_GMII ( MIIM_TYPE, 0,
1113         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 0, 0,
1114         NULL, OK, ER );
1115     TMII_DONE
1116     /* cannot combine MIIM_TYPE with some other flags */
1117     TMII_INSMI( MIIM_TYPE|MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, ER );
1118     TMII_DONE
1119     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1120     TMII_GMII ( MIIM_TYPE|MIIM_STRING, 80,
1121         0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1122         NULL, ER, OK );
1123     TMII_DONE
1124     TMII_INSMI( MIIM_TYPE|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, ER );
1125     TMII_DONE
1126     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1127     TMII_GMII ( MIIM_TYPE|MIIM_FTYPE, 80,
1128         0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1129         NULL, ER, OK );
1130     TMII_DONE
1131     TMII_INSMI( MIIM_TYPE|MIIM_BITMAP, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, 6, hbm, ER );
1132     TMII_DONE
1133         /* but succeeds with some others */
1134     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1135     TMII_GMII ( MIIM_TYPE|MIIM_SUBMENU, 80,
1136         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1137         txt, OK, OK );
1138     TMII_DONE
1139     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1140     TMII_GMII ( MIIM_TYPE|MIIM_STATE, 80,
1141         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1142         txt, OK, OK );
1143     TMII_DONE
1144     TMII_INSMI( MIIM_TYPE|MIIM_ID, MFT_STRING, -1, 888, 0, 0, 0, -1, txt, 6, 0, OK );
1145     TMII_GMII ( MIIM_TYPE|MIIM_ID, 80,
1146         MFT_STRING, 0, 888, 0, 0, 0, 0, string, 4, 0,
1147         txt, OK, OK );
1148     TMII_DONE
1149     TMII_INSMI( MIIM_TYPE|MIIM_DATA, MFT_STRING, -1, -1, 0, 0, 0, 999, txt, 6, 0, OK );
1150     TMII_GMII ( MIIM_TYPE|MIIM_DATA, 80,
1151         MFT_STRING, 0, 0, 0, 0, 0, 999, string, 4, 0,
1152         txt, OK, OK );
1153     TMII_DONE
1154     /* to be continued */
1155     /* set text with MIIM_TYPE and retrieve with MIIM_STRING */
1156     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1157     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1158         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1159         txt, OK, OK );
1160     TMII_DONE
1161     /* set text with MIIM_TYPE and retrieve with MIIM_STRING; MFT_OWNERDRAW causes an empty string */
1162     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1163     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1164         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 0, 0,
1165         empty, OK, ER );
1166     TMII_DONE
1167     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1168     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1169         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 0, 0,
1170         empty, OK, ER );
1171     TMII_DONE
1172     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1173     TMII_GMII ( MIIM_FTYPE, 80,
1174         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 80, 0,
1175         init, OK, ER );
1176     TMII_DONE
1177     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1178     TMII_GMII ( 0, 80,
1179         0, 0, 0, 0, 0, 0, 0, string, 80, 0,
1180         init, OK, OK );
1181     TMII_DONE
1182     /* contrary to MIIM_TYPE,you can set the text for an owner draw menu */
1183     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1184     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1185         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 4, 0,
1186         txt, OK, OK );
1187     TMII_DONE
1188     /* same but retrieve with MIIM_TYPE */
1189     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1190     TMII_GMII ( MIIM_TYPE, 80,
1191         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1192         NULL, OK, OK );
1193     TMII_DONE
1194     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1195     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1196         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 0, 0,
1197         empty, OK, ER );
1198     TMII_DONE
1199     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1200     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1201         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 0, 0,
1202         empty, OK, ER );
1203     TMII_DONE
1204
1205     /* How is that with bitmaps? */
1206     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1207     TMII_GMII ( MIIM_TYPE, 80,
1208         MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1209         NULL, OK, ER );
1210     TMII_DONE
1211     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1212     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1213         0, 0, 0, 0, 0, 0, 0, string, 80, hbm,
1214         init, OK, ER );
1215     TMII_DONE
1216         /* MIIM_BITMAP does not like MFT_BITMAP */
1217     TMII_INSMI( MIIM_BITMAP|MIIM_FTYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, 0, -1, hbm, ER );
1218     TMII_DONE
1219         /* no problem with OWNERDRAWN */
1220     TMII_INSMI( MIIM_BITMAP|MIIM_FTYPE, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1221     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1222         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 80, hbm,
1223         init, OK, ER );
1224     TMII_DONE
1225         /* setting MFT_BITMAP with MFT_FTYPE fails anyway */
1226     TMII_INSMI( MIIM_FTYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, 0, -1, 0, ER );
1227     TMII_DONE
1228
1229     /* menu with submenu */
1230     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, submenu, 0, 0, -1, txt, 0, 0, OK );
1231     TMII_GMII ( MIIM_SUBMENU, 80,
1232         0, 0, 0, submenu, 0, 0, 0, string, 80, 0,
1233         init, OK, ER );
1234     TMII_DONE
1235     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, submenu, 0, 0, -1, empty, 0, 0, OK );
1236     TMII_GMII ( MIIM_SUBMENU, 80,
1237         0, 0, 0, submenu, 0, 0, 0, string, 80, 0,
1238         init, OK, ER );
1239     TMII_DONE
1240     /* menu with submenu, without MIIM_SUBMENU the submenufield is cleared */
1241     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, submenu, 0, 0, -1, txt, 0, 0, OK );
1242     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1243         MFT_STRING|MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 0, 0,
1244         empty, OK, ER );
1245     TMII_GMII ( MIIM_SUBMENU|MIIM_FTYPE, 80,
1246         MFT_SEPARATOR, 0, 0, submenu, 0, 0, 0, string, 80, 0,
1247         empty, OK, ER );
1248     TMII_DONE
1249     /* menu with invalid submenu */
1250     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, (HMENU)999, 0, 0, -1, txt, 0, 0, ER );
1251     TMII_DONE
1252     /* Separator */
1253     TMII_INSMI( MIIM_TYPE, MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, txt, 0, 0, OK );
1254     TMII_GMII ( MIIM_TYPE, 80,
1255         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1256         NULL, OK, ER );
1257     TMII_DONE
1258     TMII_INSMI( MIIM_TYPE, MFT_BITMAP|MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, hbm, 6, 0, OK );
1259     TMII_GMII ( MIIM_TYPE, 80,
1260         MFT_BITMAP|MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1261         NULL, OK, ER );
1262     TMII_DONE
1263      /* SEPARATOR and STRING go well together */
1264     /* BITMAP and STRING go well together */
1265     TMII_INSMI( MIIM_STRING|MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1266     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, 80,
1267         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, hbm,
1268         txt, OK, OK );
1269     TMII_DONE
1270      /* BITMAP, SEPARATOR and STRING go well together */
1271     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1272     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, 80,
1273         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 4, hbm,
1274         txt, OK, OK );
1275     TMII_DONE
1276      /* last two tests, but use MIIM_TYPE to retrieve info */
1277     TMII_INSMI( MIIM_FTYPE|MIIM_STRING, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1278     TMII_GMII ( MIIM_TYPE, 80,
1279         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1280         NULL, OK, OK );
1281     TMII_DONE
1282     TMII_INSMI( MIIM_STRING|MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1283     TMII_GMII ( MIIM_TYPE, 80,
1284         MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1285         NULL, OK, OK );
1286     TMII_DONE
1287     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1288     TMII_GMII ( MIIM_TYPE, 80,
1289         MFT_SEPARATOR|MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1290         NULL, OK, OK );
1291     TMII_DONE
1292      /* same three with MFT_OWNERDRAW */
1293     TMII_INSMI( MIIM_FTYPE|MIIM_STRING, MFT_SEPARATOR|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1294     TMII_GMII ( MIIM_TYPE, 80,
1295         MFT_SEPARATOR|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1296         NULL, OK, OK );
1297     TMII_DONE
1298     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1299     TMII_GMII ( MIIM_TYPE, 80,
1300         MFT_BITMAP|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1301         NULL, OK, OK );
1302     TMII_DONE
1303     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1304     TMII_GMII ( MIIM_TYPE, 80,
1305         MFT_SEPARATOR|MFT_BITMAP|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1306         NULL, OK, OK );
1307     TMII_DONE
1308
1309     TMII_INSMI( MIIM_STRING|MIIM_FTYPE|MIIM_ID, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1310     TMII_GMII ( MIIM_TYPE, 80,
1311         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1312         NULL,  OK, OK );
1313     TMII_DONE
1314     /* test with modifymenu: string is preserved after setting OWNERDRAW */
1315     TMII_INSMI( MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1316     TMII_MODM( MFT_OWNERDRAW, -1, (void*)787 );
1317     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_DATA, 80,
1318         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 787, string, 4, 0,
1319         txt,  OK, OK );
1320     TMII_DONE
1321     /* same with bitmap: now the text is cleared */
1322     TMII_INSMI( MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1323     TMII_MODM( MFT_BITMAP, 545, hbm );
1324     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1325         MFT_BITMAP, 0, 545, 0, 0, 0, 0, string, 0, hbm,
1326         empty,  OK, ER );
1327     TMII_DONE
1328     /* start with bitmap: now setting text clears it (though he flag is raised) */
1329     TMII_INSMI( MIIM_BITMAP, MFT_STRING, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1330     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1331         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 0, hbm,
1332         empty,  OK, ER );
1333     TMII_MODM( MFT_STRING, 545, txt );
1334     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1335         MFT_STRING, 0, 545, 0, 0, 0, 0, string, 4, 0,
1336         txt,  OK, OK );
1337     TMII_DONE
1338     /*repeat with text NULL */
1339     TMII_INSMI( MIIM_BITMAP, MFT_STRING, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1340     TMII_MODM( MFT_STRING, 545, NULL );
1341     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1342         MFT_SEPARATOR, 0, 545, 0, 0, 0, 0, string, 0, 0,
1343         empty,  OK, ER );
1344     TMII_DONE
1345     /* repeat with text "" */
1346     TMII_INSMI( MIIM_BITMAP, -1 , -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1347     TMII_MODM( MFT_STRING, 545, empty );
1348     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1349         MFT_STRING, 0, 545, 0, 0, 0, 0, string, 0, 0,
1350         empty,  OK, ER );
1351     TMII_DONE
1352     /* start with bitmap: set ownerdraw */
1353     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1354     TMII_MODM( MFT_OWNERDRAW, -1, (void *)232 );
1355     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_DATA, 80,
1356         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 232, string, 0, hbm,
1357         empty,  OK, ER );
1358     TMII_DONE
1359     /* ask nothing */
1360     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1361     TMII_GMII ( 0, 80,
1362                 0, 0, 0,  0, 0, 0, 0, string, 80, 0,
1363         init, OK, OK );
1364     TMII_DONE
1365     /* some tests with small cbSize: the hbmpItem is to be ignored */
1366     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, dummy_hbm, OK );
1367     TMII_GMII ( MIIM_TYPE, 80,
1368         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, NULL, 0, NULL,
1369         NULL, OK, ER );
1370     TMII_DONE
1371     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, dummy_hbm, OK );
1372     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1373         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 80, NULL,
1374         init, OK, ER );
1375     TMII_DONE
1376     TMII_INSMI( MIIM_STRING|MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1377     TMII_GMII ( MIIM_TYPE, 80,
1378         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, NULL,
1379         txt, OK, OK );
1380     TMII_DONE
1381     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1382     TMII_GMII ( MIIM_TYPE, 80,
1383         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1384         NULL, OK, OK );
1385     TMII_DONE
1386     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1387     TMII_GMII ( MIIM_TYPE, 80,
1388         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1389         NULL, OK, OK );
1390     TMII_DONE
1391     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1392     TMII_GMII ( MIIM_TYPE, 80,
1393         MFT_SEPARATOR|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1394         NULL, OK, OK );
1395     TMII_DONE
1396     /* MIIM_TYPE by itself does not get/set the dwItemData for OwnerDrawn menus  */
1397     TMII_INSMI( MIIM_TYPE|MIIM_DATA, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, 343, txt, 0, 0, OK );
1398     TMII_GMII ( MIIM_TYPE|MIIM_DATA, 80,
1399         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 343, 0, 0, 0,
1400         NULL, OK, ER );
1401     TMII_DONE
1402     TMII_INSMI( MIIM_TYPE|MIIM_DATA, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, 343, txt, 0, 0, OK );
1403     TMII_GMII ( MIIM_TYPE, 80,
1404         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1405         NULL, OK, ER );
1406     TMII_DONE
1407     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, 343, txt, 0, 0, OK );
1408     TMII_GMII ( MIIM_TYPE|MIIM_DATA, 80,
1409         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1410         NULL, OK, ER );
1411     TMII_DONE
1412     /* set a string menu to ownerdraw with MIIM_TYPE */
1413     TMII_INSMI( MIIM_TYPE, MFT_STRING, -2, -2, 0, 0, 0, -2, txt, -2, 0, OK );
1414     TMII_SMII ( MIIM_TYPE, MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0 );
1415     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1416         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 4, 0,
1417         txt, OK, OK );
1418     TMII_DONE
1419     /* test with modifymenu add submenu */
1420     TMII_INSMI( MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1421     TMII_MODM( MF_POPUP, (UINT_PTR)submenu, txt );
1422     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_SUBMENU, 80,
1423         MFT_STRING, 0, 0, submenu, 0, 0, 0, string, 4, 0,
1424         txt,  OK, OK );
1425     TMII_GMII ( MIIM_TYPE, 80,
1426         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1427         txt,  OK, OK );
1428     TMII_DONE
1429     /* MFT_SEPARATOR bit is kept when the text is added */
1430     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1431     TMII_SMII( MIIM_STRING, 0, 0, 0, 0, 0, 0, 0, txt, 0, 0 );
1432     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1433         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 4, 0,
1434         txt, OK, OK );
1435     TMII_DONE
1436     /* MFT_SEPARATOR bit is kept when bitmap is added */
1437     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1438     TMII_SMII( MIIM_BITMAP, 0, 0, 0, 0, 0, 0, 0, 0, 0, hbm );
1439     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1440         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 80, hbm,
1441         init, OK, ER );
1442     TMII_DONE
1443     /* Bitmaps inserted with MIIM_TYPE and MFT_BITMAP:
1444        Only the low word of the dwTypeData is used.
1445        Use a magic bitmap here (Word 95 uses this to create its MDI menu buttons) */
1446     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_RIGHTJUSTIFY, -1, -1, 0, 0, 0, -1,
1447                 (HMENU)MAKELONG(HBMMENU_MBAR_CLOSE, 0x1234), -1, 0, OK );
1448     TMII_GMII ( MIIM_TYPE, 80,
1449         MFT_BITMAP | MFT_RIGHTJUSTIFY, 0, 0, 0, 0, 0, 0, HBMMENU_MBAR_CLOSE, 0, HBMMENU_MBAR_CLOSE,
1450         NULL, OK, OK );
1451     TMII_DONE
1452     /* Type flags */
1453     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1454     TMII_GMII ( MIIM_TYPE, 80,
1455         MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1456         NULL, OK, OK );
1457     TMII_DONE
1458     /* State flags */
1459     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1460     TMII_SMII( MIIM_STATE, -1, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 0, 0 );
1461     TMII_GMII ( MIIM_STATE, 80,
1462         0, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 80, 0,
1463         NULL, OK, OK );
1464     TMII_DONE
1465     /* The style MFT_RADIOCHECK cannot be set with MIIM_CHECKMARKS only */
1466     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1467     TMII_SMII( MIIM_CHECKMARKS, MFT_RADIOCHECK, 0, 0, 0, hbm, hbm, 0, 0, 0, 0 );
1468     TMII_GMII ( MIIM_CHECKMARKS | MIIM_TYPE, 80,
1469         MFT_BITMAP, 0, 0, 0, hbm, hbm, 0, hbm, 0, hbm,
1470         NULL, OK, OK );
1471     TMII_DONE
1472     /* MFT_BITMAP is added automatically by GetMenuItemInfo() for MIIM_TYPE */
1473     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1474     TMII_SMII( MIIM_FTYPE, MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, (HBITMAP)0x1234, 0, 0 );
1475     TMII_GMII ( MIIM_FTYPE, 80,
1476         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1477         NULL, OK, OK );
1478     TMII_GMII ( MIIM_TYPE, 80,
1479         MFT_BITMAP | MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1480         NULL, OK, OK );
1481     TMII_GMII ( MIIM_FTYPE, 80,
1482         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1483         NULL, OK, OK );
1484     TMII_SMII( MIIM_BITMAP, 0, 0, 0, 0, 0, 0, 0, 0, 0, NULL );
1485     TMII_GMII ( MIIM_TYPE, 80,
1486         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 0, NULL,
1487         NULL, OK, OK );
1488     TMII_DONE
1489     /* Bitmaps inserted with MIIM_TYPE and MFT_BITMAP:
1490        Only the low word of the dwTypeData is used.
1491        Use a magic bitmap here (Word 95 uses this to create its MDI menu buttons) */
1492     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_RIGHTJUSTIFY, -1, -1, 0, 0, 0, -1,
1493                 (HMENU)MAKELONG(HBMMENU_MBAR_CLOSE, 0x1234), -1, 0, OK );
1494     TMII_GMII ( MIIM_TYPE, 80,
1495         MFT_BITMAP | MFT_RIGHTJUSTIFY, 0, 0, 0, 0, 0, 0, HBMMENU_MBAR_CLOSE, 0, HBMMENU_MBAR_CLOSE,
1496         NULL, OK, OK );
1497     TMII_DONE
1498     /* Type flags */
1499     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1500     TMII_GMII ( MIIM_TYPE, 80,
1501         MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1502         NULL, OK, OK );
1503     TMII_DONE
1504     /* State flags */
1505     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1506     TMII_SMII( MIIM_STATE, -1, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 0, 0 );
1507     TMII_GMII ( MIIM_STATE, 80,
1508         0, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 80, 0,
1509         NULL, OK, OK );
1510     TMII_DONE
1511     /* The style MFT_RADIOCHECK cannot be set with MIIM_CHECKMARKS only */
1512     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1513     TMII_SMII( MIIM_CHECKMARKS, MFT_RADIOCHECK, 0, 0, 0, hbm, hbm, 0, 0, 0, 0 );
1514     TMII_GMII ( MIIM_CHECKMARKS | MIIM_TYPE, 80,
1515         MFT_BITMAP, 0, 0, 0, hbm, hbm, 0, hbm, 0, hbm,
1516         NULL, OK, OK );
1517     TMII_DONE
1518     /* MFT_BITMAP is added automatically by GetMenuItemInfo() for MIIM_TYPE */
1519     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1520     TMII_SMII( MIIM_FTYPE, MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, (HBITMAP)0x1234, 0, 0 );
1521     TMII_GMII ( MIIM_FTYPE, 80,
1522         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1523         NULL, OK, OK );
1524     TMII_GMII ( MIIM_TYPE, 80,
1525         MFT_BITMAP | MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1526         NULL, OK, OK );
1527     TMII_GMII ( MIIM_FTYPE, 80,
1528         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1529         NULL, OK, OK );
1530     TMII_SMII( MIIM_BITMAP, 0, 0, 0, 0, 0, 0, 0, 0, 0, NULL );
1531     TMII_GMII ( MIIM_TYPE, 80,
1532         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 0, NULL,
1533         NULL, OK, OK );
1534     TMII_DONE
1535   } while( !(ansi = !ansi) );
1536   DeleteObject( hbm);
1537 }
1538
1539 /*
1540    The following tests try to confirm the algorithm used to return the menu items
1541    when there is a collision between a menu item and a popup menu
1542  */
1543 static void test_menu_search_bycommand( void )
1544 {
1545     HMENU        hmenu, hmenuSub, hmenuSub2;
1546     MENUITEMINFO info;
1547     BOOL         rc;
1548     UINT         id;
1549     char         strback[0x80];
1550     char         strIn[0x80];
1551     static CHAR menuitem[]  = "MenuItem",
1552                 menuitem2[] = "MenuItem 2";
1553
1554     /* Case 1: Menu containing a menu item */
1555     hmenu = CreateMenu();
1556
1557     memset( &info, 0, sizeof info );
1558     info.cbSize = sizeof info;
1559     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1560     info.fType = MFT_STRING;
1561     strcpy(strIn, "Case 1 MenuItem");
1562     info.dwTypeData = strIn;
1563     info.wID = (UINT) 0x1234;
1564
1565     rc = InsertMenuItem(hmenu, 0, TRUE, &info );
1566     ok (rc, "Inserting the menuitem failed\n");
1567
1568     id = GetMenuItemID(hmenu, 0);
1569     ok (id == 0x1234, "Getting the menuitem id failed(gave %x)\n", id);
1570
1571     /* Confirm the menuitem was given the id supplied (getting by position) */
1572     memset( &info, 0, sizeof info );
1573     strback[0] = 0x00;
1574     info.cbSize = sizeof(MENUITEMINFO);
1575     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1576     info.dwTypeData = strback;
1577     info.cch = sizeof(strback);
1578
1579     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info); /* Get by position */
1580     ok (rc, "Getting the menu items info failed\n");
1581     ok (info.wID == 0x1234, "IDs differ for the menuitem\n");
1582     ok (!strcmp(info.dwTypeData, "Case 1 MenuItem"), "Returned item has wrong label\n");
1583
1584     /* Search by id - Should return the item */
1585     memset( &info, 0, sizeof info );
1586     strback[0] = 0x00;
1587     info.cbSize = sizeof(MENUITEMINFO);
1588     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1589     info.dwTypeData = strback;
1590     info.cch = sizeof(strback);
1591     rc = GetMenuItemInfo(hmenu, 0x1234, FALSE, &info); /* Get by ID */
1592
1593     ok (rc, "Getting the menu items info failed\n");
1594     ok (info.wID == 0x1234, "IDs differ for the menuitem\n");
1595     ok (!strcmp(info.dwTypeData, "Case 1 MenuItem"), "Returned item has wrong label\n");
1596
1597     DestroyMenu( hmenu );
1598
1599     /* Case 2: Menu containing a popup menu */
1600     hmenu = CreateMenu();
1601     hmenuSub = CreateMenu();
1602
1603     strcpy(strIn, "Case 2 SubMenu");
1604     rc = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, strIn);
1605     ok (rc, "Inserting the popup menu into the main menu failed\n");
1606
1607     id = GetMenuItemID(hmenu, 0);
1608     ok (id == -1, "Getting the menuitem id unexpectedly worked (gave %x)\n", id);
1609
1610     /* Confirm the menuitem itself was given an id the same as the HMENU, (getting by position) */
1611     memset( &info, 0, sizeof info );
1612     strback[0] = 0x00;
1613     info.cbSize = sizeof(MENUITEMINFO);
1614     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1615     info.dwTypeData = strback;
1616     info.cch = sizeof(strback);
1617     info.wID = 0xdeadbeef;
1618
1619     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info); /* Get by position */
1620     ok (rc, "Getting the menu items info failed\n");
1621     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the menuitem\n");
1622     ok (!strcmp(info.dwTypeData, "Case 2 SubMenu"), "Returned item has wrong label\n");
1623
1624     /* Search by id - returns the popup menu itself */
1625     memset( &info, 0, sizeof info );
1626     strback[0] = 0x00;
1627     info.cbSize = sizeof(MENUITEMINFO);
1628     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1629     info.dwTypeData = strback;
1630     info.cch = sizeof(strback);
1631     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info); /* Get by ID */
1632
1633     ok (rc, "Getting the menu items info failed\n");
1634     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the popup menu\n");
1635     ok (!strcmp(info.dwTypeData, "Case 2 SubMenu"), "Returned item has wrong label\n");
1636
1637     /*
1638         Now add an item after it with the same id
1639      */
1640     memset( &info, 0, sizeof info );
1641     info.cbSize = sizeof info;
1642     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1643     info.fType = MFT_STRING;
1644     strcpy(strIn, "Case 2 MenuItem 1");
1645     info.dwTypeData = strIn;
1646     info.wID = (UINT_PTR) hmenuSub;
1647     rc = InsertMenuItem(hmenu, -1, TRUE, &info );
1648     ok (rc, "Inserting the menuitem failed\n");
1649
1650     /* Search by id - returns the item which follows the popup menu */
1651     memset( &info, 0, sizeof info );
1652     strback[0] = 0x00;
1653     info.cbSize = sizeof(MENUITEMINFO);
1654     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1655     info.dwTypeData = strback;
1656     info.cch = sizeof(strback);
1657     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info); /* Get by ID */
1658
1659     ok (rc, "Getting the menu items info failed\n");
1660     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the popup menu\n");
1661     ok (!strcmp(info.dwTypeData, "Case 2 MenuItem 1"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1662
1663     /*
1664         Now add an item before the popup (with the same id)
1665      */
1666     memset( &info, 0, sizeof info );
1667     info.cbSize = sizeof info;
1668     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1669     info.fType = MFT_STRING;
1670     strcpy(strIn, "Case 2 MenuItem 2");
1671     info.dwTypeData = strIn;
1672     info.wID = (UINT_PTR) hmenuSub;
1673     rc = InsertMenuItem(hmenu, 0, TRUE, &info );
1674     ok (rc, "Inserting the menuitem failed\n");
1675
1676     /* Search by id - returns the item which precedes the popup menu */
1677     memset( &info, 0, sizeof info );
1678     strback[0] = 0x00;
1679     info.cbSize = sizeof(MENUITEMINFO);
1680     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1681     info.dwTypeData = strback;
1682     info.cch = sizeof(strback);
1683     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info); /* Get by ID */
1684
1685     ok (rc, "Getting the menu items info failed\n");
1686     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the popup menu\n");
1687     ok (!strcmp(info.dwTypeData, "Case 2 MenuItem 2"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1688
1689     DestroyMenu( hmenu );
1690     DestroyMenu( hmenuSub );
1691
1692     /*
1693         Case 3: Menu containing a popup menu which in turn
1694            contains 2 items with the same id as the popup itself
1695      */
1696
1697     hmenu = CreateMenu();
1698     hmenuSub = CreateMenu();
1699
1700     memset( &info, 0, sizeof info );
1701     info.cbSize = sizeof info;
1702     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1703     info.fType = MFT_STRING;
1704     info.dwTypeData = menuitem;
1705     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1706
1707     rc = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, "Submenu");
1708     ok (rc, "Inserting the popup menu into the main menu failed\n");
1709
1710     rc = InsertMenuItem(hmenuSub, 0, TRUE, &info );
1711     ok (rc, "Inserting the sub menu menuitem failed\n");
1712
1713     memset( &info, 0, sizeof info );
1714     info.cbSize = sizeof info;
1715     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1716     info.fType = MFT_STRING;
1717     info.dwTypeData = menuitem2;
1718     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1719
1720     rc = InsertMenuItem(hmenuSub, 1, TRUE, &info );
1721     ok (rc, "Inserting the sub menu menuitem 2 failed\n");
1722
1723     /* Prove that you can't query the id of a popup directly (By position) */
1724     id = GetMenuItemID(hmenu, 0);
1725     ok (id == -1, "Getting the sub menu id should have failed because its a popup (gave %x)\n", id);
1726
1727     /* Prove getting the item info via ID returns the first item (not the popup or 2nd item)*/
1728     memset( &info, 0, sizeof info );
1729     strback[0] = 0x00;
1730     info.cbSize = sizeof(MENUITEMINFO);
1731     info.fMask = MIIM_STRING | MIIM_ID;
1732     info.dwTypeData = strback;
1733     info.cch = sizeof(strback);
1734
1735     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info);
1736     ok (rc, "Getting the menus info failed\n");
1737     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for popup menu\n");
1738     ok (!strcmp(info.dwTypeData, "MenuItem"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1739     DestroyMenu( hmenu );
1740     DestroyMenu( hmenuSub );
1741
1742     /*
1743         Case 4: Menu containing 2 popup menus, the second
1744            contains 2 items with the same id as the first popup menu
1745      */
1746     hmenu = CreateMenu();
1747     hmenuSub = CreateMenu();
1748     hmenuSub2 = CreateMenu();
1749
1750     rc = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, "Submenu");
1751     ok (rc, "Inserting the popup menu into the main menu failed\n");
1752
1753     rc = InsertMenu(hmenu, 1, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub2, "Submenu2");
1754     ok (rc, "Inserting the popup menu into the main menu failed\n");
1755
1756     memset( &info, 0, sizeof info );
1757     info.cbSize = sizeof info;
1758     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1759     info.fType = MFT_STRING;
1760     info.dwTypeData = menuitem;
1761     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1762
1763     rc = InsertMenuItem(hmenuSub2, 0, TRUE, &info );
1764     ok (rc, "Inserting the sub menu menuitem failed\n");
1765
1766     memset( &info, 0, sizeof info );
1767     info.cbSize = sizeof info;
1768     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1769     info.fType = MFT_STRING;
1770     info.dwTypeData = menuitem2;
1771     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1772
1773     rc = InsertMenuItem(hmenuSub2, 1, TRUE, &info );
1774     ok (rc, "Inserting the sub menu menuitem 2 failed\n");
1775
1776     /* Prove getting the item info via ID returns the first item (not the popup or 2nd item)*/
1777     memset( &info, 0, sizeof info );
1778     strback[0] = 0x00;
1779     info.cbSize = sizeof(MENUITEMINFO);
1780     info.fMask = MIIM_STRING | MIIM_ID;
1781     info.dwTypeData = strback;
1782     info.cch = sizeof(strback);
1783
1784     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info);
1785     ok (rc, "Getting the menus info failed\n");
1786     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for popup menu\n");
1787     ok (!strcmp(info.dwTypeData, "MenuItem"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1788
1789     memset( &info, 0, sizeof info );
1790     strback[0] = 0x00;
1791     info.cbSize = sizeof(MENUITEMINFO);
1792     info.fMask = MIIM_STRING | MIIM_ID;
1793     info.dwTypeData = strback;
1794     info.cch = sizeof(strback);
1795
1796     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub2, FALSE, &info);
1797     ok (rc, "Getting the menus info failed\n");
1798     ok (info.wID == (UINT_PTR)hmenuSub2, "IDs differ for popup menu\n");
1799     ok (!strcmp(info.dwTypeData, "Submenu2"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1800
1801     DestroyMenu( hmenu );
1802     DestroyMenu( hmenuSub );
1803     DestroyMenu( hmenuSub2 );
1804
1805
1806     /*
1807         Case 5: Menu containing a popup menu which in turn
1808            contains an item with a different id than the popup menu.
1809            This tests the fallback to a popup menu ID.
1810      */
1811
1812     hmenu = CreateMenu();
1813     hmenuSub = CreateMenu();
1814
1815     rc = AppendMenu(hmenu, MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, "Submenu");
1816     ok (rc, "Appending the popup menu to the main menu failed\n");
1817
1818     rc = AppendMenu(hmenuSub, MF_STRING, 102, "Item");
1819     ok (rc, "Appending the item to the popup menu failed\n");
1820
1821     /* Set the ID for hmenuSub */
1822     info.cbSize = sizeof(info);
1823     info.fMask = MIIM_ID;
1824     info.wID = 101;
1825
1826     rc = SetMenuItemInfo(hmenu, 0, TRUE, &info);
1827     ok(rc, "Setting the ID for the popup menu failed\n");
1828
1829     /* Check if the ID has been set */
1830     info.wID = 0;
1831     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info);
1832     ok(rc, "Getting the ID for the popup menu failed\n");
1833     ok(info.wID == 101, "The ID for the popup menu has not been set\n");
1834
1835     /* Prove getting the item info via ID returns the popup menu */
1836     memset( &info, 0, sizeof(info));
1837     strback[0] = 0x00;
1838     info.cbSize = sizeof(MENUITEMINFO);
1839     info.fMask = MIIM_STRING | MIIM_ID;
1840     info.dwTypeData = strback;
1841     info.cch = sizeof(strback);
1842
1843     rc = GetMenuItemInfo(hmenu, 101, FALSE, &info);
1844     ok (rc, "Getting the menu info failed\n");
1845     ok (info.wID == 101, "IDs differ\n");
1846     ok (!strcmp(info.dwTypeData, "Submenu"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1847
1848     /* Also look for the menu item  */
1849     memset( &info, 0, sizeof(info));
1850     strback[0] = 0x00;
1851     info.cbSize = sizeof(MENUITEMINFO);
1852     info.fMask = MIIM_STRING | MIIM_ID;
1853     info.dwTypeData = strback;
1854     info.cch = sizeof(strback);
1855
1856     rc = GetMenuItemInfo(hmenu, 102, FALSE, &info);
1857     ok (rc, "Getting the menu info failed\n");
1858     ok (info.wID == 102, "IDs differ\n");
1859     ok (!strcmp(info.dwTypeData, "Item"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1860
1861     DestroyMenu(hmenu);
1862     DestroyMenu(hmenuSub);
1863 }
1864
1865 struct menu_item_pair_s {
1866     UINT uMenu; /* 1 - top level menu, [0-Menu 1-Enabled 2-Disabled]
1867                  * 2 - 2nd level menu, [0-Popup 1-Enabled 2-Disabled]
1868                  * 3 - 3rd level menu, [0-Enabled 1-Disabled] */
1869     UINT uItem;
1870 };
1871
1872 static struct menu_mouse_tests_s {
1873     DWORD type;
1874     struct menu_item_pair_s menu_item_pairs[5]; /* for mousing */
1875     WORD wVk[5]; /* keys */
1876     BOOL bMenuVisible;
1877     BOOL _todo_wine;
1878 } menu_tests[] = {
1879     /* for each test, send keys or clicks and check for menu visibility */
1880     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 0}, TRUE, FALSE }, /* test 0 */
1881     { INPUT_KEYBOARD, {{0}}, {VK_ESCAPE, 0}, FALSE, FALSE },
1882     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 0}, TRUE, FALSE },
1883     { INPUT_KEYBOARD, {{0}}, {'D', 0}, FALSE, FALSE },
1884     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 0}, TRUE, FALSE },
1885     { INPUT_KEYBOARD, {{0}}, {'E', 0}, FALSE, FALSE },
1886     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 0}, TRUE, FALSE },
1887     { INPUT_KEYBOARD, {{0}}, {VK_ESCAPE, VK_ESCAPE, 0}, FALSE, FALSE },
1888     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', VK_ESCAPE, 0}, TRUE, FALSE },
1889     { INPUT_KEYBOARD, {{0}}, {VK_ESCAPE, 0}, FALSE, FALSE },
1890     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 0}, TRUE, FALSE },
1891     { INPUT_KEYBOARD, {{0}}, {'D', 0}, FALSE, FALSE },
1892     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 0}, TRUE, FALSE },
1893     { INPUT_KEYBOARD, {{0}}, {'E', 0}, FALSE, FALSE },
1894     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 'P', 0}, TRUE, FALSE },
1895     { INPUT_KEYBOARD, {{0}}, {'D', 0}, FALSE, FALSE },
1896     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 'P', 0}, TRUE, FALSE },
1897     { INPUT_KEYBOARD, {{0}}, {'E', 0}, FALSE, FALSE },
1898     { INPUT_KEYBOARD, {{0}}, {VK_F10, 0}, TRUE, FALSE },
1899     { INPUT_KEYBOARD, {{0}}, {VK_F10, 0}, FALSE, FALSE },
1900
1901     { INPUT_MOUSE, {{1, 2}, {0}}, {0}, TRUE, TRUE }, /* test 20 */
1902     { INPUT_MOUSE, {{1, 1}, {0}}, {0}, FALSE, FALSE },
1903     { INPUT_MOUSE, {{1, 0}, {0}}, {0}, TRUE, TRUE },
1904     { INPUT_MOUSE, {{1, 1}, {0}}, {0}, FALSE, FALSE },
1905     { INPUT_MOUSE, {{1, 0}, {2, 2}, {0}}, {0}, TRUE, TRUE },
1906     { INPUT_MOUSE, {{2, 1}, {0}}, {0}, FALSE, FALSE },
1907     { INPUT_MOUSE, {{1, 0}, {2, 0}, {0}}, {0}, TRUE, TRUE },
1908     { INPUT_MOUSE, {{3, 0}, {0}}, {0}, FALSE, FALSE },
1909     { INPUT_MOUSE, {{1, 0}, {2, 0}, {0}}, {0}, TRUE, TRUE },
1910     { INPUT_MOUSE, {{3, 1}, {0}}, {0}, TRUE, TRUE },
1911     { INPUT_MOUSE, {{1, 1}, {0}}, {0}, FALSE, FALSE },
1912     { -1 }
1913 };
1914
1915 static void send_key(WORD wVk)
1916 {
1917     TEST_INPUT i[2];
1918     memset(i, 0, sizeof(i));
1919     i[0].type = i[1].type = INPUT_KEYBOARD;
1920     i[0].u.ki.wVk = i[1].u.ki.wVk = wVk;
1921     i[1].u.ki.dwFlags = KEYEVENTF_KEYUP;
1922     pSendInput(2, (INPUT *) i, sizeof(INPUT));
1923 }
1924
1925 static BOOL click_menu(HANDLE hWnd, struct menu_item_pair_s *mi)
1926 {
1927     HMENU hMenu = hMenus[mi->uMenu];
1928     TEST_INPUT i[3];
1929     MSG msg;
1930     RECT r;
1931     int screen_w = GetSystemMetrics(SM_CXSCREEN);
1932     int screen_h = GetSystemMetrics(SM_CYSCREEN);
1933     BOOL ret = GetMenuItemRect(mi->uMenu > 2 ? NULL : hWnd, hMenu, mi->uItem, &r);
1934     if(!ret) return FALSE;
1935
1936     memset(i, 0, sizeof(i));
1937     i[0].type = i[1].type = i[2].type = INPUT_MOUSE;
1938     i[0].u.mi.dx = i[1].u.mi.dx = i[2].u.mi.dx
1939             = ((r.left + 5) * 65535) / screen_w;
1940     i[0].u.mi.dy = i[1].u.mi.dy = i[2].u.mi.dy
1941             = ((r.top + 5) * 65535) / screen_h;
1942     i[0].u.mi.dwFlags = i[1].u.mi.dwFlags = i[2].u.mi.dwFlags
1943             = MOUSEEVENTF_ABSOLUTE;
1944     i[0].u.mi.dwFlags |= MOUSEEVENTF_MOVE;
1945     i[1].u.mi.dwFlags |= MOUSEEVENTF_LEFTDOWN;
1946     i[2].u.mi.dwFlags |= MOUSEEVENTF_LEFTUP;
1947     ret = pSendInput(3, (INPUT *) i, sizeof(INPUT));
1948
1949     /* hack to prevent mouse message buildup in Wine */
1950     while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessageA( &msg );
1951     return ret;
1952 }
1953
1954 static DWORD WINAPI test_menu_input_thread(LPVOID lpParameter)
1955 {
1956     int i, j;
1957     HANDLE hWnd = lpParameter;
1958
1959     Sleep(500);
1960     /* mixed keyboard/mouse test */
1961     for (i = 0; menu_tests[i].type != -1; i++)
1962     {
1963         int ret = TRUE, elapsed = 0;
1964
1965         got_input = i && menu_tests[i-1].bMenuVisible;
1966
1967         if (menu_tests[i].type == INPUT_KEYBOARD)
1968             for (j = 0; menu_tests[i].wVk[j] != 0; j++)
1969                 send_key(menu_tests[i].wVk[j]);
1970         else
1971             for (j = 0; menu_tests[i].menu_item_pairs[j].uMenu != 0; j++)
1972                 if (!(ret = click_menu(hWnd, &menu_tests[i].menu_item_pairs[j]))) break;
1973
1974         if (!ret)
1975         {
1976             skip( "test %u: failed to send input\n", i );
1977             PostMessage( hWnd, WM_CANCELMODE, 0, 0 );
1978             return 0;
1979         }
1980         while (menu_tests[i].bMenuVisible != bMenuVisible)
1981         {
1982             if (elapsed > 200)
1983                 break;
1984             elapsed += 20;
1985             Sleep(20);
1986         }
1987
1988         if (!got_input)
1989         {
1990             skip( "test %u: didn't receive input\n", i );
1991             PostMessage( hWnd, WM_CANCELMODE, 0, 0 );
1992             return 0;
1993         }
1994
1995         if (menu_tests[i]._todo_wine)
1996         {
1997             todo_wine {
1998                 ok(menu_tests[i].bMenuVisible == bMenuVisible, "test %d\n", i);
1999             }
2000         }
2001         else
2002             ok(menu_tests[i].bMenuVisible == bMenuVisible, "test %d\n", i);
2003     }
2004     return 0;
2005 }
2006
2007 static LRESULT CALLBACK WndProc(HWND hWnd, UINT msg, WPARAM wParam,
2008         LPARAM lParam)
2009 {
2010     switch (msg) {
2011         case WM_ENTERMENULOOP:
2012             bMenuVisible = TRUE;
2013             break;
2014         case WM_EXITMENULOOP:
2015             bMenuVisible = FALSE;
2016             break;
2017         case WM_KEYDOWN:
2018         case WM_SYSKEYDOWN:
2019         case WM_MOUSEMOVE:
2020         case WM_LBUTTONDOWN:
2021         case WM_LBUTTONUP:
2022         case WM_NCMOUSEMOVE:
2023         case WM_NCLBUTTONDOWN:
2024         case WM_NCLBUTTONUP:
2025             got_input = TRUE;
2026             /* fall through */
2027         default:
2028             return( DefWindowProcA( hWnd, msg, wParam, lParam ) );
2029     }
2030     return 0;
2031 }
2032
2033 static void test_menu_input(void) {
2034     MSG msg;
2035     WNDCLASSA  wclass;
2036     HINSTANCE hInstance = GetModuleHandleA( NULL );
2037     HANDLE hThread, hWnd;
2038     DWORD tid;
2039
2040     if (!pSendInput)
2041     {
2042         win_skip("SendInput is not available\n");
2043         return;
2044     }
2045
2046     wclass.lpszClassName = "MenuTestClass";
2047     wclass.style         = CS_HREDRAW | CS_VREDRAW;
2048     wclass.lpfnWndProc   = WndProc;
2049     wclass.hInstance     = hInstance;
2050     wclass.hIcon         = LoadIconA( 0, IDI_APPLICATION );
2051     wclass.hCursor       = LoadCursorA( NULL, IDC_ARROW );
2052     wclass.hbrBackground = (HBRUSH)( COLOR_WINDOW + 1 );
2053     wclass.lpszMenuName  = 0;
2054     wclass.cbClsExtra    = 0;
2055     wclass.cbWndExtra    = 0;
2056     assert (RegisterClassA( &wclass ));
2057     assert (hWnd = CreateWindowA( wclass.lpszClassName, "MenuTest",
2058                                   WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, 0,
2059                                   400, 200, NULL, NULL, hInstance, NULL) );
2060
2061     /* fixed menus */
2062     hMenus[3] = CreatePopupMenu();
2063     AppendMenu(hMenus[3], MF_STRING, 0, "&Enabled");
2064     AppendMenu(hMenus[3], MF_STRING|MF_DISABLED, 0, "&Disabled");
2065
2066     hMenus[2] = CreatePopupMenu();
2067     AppendMenu(hMenus[2], MF_STRING|MF_POPUP, (UINT_PTR) hMenus[3], "&Popup");
2068     AppendMenu(hMenus[2], MF_STRING, 0, "&Enabled");
2069     AppendMenu(hMenus[2], MF_STRING|MF_DISABLED, 0, "&Disabled");
2070
2071     hMenus[1] = CreateMenu();
2072     AppendMenu(hMenus[1], MF_STRING|MF_POPUP, (UINT_PTR) hMenus[2], "&Menu");
2073     AppendMenu(hMenus[1], MF_STRING, 0, "&Enabled");
2074     AppendMenu(hMenus[1], MF_STRING|MF_DISABLED, 0, "&Disabled");
2075
2076     SetMenu(hWnd, hMenus[1]);
2077     ShowWindow(hWnd, SW_SHOW);
2078     UpdateWindow(hWnd);
2079
2080     hThread = CreateThread(NULL, 0, test_menu_input_thread, hWnd, 0, &tid);
2081     while(1)
2082     {
2083         if (WAIT_TIMEOUT != WaitForSingleObject(hThread, 50))
2084             break;
2085         while (PeekMessage(&msg, 0, 0, 0, PM_REMOVE)) DispatchMessageA(&msg);
2086     }
2087     DestroyWindow(hWnd);
2088 }
2089
2090 static void test_menu_flags( void )
2091 {
2092     HMENU hMenu, hPopupMenu;
2093
2094     hMenu = CreateMenu();
2095     hPopupMenu = CreatePopupMenu();
2096
2097     AppendMenu(hMenu, MF_POPUP | MF_STRING, (UINT_PTR)hPopupMenu, "Popup");
2098
2099     AppendMenu(hPopupMenu, MF_STRING | MF_HILITE | MF_DEFAULT, 101, "Item 1");
2100     InsertMenu(hPopupMenu, 1, MF_BYPOSITION | MF_STRING | MF_HILITE | MF_DEFAULT, 102, "Item 2");
2101     AppendMenu(hPopupMenu, MF_STRING, 103, "Item 3");
2102     ModifyMenu(hPopupMenu, 2, MF_BYPOSITION | MF_STRING | MF_HILITE | MF_DEFAULT, 103, "Item 3");
2103
2104     ok(GetMenuState(hPopupMenu, 0, MF_BYPOSITION) & MF_HILITE,
2105       "AppendMenu should accept MF_HILITE\n");
2106     ok(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE,
2107       "InsertMenu should accept MF_HILITE\n");
2108     ok(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_HILITE,
2109       "ModifyMenu should accept MF_HILITE\n");
2110
2111     ok(!(GetMenuState(hPopupMenu, 0, MF_BYPOSITION) & MF_DEFAULT),
2112       "AppendMenu must not accept MF_DEFAULT\n");
2113     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_DEFAULT),
2114       "InsertMenu must not accept MF_DEFAULT\n");
2115     ok(!(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_DEFAULT),
2116       "ModifyMenu must not accept MF_DEFAULT\n");
2117
2118     DestroyMenu(hMenu);
2119 }
2120
2121 static void test_menu_hilitemenuitem( void )
2122 {
2123     HMENU hMenu, hPopupMenu;
2124     WNDCLASSA wclass;
2125     HWND hWnd;
2126
2127     wclass.lpszClassName = "HiliteMenuTestClass";
2128     wclass.style         = CS_HREDRAW | CS_VREDRAW;
2129     wclass.lpfnWndProc   = WndProc;
2130     wclass.hInstance     = GetModuleHandleA( NULL );
2131     wclass.hIcon         = LoadIconA( 0, IDI_APPLICATION );
2132     wclass.hCursor       = LoadCursorA( NULL, IDC_ARROW );
2133     wclass.hbrBackground = (HBRUSH)( COLOR_WINDOW + 1 );
2134     wclass.lpszMenuName  = 0;
2135     wclass.cbClsExtra    = 0;
2136     wclass.cbWndExtra    = 0;
2137     assert (RegisterClassA( &wclass ));
2138     assert (hWnd = CreateWindowA( wclass.lpszClassName, "HiliteMenuTest",
2139                                   WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, 0,
2140                                   400, 200, NULL, NULL, wclass.hInstance, NULL) );
2141
2142     hMenu = CreateMenu();
2143     hPopupMenu = CreatePopupMenu();
2144
2145     AppendMenu(hMenu, MF_POPUP | MF_STRING, (UINT_PTR)hPopupMenu, "Popup");
2146
2147     AppendMenu(hPopupMenu, MF_STRING, 101, "Item 1");
2148     AppendMenu(hPopupMenu, MF_STRING, 102, "Item 2");
2149     AppendMenu(hPopupMenu, MF_STRING, 103, "Item 3");
2150
2151     SetMenu(hWnd, hMenu);
2152
2153     /* test invalid arguments */
2154
2155     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2156       "HiliteMenuItem: Item 2 is hilited\n");
2157
2158     SetLastError(0xdeadbeef);
2159     todo_wine
2160     {
2161     ok(!HiliteMenuItem(NULL, hPopupMenu, 1, MF_HILITE | MF_BYPOSITION),
2162       "HiliteMenuItem: call should have failed.\n");
2163     }
2164     ok(GetLastError() == 0xdeadbeef || /* 9x */
2165        GetLastError() == ERROR_INVALID_WINDOW_HANDLE /* NT */,
2166       "HiliteMenuItem: expected error ERROR_INVALID_WINDOW_HANDLE, got: %d\n", GetLastError());
2167
2168     SetLastError(0xdeadbeef);
2169     ok(!HiliteMenuItem(hWnd, NULL, 1, MF_HILITE | MF_BYPOSITION),
2170       "HiliteMenuItem: call should have failed.\n");
2171     ok(GetLastError() == 0xdeadbeef || /* 9x */
2172        GetLastError() == ERROR_INVALID_MENU_HANDLE /* NT */,
2173       "HiliteMenuItem: expected error ERROR_INVALID_MENU_HANDLE, got: %d\n", GetLastError());
2174
2175     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2176       "HiliteMenuItem: Item 2 is hilited\n");
2177
2178     /* either MF_HILITE or MF_UNHILITE *and* MF_BYCOMMAND or MF_BYPOSITION need to be set */
2179
2180     SetLastError(0xdeadbeef);
2181     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_BYPOSITION),
2182       "HiliteMenuItem: call should have succeeded.\n");
2183     ok(GetLastError() == 0xdeadbeef,
2184       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2185
2186     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2187       "HiliteMenuItem: Item 2 is hilited\n");
2188
2189     SetLastError(0xdeadbeef);
2190     todo_wine
2191     {
2192     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_HILITE),
2193       "HiliteMenuItem: call should have succeeded.\n");
2194     }
2195     ok(GetLastError() == 0xdeadbeef,
2196       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2197
2198     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2199       "HiliteMenuItem: Item 2 is hilited\n");
2200
2201     /* hilite a menu item (by position) */
2202
2203     SetLastError(0xdeadbeef);
2204     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_HILITE | MF_BYPOSITION),
2205       "HiliteMenuItem: call should not have failed.\n");
2206     ok(GetLastError() == 0xdeadbeef,
2207       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2208
2209     todo_wine
2210     {
2211     ok(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE,
2212       "HiliteMenuItem: Item 2 is not hilited\n");
2213     }
2214
2215     /* unhilite a menu item (by position) */
2216
2217     SetLastError(0xdeadbeef);
2218     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_UNHILITE | MF_BYPOSITION),
2219       "HiliteMenuItem: call should not have failed.\n");
2220     ok(GetLastError() == 0xdeadbeef,
2221       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2222
2223     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2224       "HiliteMenuItem: Item 2 is hilited\n");
2225
2226     /* hilite a menu item (by command) */
2227
2228     SetLastError(0xdeadbeef);
2229     ok(HiliteMenuItem(hWnd, hPopupMenu, 103, MF_HILITE | MF_BYCOMMAND),
2230       "HiliteMenuItem: call should not have failed.\n");
2231     ok(GetLastError() == 0xdeadbeef,
2232       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2233
2234     todo_wine
2235     {
2236     ok(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_HILITE,
2237       "HiliteMenuItem: Item 3 is not hilited\n");
2238     }
2239
2240     /* unhilite a menu item (by command) */
2241
2242     SetLastError(0xdeadbeef);
2243     ok(HiliteMenuItem(hWnd, hPopupMenu, 103, MF_UNHILITE | MF_BYCOMMAND),
2244       "HiliteMenuItem: call should not have failed.\n");
2245     ok(GetLastError() == 0xdeadbeef,
2246       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2247
2248     ok(!(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_HILITE),
2249       "HiliteMenuItem: Item 3 is hilited\n");
2250
2251     DestroyWindow(hWnd);
2252 }
2253
2254 static void check_menu_items(HMENU hmenu, UINT checked_cmd, UINT checked_type,
2255                              UINT checked_state)
2256 {
2257     INT i, count;
2258
2259     count = GetMenuItemCount(hmenu);
2260     ok (count != -1, "GetMenuItemCount returned -1\n");
2261
2262     for (i = 0; i < count; i++)
2263     {
2264         BOOL ret;
2265         MENUITEMINFO mii;
2266
2267         memset(&mii, 0, sizeof(mii));
2268         mii.cbSize = sizeof(mii);
2269         mii.fMask  = MIIM_FTYPE | MIIM_STATE | MIIM_ID | MIIM_SUBMENU;
2270         ret = GetMenuItemInfo(hmenu, i, TRUE, &mii);
2271         ok(ret, "GetMenuItemInfo(%u) failed\n", i);
2272 #if 0
2273         trace("item #%u: fType %04x, fState %04x, wID %u, hSubMenu %p\n",
2274                i, mii.fType, mii.fState, mii.wID, mii.hSubMenu);
2275 #endif
2276         if (mii.hSubMenu)
2277         {
2278             ok(mii.wID == (UINT_PTR)mii.hSubMenu, "id %u: wID should be equal to hSubMenu\n", checked_cmd);
2279             check_menu_items(mii.hSubMenu, checked_cmd, checked_type, checked_state);
2280         }
2281         else
2282         {
2283             if (mii.wID == checked_cmd)
2284             {
2285                 ok(mii.fType == checked_type, "id %u: expected fType %04x, got %04x\n", checked_cmd, checked_type, mii.fType);
2286                 ok(mii.fState == checked_state, "id %u: expected fState %04x, got %04x\n", checked_cmd, checked_state, mii.fState);
2287                 ok(mii.wID != 0, "id %u: not expected wID 0\n", checked_cmd);
2288             }
2289             else
2290             {
2291                 ok(mii.fType != MFT_RADIOCHECK, "id %u: not expected fType MFT_RADIOCHECK on cmd %u\n", checked_cmd, mii.wID);
2292
2293                 if (mii.fType == MFT_SEPARATOR)
2294                 {
2295                     ok(mii.fState == MFS_GRAYED, "id %u: expected fState MFS_GRAYED, got %04x\n", checked_cmd, mii.fState);
2296                     ok(mii.wID == 0, "id %u: expected wID 0, got %u\n", checked_cmd, mii.wID);
2297                 }
2298                 else
2299                 {
2300                     ok(mii.fState == 0, "id %u: expected fState 0, got %04x\n", checked_cmd, mii.fState);
2301                     ok(mii.wID != 0, "id %u: not expected wID 0\n", checked_cmd);
2302                 }
2303             }
2304         }
2305     }
2306 }
2307
2308 static void clear_ftype_and_state(HMENU hmenu, UINT id, UINT flags)
2309 {
2310     BOOL ret;
2311     MENUITEMINFO mii;
2312
2313     memset(&mii, 0, sizeof(mii));
2314     mii.cbSize = sizeof(mii);
2315     mii.fMask  = MIIM_FTYPE | MIIM_STATE;
2316     ret = SetMenuItemInfo(hmenu, id, (flags & MF_BYPOSITION) != 0, &mii);
2317     ok(ret, "SetMenuItemInfo(%u) failed\n", id);
2318 }
2319
2320 static void test_CheckMenuRadioItem(void)
2321 {
2322     BOOL ret;
2323     HMENU hmenu;
2324
2325     hmenu = LoadMenu(GetModuleHandle(0), MAKEINTRESOURCE(1));
2326     assert(hmenu != 0);
2327
2328     check_menu_items(hmenu, -1, 0, 0);
2329
2330     ret = CheckMenuRadioItem(hmenu, 100, 100, 100, MF_BYCOMMAND);
2331     ok(ret, "CheckMenuRadioItem failed\n");
2332     check_menu_items(hmenu, 100, MFT_RADIOCHECK, MFS_CHECKED);
2333
2334     /* MSDN is wrong, Windows does not remove MFT_RADIOCHECK */
2335     ret = CheckMenuRadioItem(hmenu, 100, 100, -1, MF_BYCOMMAND);
2336     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2337     check_menu_items(hmenu, 100, MFT_RADIOCHECK, 0);
2338
2339     /* clear check */
2340     clear_ftype_and_state(hmenu, 100, MF_BYCOMMAND);
2341     check_menu_items(hmenu, -1, 0, 0);
2342
2343     /* first and checked items are on different menus */
2344     ret = CheckMenuRadioItem(hmenu, 0, 300, 202, MF_BYCOMMAND);
2345     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2346     check_menu_items(hmenu, -1, 0, 0);
2347
2348     ret = CheckMenuRadioItem(hmenu, 200, 300, 202, MF_BYCOMMAND);
2349     ok(ret, "CheckMenuRadioItem failed\n");
2350     check_menu_items(hmenu, 202, MFT_RADIOCHECK, MFS_CHECKED);
2351
2352     /* MSDN is wrong, Windows does not remove MFT_RADIOCHECK */
2353     ret = CheckMenuRadioItem(hmenu, 202, 202, -1, MF_BYCOMMAND);
2354     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2355     check_menu_items(hmenu, 202, MFT_RADIOCHECK, 0);
2356
2357     /* clear check */
2358     clear_ftype_and_state(hmenu, 202, MF_BYCOMMAND);
2359     check_menu_items(hmenu, -1, 0, 0);
2360
2361     /* just for fun, try to check separator */
2362     ret = CheckMenuRadioItem(hmenu, 0, 300, 0, MF_BYCOMMAND);
2363     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2364     check_menu_items(hmenu, -1, 0, 0);
2365 }
2366
2367 static void test_menu_resource_layout(void)
2368 {
2369     static const struct
2370     {
2371         MENUITEMTEMPLATEHEADER mith;
2372         WORD data[14];
2373     } menu_template =
2374     {
2375         { 0, 0 }, /* versionNumber, offset */
2376         {
2377             /* mtOption, mtID, mtString[] '\0' terminated */
2378             MF_STRING, 1, 'F', 0,
2379             MF_STRING, 2, 0,
2380             MF_SEPARATOR, 3, 0,
2381             /* MF_SEPARATOR, 4, 'S', 0, FIXME: Wine ignores 'S' */
2382             MF_STRING|MF_GRAYED|MF_END, 5, 'E', 0
2383         }
2384     };
2385     static const struct
2386     {
2387         UINT type, state, id;
2388         const char *str;
2389     } menu_data[] =
2390     {
2391         { MF_STRING, MF_ENABLED, 1, "F" },
2392         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 2, "" },
2393         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 3, "" },
2394         /*{ MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 4, "S" }, FIXME: Wine ignores 'S'*/
2395         { MF_STRING, MF_GRAYED, 5, "E" },
2396         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 6, "" },
2397         { MF_STRING, MF_ENABLED, 7, "" },
2398         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 8, "" }
2399     };
2400     HMENU hmenu;
2401     INT count, i;
2402     BOOL ret;
2403
2404     hmenu = LoadMenuIndirect(&menu_template);
2405     ok(hmenu != 0, "LoadMenuIndirect error %u\n", GetLastError());
2406
2407     ret = AppendMenu(hmenu, MF_STRING, 6, NULL);
2408     ok(ret, "AppendMenu failed\n");
2409     ret = AppendMenu(hmenu, MF_STRING, 7, "\0");
2410     ok(ret, "AppendMenu failed\n");
2411     ret = AppendMenu(hmenu, MF_SEPARATOR, 8, "separator");
2412     ok(ret, "AppendMenu failed\n");
2413
2414     count = GetMenuItemCount(hmenu);
2415     ok(count == sizeof(menu_data)/sizeof(menu_data[0]),
2416        "expected %u menu items, got %u\n",
2417        (UINT)(sizeof(menu_data)/sizeof(menu_data[0])), count);
2418
2419     for (i = 0; i < count; i++)
2420     {
2421         char buf[20];
2422         MENUITEMINFO mii;
2423
2424         memset(&mii, 0, sizeof(mii));
2425         mii.cbSize = sizeof(mii);
2426         mii.dwTypeData = buf;
2427         mii.cch = sizeof(buf);
2428         mii.fMask  = MIIM_FTYPE | MIIM_STATE | MIIM_ID | MIIM_STRING;
2429         ret = GetMenuItemInfo(hmenu, i, TRUE, &mii);
2430         ok(ret, "GetMenuItemInfo(%u) failed\n", i);
2431 #if 0
2432         trace("item #%u: fType %04x, fState %04x, wID %u, dwTypeData %s\n",
2433                i, mii.fType, mii.fState, mii.wID, (LPCSTR)mii.dwTypeData);
2434 #endif
2435         ok(mii.fType == menu_data[i].type,
2436            "%u: expected fType %04x, got %04x\n", i, menu_data[i].type, mii.fType);
2437         ok(mii.fState == menu_data[i].state,
2438            "%u: expected fState %04x, got %04x\n", i, menu_data[i].state, mii.fState);
2439         ok(mii.wID == menu_data[i].id,
2440            "%u: expected wID %04x, got %04x\n", i, menu_data[i].id, mii.wID);
2441         ok(mii.cch == strlen(menu_data[i].str),
2442            "%u: expected cch %u, got %u\n", i, (UINT)strlen(menu_data[i].str), mii.cch);
2443         ok(!strcmp(mii.dwTypeData, menu_data[i].str),
2444            "%u: expected dwTypeData %s, got %s\n", i, menu_data[i].str, (LPCSTR)mii.dwTypeData);
2445     }
2446
2447     DestroyMenu(hmenu);
2448 }
2449
2450 struct menu_data
2451 {
2452     UINT type, id;
2453     const char *str;
2454 };
2455
2456 static HMENU create_menu_from_data(const struct menu_data *item, INT item_count)
2457 {
2458     HMENU hmenu;
2459     INT i;
2460     BOOL ret;
2461
2462     hmenu = CreateMenu();
2463     assert(hmenu != 0);
2464
2465     for (i = 0; i < item_count; i++)
2466     {
2467         SetLastError(0xdeadbeef);
2468         ret = AppendMenu(hmenu, item[i].type, item[i].id, item[i].str);
2469         ok(ret, "%d: AppendMenu(%04x, %04x, %p) error %u\n",
2470            i, item[i].type, item[i].id, item[i].str, GetLastError());
2471     }
2472     return hmenu;
2473 }
2474
2475 /* use InsertMenuItem: does not set the MFT_BITMAP flag,
2476  * and does not accept non-magic bitmaps with invalid
2477  * bitmap handles */
2478 static HMENU create_menuitem_from_data(const struct menu_data *item, INT item_count)
2479 {
2480     HMENU hmenu;
2481     INT i;
2482     BOOL ret;
2483     MENUITEMINFO mii = { sizeof( MENUITEMINFO)};
2484
2485     hmenu = CreateMenu();
2486     assert(hmenu != 0);
2487
2488     for (i = 0; i < item_count; i++)
2489     {
2490         SetLastError(0xdeadbeef);
2491
2492         mii.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STATE;
2493         mii.fType = 0;
2494         if(  item[i].type & MFT_BITMAP)
2495         {
2496             mii.fMask |= MIIM_BITMAP;
2497             mii.hbmpItem =  (HBITMAP)item[i].str;
2498         }
2499         else if(  item[i].type & MFT_SEPARATOR)
2500             mii.fType = MFT_SEPARATOR;
2501         else
2502         {
2503             mii.fMask |= MIIM_STRING;
2504             mii.dwTypeData =  (LPSTR)item[i].str;
2505             mii.cch = strlen(  item[i].str);
2506         }
2507         mii.fState = 0;
2508         if(  item[i].type & MF_HELP) mii.fType |= MF_HELP;
2509         mii.wID = item[i].id;
2510         ret = InsertMenuItem( hmenu, -1, TRUE, &mii);
2511         ok(ret, "%d: InsertMenuItem(%04x, %04x, %p) error %u\n",
2512            i, item[i].type, item[i].id, item[i].str, GetLastError());
2513     }
2514     return hmenu;
2515 }
2516
2517 static void compare_menu_data(HMENU hmenu, const struct menu_data *item, INT item_count)
2518 {
2519     INT count, i;
2520     BOOL ret;
2521
2522     count = GetMenuItemCount(hmenu);
2523     ok(count == item_count, "expected %d, got %d menu items\n", count, item_count);
2524
2525     for (i = 0; i < count; i++)
2526     {
2527         char buf[20];
2528         MENUITEMINFO mii;
2529
2530         memset(&mii, 0, sizeof(mii));
2531         mii.cbSize = sizeof(mii);
2532         mii.dwTypeData = buf;
2533         mii.cch = sizeof(buf);
2534         mii.fMask  = MIIM_FTYPE | MIIM_ID | MIIM_STRING | MIIM_BITMAP;
2535         ret = GetMenuItemInfo(hmenu, i, TRUE, &mii);
2536         ok(ret, "GetMenuItemInfo(%u) failed\n", i);
2537 #if 0
2538         trace("item #%u: fType %04x, fState %04x, wID %04x, hbmp %p\n",
2539                i, mii.fType, mii.fState, mii.wID, mii.hbmpItem);
2540 #endif
2541         ok(mii.fType == item[i].type,
2542            "%u: expected fType %04x, got %04x\n", i, item[i].type, mii.fType);
2543         ok(mii.wID == item[i].id,
2544            "%u: expected wID %04x, got %04x\n", i, item[i].id, mii.wID);
2545         if (mii.hbmpItem || !item[i].str)
2546             /* For some reason Windows sets high word to not 0 for
2547              * not "magic" ids.
2548              */
2549             ok(LOWORD(mii.hbmpItem) == LOWORD(item[i].str),
2550                "%u: expected hbmpItem %p, got %p\n", i, item[i].str, mii.hbmpItem);
2551         else
2552         {
2553             ok(mii.cch == strlen(item[i].str),
2554                "%u: expected cch %u, got %u\n", i, (UINT)strlen(item[i].str), mii.cch);
2555             ok(!strcmp(mii.dwTypeData, item[i].str),
2556                "%u: expected dwTypeData %s, got %s\n", i, item[i].str, (LPCSTR)mii.dwTypeData);
2557         }
2558     }
2559 }
2560
2561 static void test_InsertMenu(void)
2562 {
2563     HBITMAP hbm = CreateBitmap(1,1,1,1,NULL);
2564     /* Note: XP treats only bitmap handles 1 - 6 as "magic" ones
2565      * regardless of their id.
2566      */
2567     static const struct menu_data in1[] =
2568     {
2569         { MF_STRING, 1, "File" },
2570         { MF_BITMAP|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) },
2571         { MF_STRING|MF_HELP, 2, "Help" }
2572     };
2573     static const struct menu_data out1[] =
2574     {
2575         { MF_STRING, 1, "File" },
2576         { MF_STRING|MF_HELP, 2, "Help" },
2577         { MF_BITMAP|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) }
2578     };
2579     static const struct menu_data out1a[] =
2580     {
2581         { MF_STRING, 1, "File" },
2582         { MF_STRING|MF_HELP, 2, "Help" },
2583         { MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) }
2584     };
2585     const struct menu_data in2[] =
2586     {
2587         { MF_STRING, 1, "File" },
2588         { MF_BITMAP|MF_HELP, SC_CLOSE, (char*)hbm },
2589         { MF_STRING|MF_HELP, 2, "Help" }
2590     };
2591     const struct menu_data out2[] =
2592     {
2593         { MF_STRING, 1, "File" },
2594         { MF_BITMAP|MF_HELP, SC_CLOSE, (char*)hbm },
2595         { MF_STRING|MF_HELP, 2, "Help" }
2596     };
2597     const struct menu_data out2a[] =
2598     {
2599         { MF_STRING, 1, "File" },
2600         { MF_HELP, SC_CLOSE, (char*)hbm },
2601         { MF_STRING|MF_HELP, 2, "Help" }
2602     };
2603     static const struct menu_data in3[] =
2604     {
2605         { MF_STRING, 1, "File" },
2606         { MF_SEPARATOR|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) },
2607         { MF_STRING|MF_HELP, 2, "Help" }
2608     };
2609     static const struct menu_data out3[] =
2610     {
2611         { MF_STRING, 1, "File" },
2612         { MF_SEPARATOR|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(0) },
2613         { MF_STRING|MF_HELP, 2, "Help" },
2614     };
2615     static const struct menu_data in4[] =
2616     {
2617         { MF_STRING, 1, "File" },
2618         { MF_BITMAP|MF_HELP, 1, MAKEINTRESOURCE(1) },
2619         { MF_STRING|MF_HELP, 2, "Help" }
2620     };
2621     static const struct menu_data out4[] =
2622     {
2623         { MF_STRING, 1, "File" },
2624         { MF_STRING|MF_HELP, 2, "Help" },
2625         { MF_BITMAP|MF_HELP, 1, MAKEINTRESOURCE(1) }
2626     };
2627     static const struct menu_data out4a[] =
2628     {
2629         { MF_STRING, 1, "File" },
2630         { MF_STRING|MF_HELP, 2, "Help" },
2631         { MF_HELP, 1, MAKEINTRESOURCE(1) }
2632     };
2633     HMENU hmenu;
2634
2635 #define create_menu(a) create_menu_from_data((a), sizeof(a)/sizeof((a)[0]))
2636 #define create_menuitem(a) create_menuitem_from_data((a), sizeof(a)/sizeof((a)[0]))
2637 #define compare_menu(h, a) compare_menu_data((h), (a), sizeof(a)/sizeof((a)[0]))
2638
2639     hmenu = create_menu(in1);
2640     compare_menu(hmenu, out1);
2641     DestroyMenu(hmenu);
2642
2643     hmenu = create_menu(in2);
2644     compare_menu(hmenu, out2);
2645     DestroyMenu(hmenu);
2646
2647     hmenu = create_menu(in3);
2648     compare_menu(hmenu, out3);
2649     DestroyMenu(hmenu);
2650
2651     hmenu = create_menu(in4);
2652     compare_menu(hmenu, out4);
2653     DestroyMenu(hmenu);
2654
2655     /* now using InsertMenuItemInfo */
2656     hmenu = create_menuitem(in1);
2657     compare_menu(hmenu, out1a);
2658     DestroyMenu(hmenu);
2659
2660     hmenu = create_menuitem(in2);
2661     compare_menu(hmenu, out2a);
2662     DestroyMenu(hmenu);
2663
2664     hmenu = create_menuitem(in3);
2665     compare_menu(hmenu, out3);
2666     DestroyMenu(hmenu);
2667
2668     hmenu = create_menuitem(in4);
2669     compare_menu(hmenu, out4a);
2670     DestroyMenu(hmenu);
2671
2672 #undef create_menu
2673 #undef create_menuitem
2674 #undef compare_menu
2675 }
2676
2677 static void test_menu_getmenuinfo(void)
2678 {
2679     HMENU hmenu;
2680     MENUINFO mi = {0};
2681     BOOL ret;
2682     DWORD gle;
2683
2684     if (!pGetMenuInfo)
2685     {
2686         win_skip("GetMenuInfo is not available\n");
2687         return;
2688     }
2689
2690     /* create a menu */
2691     hmenu = CreateMenu();
2692     assert( hmenu);
2693     /* test some parameter errors */
2694     SetLastError(0xdeadbeef);
2695     ret = pGetMenuInfo( hmenu, NULL);
2696     gle= GetLastError();
2697     ok( !ret, "GetMenuInfo() should have failed\n");
2698     ok( gle == ERROR_INVALID_PARAMETER ||
2699         broken(gle == 0xdeadbeef), /* Win98, WinME */
2700         "GetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2701     SetLastError(0xdeadbeef);
2702     mi.cbSize = 0;
2703     ret = pGetMenuInfo( hmenu, &mi);
2704     gle= GetLastError();
2705     ok( !ret, "GetMenuInfo() should have failed\n");
2706     ok( gle == ERROR_INVALID_PARAMETER ||
2707         broken(gle == 0xdeadbeef), /* Win98, WinME */
2708         "GetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2709     SetLastError(0xdeadbeef);
2710     mi.cbSize = sizeof( MENUINFO);
2711     ret = pGetMenuInfo( hmenu, &mi);
2712     gle= GetLastError();
2713     ok( ret, "GetMenuInfo() should have succeeded\n");
2714     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2715     SetLastError(0xdeadbeef);
2716     mi.cbSize = 0;
2717     ret = pGetMenuInfo( NULL, &mi);
2718     gle= GetLastError();
2719     ok( !ret, "GetMenuInfo() should have failed\n");
2720     ok( gle == ERROR_INVALID_PARAMETER ||
2721         broken(gle == 0xdeadbeef), /* Win98, WinME */
2722         "GetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2723     /* clean up */
2724     DestroyMenu( hmenu);
2725     return;
2726 }
2727
2728 static void test_menu_setmenuinfo(void)
2729 {
2730     HMENU hmenu, hsubmenu;
2731     MENUINFO mi = {0};
2732     MENUITEMINFOA mii = {sizeof( MENUITEMINFOA)};
2733     BOOL ret;
2734     DWORD gle;
2735
2736     if (!pGetMenuInfo || !pSetMenuInfo)
2737     {
2738         win_skip("Get/SetMenuInfo are not available\n");
2739         return;
2740     }
2741
2742     /* create a menu with a submenu */
2743     hmenu = CreateMenu();
2744     hsubmenu = CreateMenu();
2745     assert( hmenu && hsubmenu);
2746     mii.fMask = MIIM_SUBMENU;
2747     mii.hSubMenu = hsubmenu;
2748     ret = InsertMenuItem( hmenu, 0, FALSE, &mii);
2749     ok( ret, "InsertMenuItem failed with error %d\n", GetLastError());
2750     /* test some parameter errors */
2751     SetLastError(0xdeadbeef);
2752     ret = pSetMenuInfo( hmenu, NULL);
2753     gle= GetLastError();
2754     ok( !ret, "SetMenuInfo() should have failed\n");
2755     ok( gle == ERROR_INVALID_PARAMETER ||
2756         broken(gle == 0xdeadbeef), /* Win98, WinME */
2757         "SetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2758     SetLastError(0xdeadbeef);
2759     mi.cbSize = 0;
2760     ret = pSetMenuInfo( hmenu, &mi);
2761     gle= GetLastError();
2762     ok( !ret, "SetMenuInfo() should have failed\n");
2763     ok( gle == ERROR_INVALID_PARAMETER ||
2764         broken(gle == 0xdeadbeef), /* Win98, WinME */
2765         "SetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2766     SetLastError(0xdeadbeef);
2767     mi.cbSize = sizeof( MENUINFO);
2768     ret = pSetMenuInfo( hmenu, &mi);
2769     gle= GetLastError();
2770     ok( ret, "SetMenuInfo() should have succeeded\n");
2771     ok( gle == 0xdeadbeef, "SetMenuInfo() error got %u\n", gle);
2772     SetLastError(0xdeadbeef);
2773     mi.cbSize = 0;
2774     ret = pSetMenuInfo( NULL, &mi);
2775     gle= GetLastError();
2776     ok( !ret, "SetMenuInfo() should have failed\n");
2777     ok( gle == ERROR_INVALID_PARAMETER ||
2778         broken(gle == 0xdeadbeef), /* Win98, WinME */
2779         "SetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2780     /* functional tests */
2781     /* menu and submenu should have the CHECKORBMP style bit cleared */
2782     SetLastError(0xdeadbeef);
2783     mi.cbSize = sizeof( MENUINFO);
2784     mi.fMask = MIM_STYLE;
2785     ret = pGetMenuInfo( hmenu, &mi);
2786     gle= GetLastError();
2787     ok( ret, "GetMenuInfo() should have succeeded\n");
2788     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2789     ok( !(mi.dwStyle & MNS_CHECKORBMP), "menustyle was not expected to have the MNS_CHECKORBMP flag\n");
2790     SetLastError(0xdeadbeef);
2791     mi.cbSize = sizeof( MENUINFO);
2792     mi.fMask = MIM_STYLE;
2793     ret = pGetMenuInfo( hsubmenu, &mi);
2794     gle= GetLastError();
2795     ok( ret, "GetMenuInfo() should have succeeded\n");
2796     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2797     ok( !(mi.dwStyle & MNS_CHECKORBMP), "menustyle was not expected to have the MNS_CHECKORBMP flag\n");
2798     /* SetMenuInfo() */
2799     SetLastError(0xdeadbeef);
2800     mi.cbSize = sizeof( MENUINFO);
2801     mi.fMask = MIM_STYLE | MIM_APPLYTOSUBMENUS;
2802     mi.dwStyle = MNS_CHECKORBMP;
2803     ret = pSetMenuInfo( hmenu, &mi);
2804     gle= GetLastError();
2805     ok( ret, "SetMenuInfo() should have succeeded\n");
2806     ok( gle == 0xdeadbeef, "SetMenuInfo() error got %u\n", gle);
2807     /* Now both menus should have the MNS_CHECKORBMP style bit set */
2808     SetLastError(0xdeadbeef);
2809     mi.cbSize = sizeof( MENUINFO);
2810     mi.fMask = MIM_STYLE;
2811     ret = pGetMenuInfo( hmenu, &mi);
2812     gle= GetLastError();
2813     ok( ret, "GetMenuInfo() should have succeeded\n");
2814     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2815     ok( mi.dwStyle & MNS_CHECKORBMP, "menustyle was expected to have the MNS_CHECKORBMP flag\n");
2816     SetLastError(0xdeadbeef);
2817     mi.cbSize = sizeof( MENUINFO);
2818     mi.fMask = MIM_STYLE;
2819     ret = pGetMenuInfo( hsubmenu, &mi);
2820     gle= GetLastError();
2821     ok( ret, "GetMenuInfo() should have succeeded\n");
2822     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2823     ok( mi.dwStyle & MNS_CHECKORBMP, "menustyle was expected to have the MNS_CHECKORBMP flag\n");
2824     /* now repeat that without the APPLYTOSUBMENUS flag and another style bit */
2825     SetLastError(0xdeadbeef);
2826     mi.cbSize = sizeof( MENUINFO);
2827     mi.fMask = MIM_STYLE ;
2828     mi.dwStyle = MNS_NOCHECK;
2829     ret = pSetMenuInfo( hmenu, &mi);
2830     gle= GetLastError();
2831     ok( ret, "SetMenuInfo() should have succeeded\n");
2832     ok( gle == 0xdeadbeef, "SetMenuInfo() error got %u\n", gle);
2833     /* Now only the top menu should have the MNS_NOCHECK style bit set */
2834     SetLastError(0xdeadbeef);
2835     mi.cbSize = sizeof( MENUINFO);
2836     mi.fMask = MIM_STYLE;
2837     ret = pGetMenuInfo( hmenu, &mi);
2838     gle= GetLastError();
2839     ok( ret, "GetMenuInfo() should have succeeded\n");
2840     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2841     ok( mi.dwStyle & MNS_NOCHECK, "menustyle was expected to have the MNS_NOCHECK flag\n");
2842     SetLastError(0xdeadbeef);
2843     mi.cbSize = sizeof( MENUINFO);
2844     mi.fMask = MIM_STYLE;
2845     ret = pGetMenuInfo( hsubmenu, &mi);
2846     gle= GetLastError();
2847     ok( ret, "GetMenuInfo() should have succeeded\n");
2848     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2849     ok( !(mi.dwStyle & MNS_NOCHECK), "menustyle was not expected to have the MNS_NOCHECK flag\n");
2850     /* clean up */
2851     DestroyMenu( hsubmenu);
2852     DestroyMenu( hmenu);
2853     return;
2854 }
2855
2856 /* little func to easy switch either TrackPopupMenu() or TrackPopupMenuEx() */
2857 static DWORD MyTrackPopupMenu( int ex, HMENU hmenu, UINT flags, INT x, INT y, HWND hwnd, LPTPMPARAMS ptpm)
2858 {
2859     return ex
2860         ? TrackPopupMenuEx( hmenu, flags, x, y, hwnd, ptpm)
2861         : TrackPopupMenu( hmenu, flags, x, y, 0, hwnd, NULL);
2862 }
2863
2864 /* some TrackPopupMenu and TrackPopupMenuEx tests */
2865 /* the LastError values differ between NO_ERROR and invalid handle */
2866 /* between all windows versions tested. The first value is that valid on XP  */
2867 /* Vista was the only that made returned different error values */
2868 /* between the TrackPopupMenu and TrackPopupMenuEx functions */
2869 static void test_menu_trackpopupmenu(void)
2870 {
2871     BOOL ret;
2872     HMENU hmenu;
2873     DWORD gle;
2874     int Ex;
2875     HWND hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
2876             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
2877             NULL, NULL, NULL, NULL);
2878     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
2879     if (!hwnd) return;
2880     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_ownerdraw_wnd_proc);
2881     for( Ex = 0; Ex < 2; Ex++)
2882     {
2883         hmenu = CreatePopupMenu();
2884         ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
2885         if (!hmenu)
2886         {
2887             DestroyWindow(hwnd);
2888             return;
2889         }
2890         /* display the menu */
2891         /* start with an invalid menu handle */
2892         gle = 0xdeadbeef;
2893         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2894         ret = MyTrackPopupMenu( Ex, NULL, TPM_RETURNCMD, 100,100, hwnd, NULL);
2895         gle = GetLastError();
2896         ok( !ret, "TrackPopupMenu%s should have failed\n", Ex ? "Ex" : "");
2897         ok( gle == ERROR_INVALID_MENU_HANDLE
2898             || broken (gle == 0xdeadbeef) /* win95 */
2899             || broken (gle == NO_ERROR) /* win98/ME */
2900             ,"TrackPopupMenu%s error got %u expected %u\n",
2901             Ex ? "Ex" : "", gle, ERROR_INVALID_MENU_HANDLE);
2902         ok( !(gflag_initmenupopup || gflag_entermenuloop || gflag_initmenu),
2903                 "got unexpected message(s)%s%s%s\n",
2904                 gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2905                 gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2906                 gflag_initmenu ? "WM_INITMENU": "");
2907         /* another one but not NULL */
2908         gle = 0xdeadbeef;
2909         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2910         ret = MyTrackPopupMenu( Ex, (HMENU)hwnd, TPM_RETURNCMD, 100,100, hwnd, NULL);
2911         gle = GetLastError();
2912         ok( !ret, "TrackPopupMenu%s should have failed\n", Ex ? "Ex" : "");
2913         ok( gle == ERROR_INVALID_MENU_HANDLE
2914             || broken (gle == 0xdeadbeef) /* win95 */
2915             || broken (gle == NO_ERROR) /* win98/ME */
2916             ,"TrackPopupMenu%s error got %u expected %u\n",
2917             Ex ? "Ex" : "", gle, ERROR_INVALID_MENU_HANDLE);
2918         ok( !(gflag_initmenupopup || gflag_entermenuloop || gflag_initmenu),
2919                 "got unexpected message(s)%s%s%s\n",
2920                 gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2921                 gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2922                 gflag_initmenu ? "WM_INITMENU": "");
2923         /* now a somewhat successful call */
2924         gle = 0xdeadbeef;
2925         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2926         ret = MyTrackPopupMenu( Ex, hmenu, TPM_RETURNCMD, 100,100, hwnd, NULL);
2927         gle = GetLastError();
2928         ok( ret == 0, "TrackPopupMenu%s returned %d expected zero\n", Ex ? "Ex" : "", ret);
2929         ok( gle == NO_ERROR
2930             || gle == ERROR_INVALID_MENU_HANDLE /* NT4, win2k */
2931             || broken (gle == 0xdeadbeef) /* win95 */
2932             ,"TrackPopupMenu%s error got %u expected %u or %u\n",
2933             Ex ? "Ex" : "", gle, NO_ERROR, ERROR_INVALID_MENU_HANDLE);
2934         ok( gflag_initmenupopup && gflag_entermenuloop && gflag_initmenu,
2935                 "missed expected message(s)%s%s%s\n",
2936                 !gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2937                 !gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2938                 !gflag_initmenu ? "WM_INITMENU": "");
2939         /* and another */
2940         ret = AppendMenuA( hmenu, MF_STRING, 1, "winetest");
2941         ok( ret, "AppendMenA has failed!\n");
2942         gle = 0xdeadbeef;
2943         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2944         ret = MyTrackPopupMenu( Ex, hmenu, TPM_RETURNCMD, 100,100, hwnd, NULL);
2945         gle = GetLastError();
2946         ok( ret == 0, "TrackPopupMenu%s returned %d expected zero\n", Ex ? "Ex" : "", ret);
2947         ok( gle == NO_ERROR
2948             || gle == ERROR_INVALID_MENU_HANDLE /* NT4, win2k and Vista in the TrackPopupMenuEx case */
2949             || broken (gle == 0xdeadbeef) /* win95 */
2950             ,"TrackPopupMenu%s error got %u expected %u or %u\n",
2951             Ex ? "Ex" : "", gle, NO_ERROR, ERROR_INVALID_MENU_HANDLE);
2952         ok( gflag_initmenupopup && gflag_entermenuloop && gflag_initmenu,
2953                 "missed expected message(s)%s%s%s\n",
2954                 !gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2955                 !gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2956                 !gflag_initmenu ? "WM_INITMENU": "");
2957         DestroyMenu(hmenu);
2958     }
2959     /* clean up */
2960     DestroyWindow(hwnd);
2961 }
2962
2963 static HMENU g_hmenu;
2964
2965 static LRESULT WINAPI menu_track_again_wnd_proc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
2966 {
2967     switch (msg)
2968     {
2969     case WM_ENTERMENULOOP:
2970     {
2971         BOOL ret;
2972
2973         /* try a recursive call */
2974         SetLastError(0xdeadbeef);
2975         ret = TrackPopupMenu(g_hmenu, 0, 100, 100, 0, hwnd, NULL);
2976         ok(ret == FALSE, "got %d\n", ret);
2977         ok(GetLastError() == ERROR_POPUP_ALREADY_ACTIVE ||
2978            broken(GetLastError() == 0xdeadbeef) /* W9x */, "got %d\n", GetLastError());
2979
2980         /* exit menu modal loop
2981          * ( A SendMessage does not work on NT3.51 here ) */
2982         return PostMessage(hwnd, WM_CANCELMODE, 0, 0);
2983     }
2984     }
2985     return DefWindowProc(hwnd, msg, wparam, lparam);
2986 }
2987
2988 static void test_menu_trackagain(void)
2989 {
2990     HWND hwnd;
2991     BOOL ret;
2992
2993     hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
2994             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
2995             NULL, NULL, NULL, NULL);
2996     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
2997     if (!hwnd) return;
2998     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_track_again_wnd_proc);
2999
3000     g_hmenu = CreatePopupMenu();
3001     ok(g_hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
3002
3003     ret = TrackPopupMenu( g_hmenu, 0, 100, 100, 0, hwnd, NULL);
3004     todo_wine ok(ret == FALSE, "got %d\n", ret);
3005
3006     DestroyMenu(g_hmenu);
3007     DestroyWindow(hwnd);
3008 }
3009
3010 /* test handling of WM_CANCELMODE messages */
3011 static int g_got_enteridle;
3012 static HWND g_hwndtosend;
3013 static LRESULT WINAPI menu_cancelmode_wnd_proc(HWND hwnd, UINT msg,
3014         WPARAM wparam, LPARAM lparam)
3015 {
3016     switch (msg)
3017     {
3018         case WM_ENTERMENULOOP:
3019             g_got_enteridle = 0;
3020             return SendMessage( g_hwndtosend, WM_CANCELMODE, 0, 0);
3021         case WM_ENTERIDLE:
3022             {
3023                 if( g_got_enteridle++ == 0) {
3024                     /* little hack to get another WM_ENTERIDLE message */
3025                     PostMessage( hwnd, WM_MOUSEMOVE, 0, 0);
3026                     return SendMessage( g_hwndtosend, WM_CANCELMODE, 0, 0);
3027                 }
3028                 pEndMenu();
3029                 return TRUE;
3030             }
3031     }
3032     return DefWindowProc( hwnd, msg, wparam, lparam);
3033 }
3034
3035 static void test_menu_cancelmode(void)
3036 {
3037     DWORD ret;
3038     HWND hwnd, hwndchild;
3039     HMENU menu;
3040     if( !pEndMenu) { /* win95 */
3041         win_skip( "EndMenu is not available\n");
3042         return;
3043     }
3044     hwnd = CreateWindowEx( 0, MAKEINTATOM(atomMenuCheckClass), NULL,
3045             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
3046             NULL, NULL, NULL, NULL);
3047     hwndchild = CreateWindowEx( 0, MAKEINTATOM(atomMenuCheckClass), NULL,
3048             WS_VISIBLE | WS_CHILD, 10, 10, 20, 20,
3049             hwnd, NULL, NULL, NULL);
3050     ok( hwnd != NULL && hwndchild != NULL,
3051             "CreateWindowEx failed with error %d\n", GetLastError());
3052     g_hwndtosend = hwnd;
3053     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_cancelmode_wnd_proc);
3054     SetWindowLongPtr( hwndchild, GWLP_WNDPROC, (LONG_PTR)menu_cancelmode_wnd_proc);
3055     menu = CreatePopupMenu();
3056     ok( menu != NULL, "CreatePopupMenu failed with error %d\n", GetLastError());
3057     ret = AppendMenuA( menu, MF_STRING, 1, "winetest");
3058     ok( ret, "Functie failed lasterror is %u\n", GetLastError());
3059     /* seems to be needed only on wine :( */
3060     {MSG msg;   while (PeekMessage(&msg, 0, 0, 0, PM_REMOVE)) DispatchMessage(&msg);}
3061     /* test the effect of sending a WM_CANCELMODE message in the WM_INITMENULOOP
3062      * handler of the menu owner */
3063     /* test results is exctracted from variable g_got_enteridle. Possible values:
3064      * 0 : complete conformance. Sending WM_CANCELMODE cancels a menu initializing tracking
3065      * 1 : Sending WM_CANCELMODE cancels a menu that is in tracking state
3066      * 2 : Sending WM_CANCELMODE does not work
3067      */
3068     /* menu owner is top level window */
3069     g_hwndtosend = hwnd;
3070     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
3071     todo_wine {
3072         ok( g_got_enteridle == 0, "received %d WM_ENTERIDLE messages, none expected\n", g_got_enteridle);
3073     }
3074     ok( g_got_enteridle < 2, "received %d WM_ENTERIDLE messages, should be less than 2\n", g_got_enteridle);
3075     /* menu owner is child window */
3076     g_hwndtosend = hwndchild;
3077     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 100,100, 0, hwndchild, NULL);
3078     todo_wine {
3079         ok(g_got_enteridle == 0, "received %d WM_ENTERIDLE messages, none expected\n", g_got_enteridle);
3080     }
3081     ok(g_got_enteridle < 2, "received %d WM_ENTERIDLE messages, should be less than 2\n", g_got_enteridle);
3082     /* now send the WM_CANCELMODE messages to the WRONG window */
3083     /* those should fail ( to have any effect) */
3084     g_hwndtosend = hwnd;
3085     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 100,100, 0, hwndchild, NULL);
3086     ok( g_got_enteridle == 2, "received %d WM_ENTERIDLE messages, should be 2\n", g_got_enteridle);
3087     /* cleanup */
3088     DestroyMenu( menu);
3089     DestroyWindow( hwndchild);
3090     DestroyWindow( hwnd);
3091 }
3092
3093 /* show menu trees have a maximum depth */
3094 static void test_menu_maxdepth(void)
3095 {
3096 #define NR_MENUS 100
3097     HMENU hmenus[ NR_MENUS];
3098     int i;
3099     DWORD ret;
3100
3101     SetLastError(12345678);
3102     for( i = 0; i < NR_MENUS; i++) {
3103         hmenus[i] = CreatePopupMenu();
3104         if( !hmenus[i]) break;
3105     }
3106     ok( i == NR_MENUS, "could not create more than %d menu's\n", i);
3107     for( i = 1; i < NR_MENUS; i++) {
3108         ret = AppendMenuA( hmenus[i], MF_POPUP, (UINT_PTR)hmenus[i-1],"test");
3109         if( !ret) break;
3110     }
3111     trace("Maximum depth is %d\n", i);
3112     ok( GetLastError() == 12345678, "unexpected error %d\n",  GetLastError());
3113     ok( i < NR_MENUS ||
3114            broken( i == NR_MENUS), /* win98, NT */
3115            "no ( or very large) limit on menu depth!\n");
3116
3117     for( i = 0; i < NR_MENUS; i++)
3118         DestroyMenu( hmenus[i]);
3119 }
3120
3121 /* bug #12171 */
3122 static void test_menu_circref(void)
3123 {
3124     HMENU menu1, menu2;
3125     DWORD ret;
3126
3127     menu1 = CreatePopupMenu();
3128     menu2 = CreatePopupMenu();
3129     ok( menu1 && menu2, "error creating menus.\n");
3130     ret = AppendMenuA( menu1, MF_POPUP, (UINT_PTR)menu2, "winetest");
3131     ok( ret, "AppendMenu failed, error is %d\n", GetLastError());
3132     ret = AppendMenuA( menu1, MF_STRING | MF_HILITE, 123, "winetest");
3133     ok( ret, "AppendMenu failed, error is %d\n", GetLastError());
3134     /* app chooses an id that happens to clash with its own hmenu */
3135     ret = AppendMenuA( menu2, MF_STRING, (UINT_PTR)menu2, "winetest");
3136     ok( ret, "AppendMenu failed, error is %d\n", GetLastError());
3137     /* now attempt to change the string of the first item of menu1 */
3138     ret = ModifyMenuA( menu1, (UINT_PTR)menu2, MF_POPUP, (UINT_PTR)menu2, "menu 2");
3139     ok( !ret ||
3140             broken( ret), /* win98, NT */
3141             "ModifyMenu should have failed.\n");
3142     if( !ret) { /* will probably stack fault if the ModifyMenu succeeded */
3143         ret = GetMenuState( menu1, 123, 0);
3144         ok( ret == MF_HILITE, "GetMenuState returned %x\n",ret);
3145     }
3146     DestroyMenu( menu2);
3147     DestroyMenu( menu1);
3148 }
3149
3150 /* test how the menu texts are aligned when the menu items have
3151  * different combinations of text and bitmaps (bug #13350) */
3152 static void test_menualign(void)
3153 {
3154     BYTE bmfill[300];
3155     HMENU menu;
3156     HBITMAP hbm1, hbm2, hbm3;
3157     MENUITEMINFO mii = { sizeof(MENUITEMINFO)};
3158     DWORD ret;
3159     HWND hwnd;
3160     MENUINFO mi = { sizeof( MENUINFO)};
3161
3162     if( !winetest_interactive) {
3163         skip( "interactive alignment tests.\n");
3164         return;
3165     }
3166     hwnd = CreateWindowEx(0,
3167             "STATIC",
3168             "Menu text alignment Test\nPlease make a selection.",
3169             WS_OVERLAPPEDWINDOW,
3170             100, 100,
3171             300, 300,
3172             NULL, NULL, 0, NULL);
3173     ShowWindow( hwnd, SW_SHOW);
3174     /* create bitmaps */
3175     memset( bmfill, 0xcc, sizeof( bmfill));
3176     hbm1 = CreateBitmap( 10,10,1,1,bmfill);
3177     hbm2 = CreateBitmap( 20,20,1,1,bmfill);
3178     hbm3 = CreateBitmap( 50,6,1,1,bmfill);
3179     ok( hbm1 && hbm2 && hbm3, "Creating bitmaps failed\n");
3180     menu = CreatePopupMenu();
3181     ok( menu != NULL, "CreatePopupMenu() failed\n");
3182     if( pGetMenuInfo) {
3183         mi.fMask = MIM_STYLE;
3184         ret = pGetMenuInfo( menu, &mi);
3185         ok( menu != NULL, "GetMenuInfo() failed\n");
3186         ok( 0 == mi.dwStyle, "menuinfo style is %x\n", mi.dwStyle);
3187     }
3188     /* test 1 */
3189     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3190     mii.wID = 1;
3191     mii.hbmpItem = hbm1;
3192     mii.dwTypeData = (LPSTR) " OK: menu texts are correctly left-aligned.";
3193     ret = InsertMenuItem( menu, -1, TRUE, &mii);
3194     ok( ret, "InsertMenuItem() failed\n");
3195     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID ;
3196     mii.wID = 2;
3197     mii.hbmpItem = hbm2;
3198     mii.dwTypeData = (LPSTR) " FAIL: menu texts are NOT left-aligned.";
3199     ret = InsertMenuItem( menu, -1, TRUE, &mii);
3200     ok( ret, "InsertMenuItem() failed\n");
3201     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 110, 200, 0, hwnd, NULL);
3202     ok( ret != 2, "User indicated that menu text alignment test 1 failed %d\n", ret);
3203     /* test 2*/
3204     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3205     mii.wID = 3;
3206     mii.hbmpItem = hbm3;
3207     mii.dwTypeData = NULL;
3208     ret = InsertMenuItem( menu, 0, TRUE, &mii);
3209     ok( ret, "InsertMenuItem() failed\n");
3210     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3211     mii.wID = 1;
3212     mii.hbmpItem = hbm1;
3213     /* make the text a bit longer, to keep it readable */
3214     /* this bug is on winXP and reproduced on wine */
3215     mii.dwTypeData = (LPSTR) " OK: menu texts are to the right of the bitmaps........";
3216     ret = SetMenuItemInfo( menu, 1, TRUE, &mii);
3217     ok( ret, "SetMenuItemInfo() failed\n");
3218     mii.wID = 2;
3219     mii.hbmpItem = hbm2;
3220     mii.dwTypeData = (LPSTR) " FAIL: menu texts are below the first bitmap.  ";
3221     ret = SetMenuItemInfo( menu, 2, TRUE, &mii);
3222     ok( ret, "SetMenuItemInfo() failed\n");
3223     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 110, 200, 0, hwnd, NULL);
3224     ok( ret != 2, "User indicated that menu text alignment test 2 failed %d\n", ret);
3225     /* test 3 */
3226     mii.fMask = MIIM_TYPE | MIIM_ID;
3227     mii.wID = 3;
3228     mii.fType = MFT_BITMAP;
3229     mii.dwTypeData = (LPSTR) hbm3;
3230     ret = SetMenuItemInfo( menu, 0, TRUE, &mii);
3231     ok( ret, "SetMenuItemInfo() failed\n");
3232     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3233     mii.wID = 1;
3234     mii.hbmpItem = NULL;
3235     mii.dwTypeData = (LPSTR) " OK: menu texts are below the bitmap.";
3236     ret = SetMenuItemInfo( menu, 1, TRUE, &mii);
3237     ok( ret, "SetMenuItemInfo() failed\n");
3238     mii.wID = 2;
3239     mii.hbmpItem = NULL;
3240     mii.dwTypeData = (LPSTR) " FAIL: menu texts are NOT below the bitmap.";
3241     ret = SetMenuItemInfo( menu, 2, TRUE, &mii);
3242     ok( ret, "SetMenuItemInfo() failed\n");
3243     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 110, 200, 0, hwnd, NULL);
3244     ok( ret != 2, "User indicated that menu text alignment test 3 failed %d\n", ret);
3245     /* cleanup */
3246     DeleteObject( hbm1);
3247     DeleteObject( hbm2);
3248     DeleteObject( hbm3);
3249     DestroyMenu( menu);
3250     DestroyWindow( hwnd);
3251 }
3252
3253 START_TEST(menu)
3254 {
3255     init_function_pointers();
3256
3257     /* Wine defines MENUITEMINFO for W2K and above. NT4 and below can't
3258      * handle that.
3259      */
3260     if (correct_behavior())
3261     {
3262         test_menu_add_string();
3263         test_menu_iteminfo();
3264         test_menu_search_bycommand();
3265         test_CheckMenuRadioItem();
3266         test_menu_resource_layout();
3267         test_InsertMenu();
3268         test_menualign();
3269     }
3270
3271     register_menu_check_class();
3272
3273     test_menu_locked_by_window();
3274     test_subpopup_locked_by_menu();
3275     test_menu_ownerdraw();
3276     test_menu_bmp_and_string();
3277     test_menu_getmenuinfo();
3278     test_menu_setmenuinfo();
3279     test_menu_input();
3280     test_menu_flags();
3281
3282     test_menu_hilitemenuitem();
3283     test_menu_trackpopupmenu();
3284     test_menu_trackagain();
3285     test_menu_cancelmode();
3286     test_menu_maxdepth();
3287     test_menu_circref();
3288 }