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
 664     if( !pGetMenuInfo)
 665     {
 666         win_skip("GetMenuInfo is not available\n");
 667         return;
 668     }
 669
 670     memset( bmfill, 0xcc, sizeof( bmfill));
 671     hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL, WS_SYSMENU |
 672                           WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
 673                           NULL, NULL, NULL, NULL);
 674     hbm_arrow=LoadBitmap( 0, (CHAR*)OBM_MNARROW);
 675     GetObject( hbm_arrow, sizeof(bm), &bm);
 676     arrowwidth = bm.bmWidth;
 677     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
 678     if( !hwnd) return;
 679     /* test system menu */
 680     hsysmenu = GetSystemMenu( hwnd, FALSE);
 681     ok( hsysmenu != NULL, "GetSystemMenu failed with error %d\n", GetLastError());
 682     mi.fMask = MIM_STYLE;
 683     mi.dwStyle = 0;
 684     ok( pGetMenuInfo( hsysmenu, &mi), "GetMenuInfo failed gle=%d\n", GetLastError());
 685     ok( MNS_CHECKORBMP == mi.dwStyle, "System Menu Style is %08x, without the bit %08x\n",
 686         mi.dwStyle, MNS_CHECKORBMP);
 687     mii.fMask = MIIM_BITMAP;
 688     mii.hbmpItem = NULL;
 689     ok( GetMenuItemInfoA( hsysmenu, SC_CLOSE, FALSE, &mii), "GetMenuItemInfoA failed gle=%d\n", GetLastError());
 690     ok( HBMMENU_POPUP_CLOSE == mii.hbmpItem, "Item info did not get the right hbitmap: got %p  expected %p\n",
 691         mii.hbmpItem, HBMMENU_POPUP_CLOSE);
 692
 693     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_ownerdraw_wnd_proc);
 694
 695     if( winetest_debug)
 696         trace("    check %d,%d arrow %d avechar %d\n",
 697                 GetSystemMetrics(SM_CXMENUCHECK ),
 698                 GetSystemMetrics(SM_CYMENUCHECK ),arrowwidth, MOD_avec);
 699     count = 0;
 700     MOD_maxid = 0;
 701     for( ispop=1; ispop >= 0; ispop--){
 702         static SIZE bmsizes[]= {
 703             {10,10},{38,38},{1,30},{55,5}};
 704         for( szidx=0; szidx < sizeof( bmsizes) / sizeof( SIZE); szidx++) {
 705             HBITMAP hbm = CreateBitmap( bmsizes[szidx].cx, bmsizes[szidx].cy,1,1,bmfill);
 706             HBITMAP bitmaps[] = { HBMMENU_CALLBACK, hbm, HBMMENU_POPUP_CLOSE, NULL  };
 707             ok( hbm != 0, "CreateBitmap failed err %d\n", GetLastError());
 708             for( txtidx = 0; txtidx < sizeof(MOD_txtsizes)/sizeof(MOD_txtsizes[0]); txtidx++) {
 709                 for( hassub = 0; hassub < 2 ; hassub++) { /* add submenu item */
 710                     for( mnuopt = 0; mnuopt < 3 ; mnuopt++){ /* test MNS_NOCHECK/MNS_CHECKORBMP */
 711                         for( bmpidx = 0; bmpidx <sizeof(bitmaps)/sizeof(HBITMAP); bmpidx++) {
 712                             /* no need to test NULL bitmaps of several sizes */
 713                             if( !bitmaps[bmpidx] && szidx > 0) continue;
 714                             /* the HBMMENU_POPUP not to test for menu bars */
 715                             if( !ispop &&
 716                                 bitmaps[bmpidx] >= HBMMENU_POPUP_CLOSE &&
 717                                 bitmaps[bmpidx] <= HBMMENU_POPUP_MINIMIZE) continue;
 718                             if( !ispop && hassub) continue;
 719                             test_mbs_help( ispop, hassub, mnuopt,
 720                                     hwnd, arrowwidth, ++count,
 721                                     bitmaps[bmpidx],
 722                                     bmsizes[szidx],
 723                                     MOD_txtsizes[txtidx].text,
 724                                     MOD_txtsizes[txtidx].size,
 725                                     MOD_txtsizes[txtidx].sc_size);
 726                         }
 727                     }
 728                 }
 729             }
 730             DeleteObject( hbm);
 731         }
 732     }
 733     /* clean up */
 734     DestroyWindow(hwnd);
 735 }
 736
 737 static void test_menu_add_string( void )
 738 {
 739     HMENU hmenu;
 740     MENUITEMINFO info;
 741     BOOL rc;
 742     int ret;
 743
 744     char string[0x80];
 745     char string2[0x80];
 746
 747     char strback[0x80];
 748     WCHAR strbackW[0x80];
 749     static CHAR blah[] = "blah";
 750     static const WCHAR expectedString[] = {'D','u','m','m','y',' ','s','t','r','i','n','g', 0};
 751
 752     hmenu = CreateMenu();
 753
 754     memset( &info, 0, sizeof info );
 755     info.cbSize = sizeof info;
 756     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_STATE | MIIM_ID;
 757     info.dwTypeData = blah;
 758     info.cch = 6;
 759     info.dwItemData = 0;
 760     info.wID = 1;
 761     info.fState = 0;
 762     InsertMenuItem(hmenu, 0, TRUE, &info );
 763
 764     memset( &info, 0, sizeof info );
 765     info.cbSize = sizeof info;
 766     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_STATE | MIIM_DATA | MIIM_ID;
 767     info.dwTypeData = string;
 768     info.cch = sizeof string;
 769     string[0] = 0;
 770     GetMenuItemInfo( hmenu, 0, TRUE, &info );
 771
 772     ok( !strcmp( string, "blah" ), "menu item name differed\n");
 773
 774     /* Test combination of ownerdraw and strings with GetMenuItemString(A/W) */
 775     strcpy(string, "Dummy string");
 776     memset(&info, 0x00, sizeof(info));
 777     info.cbSize= sizeof(MENUITEMINFO);
 778     info.fMask= MIIM_FTYPE | MIIM_STRING; /* Set OwnerDraw + typeData */
 779     info.fType= MFT_OWNERDRAW;
 780     info.dwTypeData= string;
 781     rc = InsertMenuItem( hmenu, 0, TRUE, &info );
 782     ok (rc, "InsertMenuItem failed\n");
 783
 784     strcpy(string,"Garbage");
 785     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 786     ok (!strcmp( strback, "Dummy string" ), "Menu text from Ansi version incorrect\n");
 787
 788     SetLastError(0xdeadbeef);
 789     ret = GetMenuStringW( hmenu, 0, strbackW, 99, MF_BYPOSITION );
 790     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
 791         win_skip("GetMenuStringW is not implemented\n");
 792     else
 793     {
 794         ok (ret, "GetMenuStringW on ownerdraw entry failed\n");
 795         ok (!lstrcmpW( strbackW, expectedString ), "Menu text from Unicode version incorrect\n");
 796     }
 797
 798     /* Just change ftype to string and see what text is stored */
 799     memset(&info, 0x00, sizeof(info));
 800     info.cbSize= sizeof(MENUITEMINFO);
 801     info.fMask= MIIM_FTYPE; /* Set string type */
 802     info.fType= MFT_STRING;
 803     info.dwTypeData= (char *)0xdeadbeef;
 804     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 805     ok (rc, "SetMenuItemInfo failed\n");
 806
 807     /* Did we keep the old dwTypeData? */
 808     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 809     ok (!strcmp( strback, "Dummy string" ), "Menu text from Ansi version incorrect\n");
 810
 811     /* Ensure change to bitmap type fails */
 812     memset(&info, 0x00, sizeof(info));
 813     info.cbSize= sizeof(MENUITEMINFO);
 814     info.fMask= MIIM_FTYPE; /* Set as bitmap type */
 815     info.fType= MFT_BITMAP;
 816     info.dwTypeData= (char *)0xdeadbee2;
 817     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 818     ok (!rc, "SetMenuItemInfo unexpectedly worked\n");
 819
 820     /* Just change ftype back and ensure data hasn't been freed */
 821     info.fType= MFT_OWNERDRAW; /* Set as ownerdraw type */
 822     info.dwTypeData= (char *)0xdeadbee3;
 823     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 824     ok (rc, "SetMenuItemInfo failed\n");
 825
 826     /* Did we keep the old dwTypeData? */
 827     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 828     ok (!strcmp( strback, "Dummy string" ), "Menu text from Ansi version incorrect\n");
 829
 830     /* Just change string value (not type) */
 831     memset(&info, 0x00, sizeof(info));
 832     info.cbSize= sizeof(MENUITEMINFO);
 833     info.fMask= MIIM_STRING; /* Set typeData */
 834     strcpy(string2, "string2");
 835     info.dwTypeData= string2;
 836     rc = SetMenuItemInfo( hmenu, 0, TRUE, &info );
 837     ok (rc, "SetMenuItemInfo failed\n");
 838
 839     ok (GetMenuString( hmenu, 0, strback, 99, MF_BYPOSITION), "GetMenuString on ownerdraw entry failed\n");
 840     ok (!strcmp( strback, "string2" ), "Menu text from Ansi version incorrect\n");
 841
 842     /*  crashes with wine 0.9.5 */
 843     memset(&info, 0x00, sizeof(info));
 844     info.cbSize= sizeof(MENUITEMINFO);
 845     info.fMask= MIIM_FTYPE | MIIM_STRING; /* Set OwnerDraw + typeData */
 846     info.fType= MFT_OWNERDRAW;
 847     rc = InsertMenuItem( hmenu, 0, TRUE, &info );
 848     ok (rc, "InsertMenuItem failed\n");
 849     ok (!GetMenuString( hmenu, 0, NULL, 0, MF_BYPOSITION),
 850             "GetMenuString on ownerdraw entry succeeded.\n");
 851     SetLastError(0xdeadbeef);
 852     ret = GetMenuStringW( hmenu, 0, NULL, 0, MF_BYPOSITION);
 853     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
 854         win_skip("GetMenuStringW is not implemented\n");
 855     else
 856         ok (!ret, "GetMenuStringW on ownerdraw entry succeeded.\n");
 857
 858     DestroyMenu( hmenu );
 859 }
 860
 861 /* define building blocks for the menu item info tests */
 862 static int strncmpW( const WCHAR *str1, const WCHAR *str2, int n )
 863 {
 864     if (n <= 0) return 0;
 865     while ((--n > 0) && *str1 && (*str1 == *str2)) { str1++; str2++; }
 866     return *str1 - *str2;
 867 }
 868
 869 static  WCHAR *strcpyW( WCHAR *dst, const WCHAR *src )
 870 {
 871     WCHAR *p = dst;
 872     while ((*p++ = *src++));
 873     return dst;
 874 }
 875
 876 static void insert_menu_item( int line, HMENU hmenu, BOOL ansi, UINT mask, UINT type, UINT state, UINT id,
 877                               HMENU submenu, HBITMAP checked, HBITMAP unchecked, ULONG_PTR data,
 878                               void *type_data, UINT len, HBITMAP item, BOOL expect )
 879 {
 880     MENUITEMINFOA info;
 881     BOOL ret;
 882
 883     /* magic bitmap handle to test smaller cbSize */
 884     if (item == (HBITMAP)(ULONG_PTR)0xdeadbeef)
 885         info.cbSize = FIELD_OFFSET(MENUITEMINFOA,hbmpItem);
 886     else
 887         info.cbSize = sizeof(info);
 888     info.fMask = mask;
 889     info.fType = type;
 890     info.fState = state;
 891     info.wID = id;
 892     info.hSubMenu = submenu;
 893     info.hbmpChecked = checked;
 894     info.hbmpUnchecked = unchecked;
 895     info.dwItemData = data;
 896     info.dwTypeData = type_data;
 897     info.cch = len;
 898     info.hbmpItem = item;
 899     SetLastError( 0xdeadbeef );
 900     if (ansi) ret = InsertMenuItemA( hmenu, 0, TRUE, &info );
 901     else ret = InsertMenuItemW( hmenu, 0, TRUE, (MENUITEMINFOW*)&info );
 902     if (!expect) ok_(__FILE__, line)( !ret, "InsertMenuItem should have failed.\n" );
 903     else ok_(__FILE__, line)( ret, "InsertMenuItem failed, err %u\n", GetLastError());
 904 }
 905
 906 static void check_menu_item_info( int line, HMENU hmenu, BOOL ansi, UINT mask, UINT type, UINT state,
 907                                   UINT id, HMENU submenu, HBITMAP checked, HBITMAP unchecked,
 908                                   ULONG_PTR data, void *type_data, UINT in_len, UINT out_len,
 909                                   HBITMAP item, LPCSTR expname, BOOL expect, BOOL expstring )
 910 {
 911     MENUITEMINFOA info;
 912     BOOL ret;
 913     WCHAR buffer[80];
 914
 915     SetLastError( 0xdeadbeef );
 916     memset( &info, 0xcc, sizeof(info) );
 917     info.cbSize = sizeof(info);
 918     info.fMask = mask;
 919     info.dwTypeData = type_data;
 920     info.cch = in_len;
 921
 922     ret = ansi ? GetMenuItemInfoA( hmenu, 0, TRUE, &info ) :
 923                  GetMenuItemInfoW( hmenu, 0, TRUE, (MENUITEMINFOW *)&info );
 924     if (!expect)
 925     {
 926         ok_(__FILE__, line)( !ret, "GetMenuItemInfo should have failed.\n" );
 927         return;
 928     }
 929     ok_(__FILE__, line)( ret, "GetMenuItemInfo failed, err %u\n", GetLastError());
 930     if (mask & MIIM_TYPE)
 931         ok_(__FILE__, line)( info.fType == type || info.fType == LOWORD(type),
 932                              "wrong type %x/%x\n", info.fType, type );
 933     if (mask & MIIM_STATE)
 934         ok_(__FILE__, line)( info.fState == state || info.fState == LOWORD(state),
 935                              "wrong state %x/%x\n", info.fState, state );
 936     if (mask & MIIM_ID)
 937         ok_(__FILE__, line)( info.wID == id || info.wID == LOWORD(id),
 938                              "wrong id %x/%x\n", info.wID, id );
 939     if (mask & MIIM_SUBMENU)
 940         ok_(__FILE__, line)( info.hSubMenu == submenu || (ULONG_PTR)info.hSubMenu == LOWORD(submenu),
 941                              "wrong submenu %p/%p\n", info.hSubMenu, submenu );
 942     if (mask & MIIM_CHECKMARKS)
 943     {
 944         ok_(__FILE__, line)( info.hbmpChecked == checked || (ULONG_PTR)info.hbmpChecked == LOWORD(checked),
 945                              "wrong bmpchecked %p/%p\n", info.hbmpChecked, checked );
 946         ok_(__FILE__, line)( info.hbmpUnchecked == unchecked || (ULONG_PTR)info.hbmpUnchecked == LOWORD(unchecked),
 947                              "wrong bmpunchecked %p/%p\n", info.hbmpUnchecked, unchecked );
 948     }
 949     if (mask & MIIM_DATA)
 950         ok_(__FILE__, line)( info.dwItemData == data || info.dwItemData == LOWORD(data),
 951                              "wrong item data %lx/%lx\n", info.dwItemData, data );
 952     if (mask & MIIM_BITMAP)
 953         ok_(__FILE__, line)( info.hbmpItem == item || (ULONG_PTR)info.hbmpItem == LOWORD(item),
 954                              "wrong bmpitem %p/%p\n", info.hbmpItem, item );
 955     ok_(__FILE__, line)( info.dwTypeData == type_data || (ULONG_PTR)info.dwTypeData == LOWORD(type_data),
 956                          "wrong type data %p/%p\n", info.dwTypeData, type_data );
 957     ok_(__FILE__, line)( info.cch == out_len ||
 958                          broken(! ansi && info.cch == 2 * out_len) /* East-Asian */,
 959                          "wrong len %x/%x\n", info.cch, out_len );
 960     if (expname)
 961     {
 962         if(ansi)
 963             ok_(__FILE__, line)( !strncmp( expname, info.dwTypeData, out_len ),
 964                                  "menu item name differed from '%s' '%s'\n", expname, info.dwTypeData );
 965         else
 966             ok_(__FILE__, line)( !strncmpW( (WCHAR *)expname, (WCHAR *)info.dwTypeData, out_len ),
 967                                  "menu item name wrong\n" );
 968
 969         SetLastError( 0xdeadbeef );
 970         ret = ansi ? GetMenuStringA( hmenu, 0, (char *)buffer, 80, MF_BYPOSITION ) :
 971             GetMenuStringW( hmenu, 0, buffer, 80, MF_BYPOSITION );
 972         if (expstring)
 973             ok_(__FILE__, line)( ret, "GetMenuString failed, err %u\n", GetLastError());
 974         else
 975             ok_(__FILE__, line)( !ret, "GetMenuString should have failed\n" );
 976     }
 977 }
 978
 979 static void modify_menu( int line, HMENU hmenu, BOOL ansi, UINT flags, UINT_PTR id, void *data )
 980 {
 981     BOOL ret;
 982
 983     SetLastError( 0xdeadbeef );
 984     if (ansi) ret = ModifyMenuA( hmenu, 0, flags, id, data );
 985     else ret = ModifyMenuW( hmenu, 0, flags, id, data );
 986     ok_(__FILE__,line)( ret, "ModifyMenuA failed, err %u\n", GetLastError());
 987 }
 988
 989 static void set_menu_item_info( int line, HMENU hmenu, BOOL ansi, UINT mask, UINT type, UINT state,
 990                                 UINT id, HMENU submenu, HBITMAP checked, HBITMAP unchecked, ULONG_PTR data,
 991                                 void *type_data, UINT len, HBITMAP item )
 992
 993 {
 994     MENUITEMINFOA info;
 995     BOOL ret;
 996
 997     /* magic bitmap handle to test smaller cbSize */
 998     if (item == (HBITMAP)(ULONG_PTR)0xdeadbeef)
 999         info.cbSize = FIELD_OFFSET(MENUITEMINFOA,hbmpItem);
1000     else
1001         info.cbSize = sizeof(info);
1002     info.fMask = mask;
1003     info.fType = type;
1004     info.fState = state;
1005     info.wID = id;
1006     info.hSubMenu = submenu;
1007     info.hbmpChecked = checked;
1008     info.hbmpUnchecked = unchecked;
1009     info.dwItemData = data;
1010     info.dwTypeData = type_data;
1011     info.cch = len;
1012     info.hbmpItem = item;
1013     SetLastError( 0xdeadbeef );
1014     if (ansi) ret = SetMenuItemInfoA( hmenu, 0, TRUE, &info );
1015     else ret = SetMenuItemInfoW( hmenu, 0, TRUE, (MENUITEMINFOW*)&info );
1016     ok_(__FILE__, line)( ret, "SetMenuItemInfo failed, err %u\n", GetLastError());
1017 }
1018
1019 #define TMII_INSMI( c1,d1,e1,f1,g1,h1,i1,j1,k1,l1,m1,eret1 )\
1020     hmenu = CreateMenu();\
1021     submenu = CreateMenu();\
1022     if(ansi)strcpy( string, init );\
1023     else strcpyW( string, init );\
1024     insert_menu_item( __LINE__, hmenu, ansi, c1, d1, e1, f1, g1, h1, i1, j1, k1, l1, m1, eret1 )
1025
1026 /* GetMenuItemInfo + GetMenuString  */
1027 #define TMII_GMII( c2,l2,\
1028     d3,e3,f3,g3,h3,i3,j3,k3,l3,m3,\
1029     expname, eret2, eret3)\
1030     check_menu_item_info( __LINE__, hmenu, ansi, c2, d3, e3, f3, g3, h3, i3, j3, k3, l2, l3, m3, \
1031                           expname, eret2, eret3 )
1032
1033 #define TMII_DONE \
1034     RemoveMenu(hmenu, 0, TRUE );\
1035     DestroyMenu( hmenu );\
1036     DestroyMenu( submenu );
1037
1038 /* modify menu */
1039 #define TMII_MODM( flags, id, data ) \
1040     modify_menu( __LINE__, hmenu, ansi, flags, id, data )
1041
1042 /* SetMenuItemInfo */
1043 #define TMII_SMII( c1,d1,e1,f1,g1,h1,i1,j1,k1,l1,m1 ) \
1044     set_menu_item_info( __LINE__, hmenu, ansi, c1, d1, e1, f1, g1, h1, i1, j1, k1, l1, m1 )
1045
1046
1047 #define OK 1
1048 #define ER 0
1049
1050
1051 static void test_menu_iteminfo( void )
1052 {
1053   int ansi = TRUE;
1054   char txtA[]="wine";
1055   char initA[]="XYZ";
1056   char emptyA[]="";
1057   WCHAR txtW[]={'W','i','n','e',0};
1058   WCHAR initW[]={'X','Y','Z',0};
1059   WCHAR emptyW[]={0};
1060   void *txt, *init, *empty, *string;
1061   HBITMAP hbm = CreateBitmap(1,1,1,1,NULL);
1062   char stringA[0x80];
1063   HMENU hmenu, submenu=CreateMenu();
1064   HBITMAP dummy_hbm = (HBITMAP)(ULONG_PTR)0xdeadbeef;
1065
1066   do {
1067     if( ansi) {txt=txtA;init=initA;empty=emptyA;string=stringA;}
1068     else {txt=txtW;init=initW;empty=emptyW;string=stringA;}
1069     trace( "%s string %p hbm %p txt %p\n", ansi ?  "ANSI tests:   " : "Unicode tests:", string, hbm, txt);
1070     /* test all combinations of MFT_STRING, MFT_OWNERDRAW and MFT_BITMAP */
1071     /* (since MFT_STRING is zero, there are four of them) */
1072     TMII_INSMI( MIIM_TYPE, MFT_STRING, 0, 0, 0, 0, 0, 0, txt, 0, 0, OK );
1073     TMII_GMII ( MIIM_TYPE, 80,
1074         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1075         txt, OK, OK );
1076     TMII_DONE
1077     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1078     TMII_GMII ( MIIM_TYPE, 80,
1079         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1080         NULL, OK, ER );
1081     TMII_DONE
1082     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, 6, 0, OK );
1083     TMII_GMII ( MIIM_TYPE, 80,
1084         MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1085         NULL, OK, ER );
1086     TMII_DONE
1087     TMII_INSMI( MIIM_TYPE, MFT_BITMAP|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, hbm, 6, 0, OK );
1088     TMII_GMII ( MIIM_TYPE, 80,
1089         MFT_BITMAP|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1090         NULL, OK, ER );
1091     TMII_DONE
1092     /* not enough space for name*/
1093     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1094     TMII_GMII ( MIIM_TYPE, 0,
1095         MFT_STRING, 0, 0, 0, 0, 0, 0, NULL, 4, 0,
1096         NULL, OK, OK );
1097     TMII_DONE
1098     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1099     TMII_GMII ( MIIM_TYPE, 5,
1100         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1101         txt, OK, OK );
1102     TMII_DONE
1103     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1104     TMII_GMII ( MIIM_TYPE, 4,
1105         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 3, 0,
1106         txt, OK, OK );
1107     TMII_DONE
1108     TMII_INSMI( MIIM_FTYPE|MIIM_STRING, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1109     TMII_GMII ( MIIM_TYPE, 0,
1110         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 0, 0,
1111         NULL, OK, ER );
1112     TMII_DONE
1113     /* cannot combine MIIM_TYPE with some other flags */
1114     TMII_INSMI( MIIM_TYPE|MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, ER );
1115     TMII_DONE
1116     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1117     TMII_GMII ( MIIM_TYPE|MIIM_STRING, 80,
1118         0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1119         NULL, ER, OK );
1120     TMII_DONE
1121     TMII_INSMI( MIIM_TYPE|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, ER );
1122     TMII_DONE
1123     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1124     TMII_GMII ( MIIM_TYPE|MIIM_FTYPE, 80,
1125         0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1126         NULL, ER, OK );
1127     TMII_DONE
1128     TMII_INSMI( MIIM_TYPE|MIIM_BITMAP, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, 6, hbm, ER );
1129     TMII_DONE
1130         /* but succeeds with some others */
1131     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1132     TMII_GMII ( MIIM_TYPE|MIIM_SUBMENU, 80,
1133         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1134         txt, OK, OK );
1135     TMII_DONE
1136     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1137     TMII_GMII ( MIIM_TYPE|MIIM_STATE, 80,
1138         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1139         txt, OK, OK );
1140     TMII_DONE
1141     TMII_INSMI( MIIM_TYPE|MIIM_ID, MFT_STRING, -1, 888, 0, 0, 0, -1, txt, 6, 0, OK );
1142     TMII_GMII ( MIIM_TYPE|MIIM_ID, 80,
1143         MFT_STRING, 0, 888, 0, 0, 0, 0, string, 4, 0,
1144         txt, OK, OK );
1145     TMII_DONE
1146     TMII_INSMI( MIIM_TYPE|MIIM_DATA, MFT_STRING, -1, -1, 0, 0, 0, 999, txt, 6, 0, OK );
1147     TMII_GMII ( MIIM_TYPE|MIIM_DATA, 80,
1148         MFT_STRING, 0, 0, 0, 0, 0, 999, string, 4, 0,
1149         txt, OK, OK );
1150     TMII_DONE
1151     /* to be continued */
1152     /* set text with MIIM_TYPE and retrieve with MIIM_STRING */
1153     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1154     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1155         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1156         txt, OK, OK );
1157     TMII_DONE
1158     /* set text with MIIM_TYPE and retrieve with MIIM_STRING; MFT_OWNERDRAW causes an empty string */
1159     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1160     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1161         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 0, 0,
1162         empty, OK, ER );
1163     TMII_DONE
1164     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1165     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1166         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 0, 0,
1167         empty, OK, ER );
1168     TMII_DONE
1169     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1170     TMII_GMII ( MIIM_FTYPE, 80,
1171         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 80, 0,
1172         init, OK, ER );
1173     TMII_DONE
1174     TMII_INSMI( MIIM_TYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1175     TMII_GMII ( 0, 80,
1176         0, 0, 0, 0, 0, 0, 0, string, 80, 0,
1177         init, OK, OK );
1178     TMII_DONE
1179     /* contrary to MIIM_TYPE,you can set the text for an owner draw menu */
1180     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1181     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1182         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 4, 0,
1183         txt, OK, OK );
1184     TMII_DONE
1185     /* same but retrieve with MIIM_TYPE */
1186     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1187     TMII_GMII ( MIIM_TYPE, 80,
1188         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1189         NULL, OK, OK );
1190     TMII_DONE
1191     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1192     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1193         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 0, 0,
1194         empty, OK, ER );
1195     TMII_DONE
1196     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1197     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1198         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 0, 0,
1199         empty, OK, ER );
1200     TMII_DONE
1201
1202     /* How is that with bitmaps? */
1203     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1204     TMII_GMII ( MIIM_TYPE, 80,
1205         MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1206         NULL, OK, ER );
1207     TMII_DONE
1208     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1209     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1210         0, 0, 0, 0, 0, 0, 0, string, 80, hbm,
1211         init, OK, ER );
1212     TMII_DONE
1213         /* MIIM_BITMAP does not like MFT_BITMAP */
1214     TMII_INSMI( MIIM_BITMAP|MIIM_FTYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, 0, -1, hbm, ER );
1215     TMII_DONE
1216         /* no problem with OWNERDRAWN */
1217     TMII_INSMI( MIIM_BITMAP|MIIM_FTYPE, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1218     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1219         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 80, hbm,
1220         init, OK, ER );
1221     TMII_DONE
1222         /* setting MFT_BITMAP with MFT_FTYPE fails anyway */
1223     TMII_INSMI( MIIM_FTYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, 0, -1, 0, ER );
1224     TMII_DONE
1225
1226     /* menu with submenu */
1227     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, submenu, 0, 0, -1, txt, 0, 0, OK );
1228     TMII_GMII ( MIIM_SUBMENU, 80,
1229         0, 0, 0, submenu, 0, 0, 0, string, 80, 0,
1230         init, OK, ER );
1231     TMII_DONE
1232     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, submenu, 0, 0, -1, empty, 0, 0, OK );
1233     TMII_GMII ( MIIM_SUBMENU, 80,
1234         0, 0, 0, submenu, 0, 0, 0, string, 80, 0,
1235         init, OK, ER );
1236     TMII_DONE
1237     /* menu with submenu, without MIIM_SUBMENU the submenufield is cleared */
1238     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, submenu, 0, 0, -1, txt, 0, 0, OK );
1239     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1240         MFT_STRING|MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 0, 0,
1241         empty, OK, ER );
1242     TMII_GMII ( MIIM_SUBMENU|MIIM_FTYPE, 80,
1243         MFT_SEPARATOR, 0, 0, submenu, 0, 0, 0, string, 80, 0,
1244         empty, OK, ER );
1245     TMII_DONE
1246     /* menu with invalid submenu */
1247     TMII_INSMI( MIIM_SUBMENU|MIIM_FTYPE, MFT_STRING, -1, -1, (HMENU)999, 0, 0, -1, txt, 0, 0, ER );
1248     TMII_DONE
1249     /* Separator */
1250     TMII_INSMI( MIIM_TYPE, MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, txt, 0, 0, OK );
1251     TMII_GMII ( MIIM_TYPE, 80,
1252         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1253         NULL, OK, ER );
1254     TMII_DONE
1255     TMII_INSMI( MIIM_TYPE, MFT_BITMAP|MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, hbm, 6, 0, OK );
1256     TMII_GMII ( MIIM_TYPE, 80,
1257         MFT_BITMAP|MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1258         NULL, OK, ER );
1259     TMII_DONE
1260      /* SEPARATOR and STRING go well together */
1261     /* BITMAP and STRING go well together */
1262     TMII_INSMI( MIIM_STRING|MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1263     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, 80,
1264         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, hbm,
1265         txt, OK, OK );
1266     TMII_DONE
1267      /* BITMAP, SEPARATOR and STRING go well together */
1268     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1269     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, 80,
1270         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 4, hbm,
1271         txt, OK, OK );
1272     TMII_DONE
1273      /* last two tests, but use MIIM_TYPE to retrieve info */
1274     TMII_INSMI( MIIM_FTYPE|MIIM_STRING, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1275     TMII_GMII ( MIIM_TYPE, 80,
1276         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1277         NULL, OK, OK );
1278     TMII_DONE
1279     TMII_INSMI( MIIM_STRING|MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1280     TMII_GMII ( MIIM_TYPE, 80,
1281         MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1282         NULL, OK, OK );
1283     TMII_DONE
1284     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1285     TMII_GMII ( MIIM_TYPE, 80,
1286         MFT_SEPARATOR|MFT_BITMAP, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1287         NULL, OK, OK );
1288     TMII_DONE
1289      /* same three with MFT_OWNERDRAW */
1290     TMII_INSMI( MIIM_FTYPE|MIIM_STRING, MFT_SEPARATOR|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, 0, OK );
1291     TMII_GMII ( MIIM_TYPE, 80,
1292         MFT_SEPARATOR|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1293         NULL, OK, OK );
1294     TMII_DONE
1295     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1296     TMII_GMII ( MIIM_TYPE, 80,
1297         MFT_BITMAP|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1298         NULL, OK, OK );
1299     TMII_DONE
1300     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1301     TMII_GMII ( MIIM_TYPE, 80,
1302         MFT_SEPARATOR|MFT_BITMAP|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 4, hbm,
1303         NULL, OK, OK );
1304     TMII_DONE
1305
1306     TMII_INSMI( MIIM_STRING|MIIM_FTYPE|MIIM_ID, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1307     TMII_GMII ( MIIM_TYPE, 80,
1308         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1309         NULL,  OK, OK );
1310     TMII_DONE
1311     /* test with modifymenu: string is preserved after setting OWNERDRAW */
1312     TMII_INSMI( MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1313     TMII_MODM( MFT_OWNERDRAW, -1, (void*)787 );
1314     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_DATA, 80,
1315         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 787, string, 4, 0,
1316         txt,  OK, OK );
1317     TMII_DONE
1318     /* same with bitmap: now the text is cleared */
1319     TMII_INSMI( MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1320     TMII_MODM( MFT_BITMAP, 545, hbm );
1321     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1322         MFT_BITMAP, 0, 545, 0, 0, 0, 0, string, 0, hbm,
1323         empty,  OK, ER );
1324     TMII_DONE
1325     /* start with bitmap: now setting text clears it (though he flag is raised) */
1326     TMII_INSMI( MIIM_BITMAP, MFT_STRING, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1327     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1328         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 0, hbm,
1329         empty,  OK, ER );
1330     TMII_MODM( MFT_STRING, 545, txt );
1331     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1332         MFT_STRING, 0, 545, 0, 0, 0, 0, string, 4, 0,
1333         txt,  OK, OK );
1334     TMII_DONE
1335     /*repeat with text NULL */
1336     TMII_INSMI( MIIM_BITMAP, MFT_STRING, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1337     TMII_MODM( MFT_STRING, 545, NULL );
1338     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1339         MFT_SEPARATOR, 0, 545, 0, 0, 0, 0, string, 0, 0,
1340         empty,  OK, ER );
1341     TMII_DONE
1342     /* repeat with text "" */
1343     TMII_INSMI( MIIM_BITMAP, -1 , -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1344     TMII_MODM( MFT_STRING, 545, empty );
1345     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_ID, 80,
1346         MFT_STRING, 0, 545, 0, 0, 0, 0, string, 0, 0,
1347         empty,  OK, ER );
1348     TMII_DONE
1349     /* start with bitmap: set ownerdraw */
1350     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, hbm, OK );
1351     TMII_MODM( MFT_OWNERDRAW, -1, (void *)232 );
1352     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP|MIIM_DATA, 80,
1353         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 232, string, 0, hbm,
1354         empty,  OK, ER );
1355     TMII_DONE
1356     /* ask nothing */
1357     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, hbm, OK );
1358     TMII_GMII ( 0, 80,
1359                 0, 0, 0,  0, 0, 0, 0, string, 80, 0,
1360         init, OK, OK );
1361     TMII_DONE
1362     /* some tests with small cbSize: the hbmpItem is to be ignored */
1363     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, dummy_hbm, OK );
1364     TMII_GMII ( MIIM_TYPE, 80,
1365         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, NULL, 0, NULL,
1366         NULL, OK, ER );
1367     TMII_DONE
1368     TMII_INSMI( MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, 0, -1, dummy_hbm, OK );
1369     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1370         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 80, NULL,
1371         init, OK, ER );
1372     TMII_DONE
1373     TMII_INSMI( MIIM_STRING|MIIM_BITMAP, -1, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1374     TMII_GMII ( MIIM_TYPE, 80,
1375         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, NULL,
1376         txt, OK, OK );
1377     TMII_DONE
1378     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1379     TMII_GMII ( MIIM_TYPE, 80,
1380         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1381         NULL, OK, OK );
1382     TMII_DONE
1383     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1384     TMII_GMII ( MIIM_TYPE, 80,
1385         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1386         NULL, OK, OK );
1387     TMII_DONE
1388     TMII_INSMI( MIIM_FTYPE|MIIM_STRING|MIIM_BITMAP, MFT_SEPARATOR|MFT_OWNERDRAW, -1, -1, 0, 0, 0, -1, txt, 6, dummy_hbm, OK );
1389     TMII_GMII ( MIIM_TYPE, 80,
1390         MFT_SEPARATOR|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 4, NULL,
1391         NULL, OK, OK );
1392     TMII_DONE
1393     /* MIIM_TYPE by itself does not get/set the dwItemData for OwnerDrawn menus  */
1394     TMII_INSMI( MIIM_TYPE|MIIM_DATA, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, 343, txt, 0, 0, OK );
1395     TMII_GMII ( MIIM_TYPE|MIIM_DATA, 80,
1396         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 343, 0, 0, 0,
1397         NULL, OK, ER );
1398     TMII_DONE
1399     TMII_INSMI( MIIM_TYPE|MIIM_DATA, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, 343, txt, 0, 0, OK );
1400     TMII_GMII ( MIIM_TYPE, 80,
1401         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1402         NULL, OK, ER );
1403     TMII_DONE
1404     TMII_INSMI( MIIM_TYPE, MFT_STRING|MFT_OWNERDRAW, -1, -1, 0, 0, 0, 343, txt, 0, 0, OK );
1405     TMII_GMII ( MIIM_TYPE|MIIM_DATA, 80,
1406         MFT_STRING|MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0,
1407         NULL, OK, ER );
1408     TMII_DONE
1409     /* set a string menu to ownerdraw with MIIM_TYPE */
1410     TMII_INSMI( MIIM_TYPE, MFT_STRING, -2, -2, 0, 0, 0, -2, txt, -2, 0, OK );
1411     TMII_SMII ( MIIM_TYPE, MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 0, 0 );
1412     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1413         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, string, 4, 0,
1414         txt, OK, OK );
1415     TMII_DONE
1416     /* test with modifymenu add submenu */
1417     TMII_INSMI( MIIM_STRING, MFT_STRING, -1, -1, 0, 0, 0, -1, txt, 0, 0, OK );
1418     TMII_MODM( MF_POPUP, (UINT_PTR)submenu, txt );
1419     TMII_GMII ( MIIM_FTYPE|MIIM_STRING|MIIM_SUBMENU, 80,
1420         MFT_STRING, 0, 0, submenu, 0, 0, 0, string, 4, 0,
1421         txt,  OK, OK );
1422     TMII_GMII ( MIIM_TYPE, 80,
1423         MFT_STRING, 0, 0, 0, 0, 0, 0, string, 4, 0,
1424         txt,  OK, OK );
1425     TMII_DONE
1426     /* MFT_SEPARATOR bit is kept when the text is added */
1427     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1428     TMII_SMII( MIIM_STRING, 0, 0, 0, 0, 0, 0, 0, txt, 0, 0 );
1429     TMII_GMII ( MIIM_STRING|MIIM_FTYPE, 80,
1430         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 4, 0,
1431         txt, OK, OK );
1432     TMII_DONE
1433     /* MFT_SEPARATOR bit is kept when bitmap is added */
1434     TMII_INSMI( MIIM_STRING|MIIM_FTYPE, MFT_STRING, -1, -1, 0, 0, 0, -1, NULL, 0, 0, OK );
1435     TMII_SMII( MIIM_BITMAP, 0, 0, 0, 0, 0, 0, 0, 0, 0, hbm );
1436     TMII_GMII ( MIIM_BITMAP|MIIM_FTYPE, 80,
1437         MFT_SEPARATOR, 0, 0, 0, 0, 0, 0, string, 80, hbm,
1438         init, OK, ER );
1439     TMII_DONE
1440     /* Bitmaps inserted with MIIM_TYPE and MFT_BITMAP:
1441        Only the low word of the dwTypeData is used.
1442        Use a magic bitmap here (Word 95 uses this to create its MDI menu buttons) */
1443     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_RIGHTJUSTIFY, -1, -1, 0, 0, 0, -1,
1444                 (HMENU)MAKELONG(HBMMENU_MBAR_CLOSE, 0x1234), -1, 0, OK );
1445     TMII_GMII ( MIIM_TYPE, 80,
1446         MFT_BITMAP | MFT_RIGHTJUSTIFY, 0, 0, 0, 0, 0, 0, HBMMENU_MBAR_CLOSE, 0, HBMMENU_MBAR_CLOSE,
1447         NULL, OK, OK );
1448     TMII_DONE
1449     /* Type flags */
1450     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1451     TMII_GMII ( MIIM_TYPE, 80,
1452         MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1453         NULL, OK, OK );
1454     TMII_DONE
1455     /* State flags */
1456     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1457     TMII_SMII( MIIM_STATE, -1, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 0, 0 );
1458     TMII_GMII ( MIIM_STATE, 80,
1459         0, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 80, 0,
1460         NULL, OK, OK );
1461     TMII_DONE
1462     /* The style MFT_RADIOCHECK cannot be set with MIIM_CHECKMARKS only */
1463     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1464     TMII_SMII( MIIM_CHECKMARKS, MFT_RADIOCHECK, 0, 0, 0, hbm, hbm, 0, 0, 0, 0 );
1465     TMII_GMII ( MIIM_CHECKMARKS | MIIM_TYPE, 80,
1466         MFT_BITMAP, 0, 0, 0, hbm, hbm, 0, hbm, 0, hbm,
1467         NULL, OK, OK );
1468     TMII_DONE
1469     /* MFT_BITMAP is added automatically by GetMenuItemInfo() for MIIM_TYPE */
1470     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1471     TMII_SMII( MIIM_FTYPE, MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, (HBITMAP)0x1234, 0, 0 );
1472     TMII_GMII ( MIIM_FTYPE, 80,
1473         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1474         NULL, OK, OK );
1475     TMII_GMII ( MIIM_TYPE, 80,
1476         MFT_BITMAP | MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1477         NULL, OK, OK );
1478     TMII_GMII ( MIIM_FTYPE, 80,
1479         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1480         NULL, OK, OK );
1481     TMII_SMII( MIIM_BITMAP, 0, 0, 0, 0, 0, 0, 0, 0, 0, NULL );
1482     TMII_GMII ( MIIM_TYPE, 80,
1483         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 0, NULL,
1484         NULL, OK, OK );
1485     TMII_DONE
1486     /* Bitmaps inserted with MIIM_TYPE and MFT_BITMAP:
1487        Only the low word of the dwTypeData is used.
1488        Use a magic bitmap here (Word 95 uses this to create its MDI menu buttons) */
1489     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_RIGHTJUSTIFY, -1, -1, 0, 0, 0, -1,
1490                 (HMENU)MAKELONG(HBMMENU_MBAR_CLOSE, 0x1234), -1, 0, OK );
1491     TMII_GMII ( MIIM_TYPE, 80,
1492         MFT_BITMAP | MFT_RIGHTJUSTIFY, 0, 0, 0, 0, 0, 0, HBMMENU_MBAR_CLOSE, 0, HBMMENU_MBAR_CLOSE,
1493         NULL, OK, OK );
1494     TMII_DONE
1495     /* Type flags */
1496     TMII_INSMI( MIIM_TYPE, MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1497     TMII_GMII ( MIIM_TYPE, 80,
1498         MFT_BITMAP | MFT_MENUBARBREAK | MFT_RADIOCHECK | MFT_RIGHTJUSTIFY | MFT_RIGHTORDER, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1499         NULL, OK, OK );
1500     TMII_DONE
1501     /* State flags */
1502     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1503     TMII_SMII( MIIM_STATE, -1, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 0, 0 );
1504     TMII_GMII ( MIIM_STATE, 80,
1505         0, MFS_CHECKED | MFS_DEFAULT | MFS_GRAYED | MFS_HILITE, 0, 0, 0, 0, 0, 0, 80, 0,
1506         NULL, OK, OK );
1507     TMII_DONE
1508     /* The style MFT_RADIOCHECK cannot be set with MIIM_CHECKMARKS only */
1509     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1510     TMII_SMII( MIIM_CHECKMARKS, MFT_RADIOCHECK, 0, 0, 0, hbm, hbm, 0, 0, 0, 0 );
1511     TMII_GMII ( MIIM_CHECKMARKS | MIIM_TYPE, 80,
1512         MFT_BITMAP, 0, 0, 0, hbm, hbm, 0, hbm, 0, hbm,
1513         NULL, OK, OK );
1514     TMII_DONE
1515     /* MFT_BITMAP is added automatically by GetMenuItemInfo() for MIIM_TYPE */
1516     TMII_INSMI( MIIM_TYPE, MFT_BITMAP, -1, -1, 0, 0, 0, -1, hbm, -1, 0, OK );
1517     TMII_SMII( MIIM_FTYPE, MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, (HBITMAP)0x1234, 0, 0 );
1518     TMII_GMII ( MIIM_FTYPE, 80,
1519         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1520         NULL, OK, OK );
1521     TMII_GMII ( MIIM_TYPE, 80,
1522         MFT_BITMAP | MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, hbm, 0, hbm,
1523         NULL, OK, OK );
1524     TMII_GMII ( MIIM_FTYPE, 80,
1525         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, 0, 80, 0,
1526         NULL, OK, OK );
1527     TMII_SMII( MIIM_BITMAP, 0, 0, 0, 0, 0, 0, 0, 0, 0, NULL );
1528     TMII_GMII ( MIIM_TYPE, 80,
1529         MFT_OWNERDRAW, 0, 0, 0, 0, 0, 0, NULL, 0, NULL,
1530         NULL, OK, OK );
1531     TMII_DONE
1532   } while( !(ansi = !ansi) );
1533   DeleteObject( hbm);
1534 }
1535
1536 /*
1537    The following tests try to confirm the algorithm used to return the menu items
1538    when there is a collision between a menu item and a popup menu
1539  */
1540 static void test_menu_search_bycommand( void )
1541 {
1542     HMENU        hmenu, hmenuSub, hmenuSub2;
1543     MENUITEMINFO info;
1544     BOOL         rc;
1545     UINT         id;
1546     char         strback[0x80];
1547     char         strIn[0x80];
1548     static CHAR menuitem[]  = "MenuItem",
1549                 menuitem2[] = "MenuItem 2";
1550
1551     /* Case 1: Menu containing a menu item */
1552     hmenu = CreateMenu();
1553
1554     memset( &info, 0, sizeof info );
1555     info.cbSize = sizeof info;
1556     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1557     info.fType = MFT_STRING;
1558     strcpy(strIn, "Case 1 MenuItem");
1559     info.dwTypeData = strIn;
1560     info.wID = (UINT) 0x1234;
1561
1562     rc = InsertMenuItem(hmenu, 0, TRUE, &info );
1563     ok (rc, "Inserting the menuitem failed\n");
1564
1565     id = GetMenuItemID(hmenu, 0);
1566     ok (id == 0x1234, "Getting the menuitem id failed(gave %x)\n", id);
1567
1568     /* Confirm the menuitem was given the id supplied (getting by position) */
1569     memset( &info, 0, sizeof info );
1570     strback[0] = 0x00;
1571     info.cbSize = sizeof(MENUITEMINFO);
1572     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1573     info.dwTypeData = strback;
1574     info.cch = sizeof(strback);
1575
1576     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info); /* Get by position */
1577     ok (rc, "Getting the menu items info failed\n");
1578     ok (info.wID == 0x1234, "IDs differ for the menuitem\n");
1579     ok (!strcmp(info.dwTypeData, "Case 1 MenuItem"), "Returned item has wrong label\n");
1580
1581     /* Search by id - Should return the item */
1582     memset( &info, 0, sizeof info );
1583     strback[0] = 0x00;
1584     info.cbSize = sizeof(MENUITEMINFO);
1585     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1586     info.dwTypeData = strback;
1587     info.cch = sizeof(strback);
1588     rc = GetMenuItemInfo(hmenu, 0x1234, FALSE, &info); /* Get by ID */
1589
1590     ok (rc, "Getting the menu items info failed\n");
1591     ok (info.wID == 0x1234, "IDs differ for the menuitem\n");
1592     ok (!strcmp(info.dwTypeData, "Case 1 MenuItem"), "Returned item has wrong label\n");
1593
1594     DestroyMenu( hmenu );
1595
1596     /* Case 2: Menu containing a popup menu */
1597     hmenu = CreateMenu();
1598     hmenuSub = CreateMenu();
1599
1600     strcpy(strIn, "Case 2 SubMenu");
1601     rc = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, strIn);
1602     ok (rc, "Inserting the popup menu into the main menu failed\n");
1603
1604     id = GetMenuItemID(hmenu, 0);
1605     ok (id == -1, "Getting the menuitem id unexpectedly worked (gave %x)\n", id);
1606
1607     /* Confirm the menuitem itself was given an id the same as the HMENU, (getting by position) */
1608     memset( &info, 0, sizeof info );
1609     strback[0] = 0x00;
1610     info.cbSize = sizeof(MENUITEMINFO);
1611     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1612     info.dwTypeData = strback;
1613     info.cch = sizeof(strback);
1614     info.wID = 0xdeadbeef;
1615
1616     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info); /* Get by position */
1617     ok (rc, "Getting the menu items info failed\n");
1618     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the menuitem\n");
1619     ok (!strcmp(info.dwTypeData, "Case 2 SubMenu"), "Returned item has wrong label\n");
1620
1621     /* Search by id - returns the popup menu itself */
1622     memset( &info, 0, sizeof info );
1623     strback[0] = 0x00;
1624     info.cbSize = sizeof(MENUITEMINFO);
1625     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1626     info.dwTypeData = strback;
1627     info.cch = sizeof(strback);
1628     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info); /* Get by ID */
1629
1630     ok (rc, "Getting the menu items info failed\n");
1631     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the popup menu\n");
1632     ok (!strcmp(info.dwTypeData, "Case 2 SubMenu"), "Returned item has wrong label\n");
1633
1634     /*
1635         Now add an item after it with the same id
1636      */
1637     memset( &info, 0, sizeof info );
1638     info.cbSize = sizeof info;
1639     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1640     info.fType = MFT_STRING;
1641     strcpy(strIn, "Case 2 MenuItem 1");
1642     info.dwTypeData = strIn;
1643     info.wID = (UINT_PTR) hmenuSub;
1644     rc = InsertMenuItem(hmenu, -1, TRUE, &info );
1645     ok (rc, "Inserting the menuitem failed\n");
1646
1647     /* Search by id - returns the item which follows the popup menu */
1648     memset( &info, 0, sizeof info );
1649     strback[0] = 0x00;
1650     info.cbSize = sizeof(MENUITEMINFO);
1651     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1652     info.dwTypeData = strback;
1653     info.cch = sizeof(strback);
1654     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info); /* Get by ID */
1655
1656     ok (rc, "Getting the menu items info failed\n");
1657     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the popup menu\n");
1658     ok (!strcmp(info.dwTypeData, "Case 2 MenuItem 1"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1659
1660     /*
1661         Now add an item before the popup (with the same id)
1662      */
1663     memset( &info, 0, sizeof info );
1664     info.cbSize = sizeof info;
1665     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1666     info.fType = MFT_STRING;
1667     strcpy(strIn, "Case 2 MenuItem 2");
1668     info.dwTypeData = strIn;
1669     info.wID = (UINT_PTR) hmenuSub;
1670     rc = InsertMenuItem(hmenu, 0, TRUE, &info );
1671     ok (rc, "Inserting the menuitem failed\n");
1672
1673     /* Search by id - returns the item which precedes the popup menu */
1674     memset( &info, 0, sizeof info );
1675     strback[0] = 0x00;
1676     info.cbSize = sizeof(MENUITEMINFO);
1677     info.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STRING;
1678     info.dwTypeData = strback;
1679     info.cch = sizeof(strback);
1680     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info); /* Get by ID */
1681
1682     ok (rc, "Getting the menu items info failed\n");
1683     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for the popup menu\n");
1684     ok (!strcmp(info.dwTypeData, "Case 2 MenuItem 2"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1685
1686     DestroyMenu( hmenu );
1687     DestroyMenu( hmenuSub );
1688
1689     /*
1690         Case 3: Menu containing a popup menu which in turn
1691            contains 2 items with the same id as the popup itself
1692      */
1693
1694     hmenu = CreateMenu();
1695     hmenuSub = CreateMenu();
1696
1697     memset( &info, 0, sizeof info );
1698     info.cbSize = sizeof info;
1699     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1700     info.fType = MFT_STRING;
1701     info.dwTypeData = menuitem;
1702     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1703
1704     rc = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, "Submenu");
1705     ok (rc, "Inserting the popup menu into the main menu failed\n");
1706
1707     rc = InsertMenuItem(hmenuSub, 0, TRUE, &info );
1708     ok (rc, "Inserting the sub menu menuitem failed\n");
1709
1710     memset( &info, 0, sizeof info );
1711     info.cbSize = sizeof info;
1712     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1713     info.fType = MFT_STRING;
1714     info.dwTypeData = menuitem2;
1715     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1716
1717     rc = InsertMenuItem(hmenuSub, 1, TRUE, &info );
1718     ok (rc, "Inserting the sub menu menuitem 2 failed\n");
1719
1720     /* Prove that you can't query the id of a popup directly (By position) */
1721     id = GetMenuItemID(hmenu, 0);
1722     ok (id == -1, "Getting the sub menu id should have failed because its a popup (gave %x)\n", id);
1723
1724     /* Prove getting the item info via ID returns the first item (not the popup or 2nd item)*/
1725     memset( &info, 0, sizeof info );
1726     strback[0] = 0x00;
1727     info.cbSize = sizeof(MENUITEMINFO);
1728     info.fMask = MIIM_STRING | MIIM_ID;
1729     info.dwTypeData = strback;
1730     info.cch = sizeof(strback);
1731
1732     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info);
1733     ok (rc, "Getting the menus info failed\n");
1734     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for popup menu\n");
1735     ok (!strcmp(info.dwTypeData, "MenuItem"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1736     DestroyMenu( hmenu );
1737     DestroyMenu( hmenuSub );
1738
1739     /*
1740         Case 4: Menu containing 2 popup menus, the second
1741            contains 2 items with the same id as the first popup menu
1742      */
1743     hmenu = CreateMenu();
1744     hmenuSub = CreateMenu();
1745     hmenuSub2 = CreateMenu();
1746
1747     rc = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, "Submenu");
1748     ok (rc, "Inserting the popup menu into the main menu failed\n");
1749
1750     rc = InsertMenu(hmenu, 1, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub2, "Submenu2");
1751     ok (rc, "Inserting the popup menu into the main menu failed\n");
1752
1753     memset( &info, 0, sizeof info );
1754     info.cbSize = sizeof info;
1755     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1756     info.fType = MFT_STRING;
1757     info.dwTypeData = menuitem;
1758     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1759
1760     rc = InsertMenuItem(hmenuSub2, 0, TRUE, &info );
1761     ok (rc, "Inserting the sub menu menuitem failed\n");
1762
1763     memset( &info, 0, sizeof info );
1764     info.cbSize = sizeof info;
1765     info.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_ID;
1766     info.fType = MFT_STRING;
1767     info.dwTypeData = menuitem2;
1768     info.wID = (UINT_PTR) hmenuSub; /* Enforce id collisions with the hmenu of the popup submenu*/
1769
1770     rc = InsertMenuItem(hmenuSub2, 1, TRUE, &info );
1771     ok (rc, "Inserting the sub menu menuitem 2 failed\n");
1772
1773     /* Prove getting the item info via ID returns the first item (not the popup or 2nd item)*/
1774     memset( &info, 0, sizeof info );
1775     strback[0] = 0x00;
1776     info.cbSize = sizeof(MENUITEMINFO);
1777     info.fMask = MIIM_STRING | MIIM_ID;
1778     info.dwTypeData = strback;
1779     info.cch = sizeof(strback);
1780
1781     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub, FALSE, &info);
1782     ok (rc, "Getting the menus info failed\n");
1783     ok (info.wID == (UINT_PTR)hmenuSub, "IDs differ for popup menu\n");
1784     ok (!strcmp(info.dwTypeData, "MenuItem"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1785
1786     memset( &info, 0, sizeof info );
1787     strback[0] = 0x00;
1788     info.cbSize = sizeof(MENUITEMINFO);
1789     info.fMask = MIIM_STRING | MIIM_ID;
1790     info.dwTypeData = strback;
1791     info.cch = sizeof(strback);
1792
1793     rc = GetMenuItemInfo(hmenu, (UINT_PTR)hmenuSub2, FALSE, &info);
1794     ok (rc, "Getting the menus info failed\n");
1795     ok (info.wID == (UINT_PTR)hmenuSub2, "IDs differ for popup menu\n");
1796     ok (!strcmp(info.dwTypeData, "Submenu2"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1797
1798     DestroyMenu( hmenu );
1799     DestroyMenu( hmenuSub );
1800     DestroyMenu( hmenuSub2 );
1801
1802
1803     /*
1804         Case 5: Menu containing a popup menu which in turn
1805            contains an item with a different id than the popup menu.
1806            This tests the fallback to a popup menu ID.
1807      */
1808
1809     hmenu = CreateMenu();
1810     hmenuSub = CreateMenu();
1811
1812     rc = AppendMenu(hmenu, MF_POPUP | MF_STRING, (UINT_PTR)hmenuSub, "Submenu");
1813     ok (rc, "Appending the popup menu to the main menu failed\n");
1814
1815     rc = AppendMenu(hmenuSub, MF_STRING, 102, "Item");
1816     ok (rc, "Appending the item to the popup menu failed\n");
1817
1818     /* Set the ID for hmenuSub */
1819     info.cbSize = sizeof(info);
1820     info.fMask = MIIM_ID;
1821     info.wID = 101;
1822
1823     rc = SetMenuItemInfo(hmenu, 0, TRUE, &info);
1824     ok(rc, "Setting the ID for the popup menu failed\n");
1825
1826     /* Check if the ID has been set */
1827     info.wID = 0;
1828     rc = GetMenuItemInfo(hmenu, 0, TRUE, &info);
1829     ok(rc, "Getting the ID for the popup menu failed\n");
1830     ok(info.wID == 101, "The ID for the popup menu has not been set\n");
1831
1832     /* Prove getting the item info via ID returns the popup menu */
1833     memset( &info, 0, sizeof(info));
1834     strback[0] = 0x00;
1835     info.cbSize = sizeof(MENUITEMINFO);
1836     info.fMask = MIIM_STRING | MIIM_ID;
1837     info.dwTypeData = strback;
1838     info.cch = sizeof(strback);
1839
1840     rc = GetMenuItemInfo(hmenu, 101, FALSE, &info);
1841     ok (rc, "Getting the menu info failed\n");
1842     ok (info.wID == 101, "IDs differ\n");
1843     ok (!strcmp(info.dwTypeData, "Submenu"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1844
1845     /* Also look for the menu item  */
1846     memset( &info, 0, sizeof(info));
1847     strback[0] = 0x00;
1848     info.cbSize = sizeof(MENUITEMINFO);
1849     info.fMask = MIIM_STRING | MIIM_ID;
1850     info.dwTypeData = strback;
1851     info.cch = sizeof(strback);
1852
1853     rc = GetMenuItemInfo(hmenu, 102, FALSE, &info);
1854     ok (rc, "Getting the menu info failed\n");
1855     ok (info.wID == 102, "IDs differ\n");
1856     ok (!strcmp(info.dwTypeData, "Item"), "Returned item has wrong label (%s)\n", info.dwTypeData);
1857
1858     DestroyMenu(hmenu);
1859     DestroyMenu(hmenuSub);
1860 }
1861
1862 struct menu_item_pair_s {
1863     UINT uMenu; /* 1 - top level menu, [0-Menu 1-Enabled 2-Disabled]
1864                  * 2 - 2nd level menu, [0-Popup 1-Enabled 2-Disabled]
1865                  * 3 - 3rd level menu, [0-Enabled 1-Disabled] */
1866     UINT uItem;
1867 };
1868
1869 static struct menu_mouse_tests_s {
1870     DWORD type;
1871     struct menu_item_pair_s menu_item_pairs[5]; /* for mousing */
1872     WORD wVk[5]; /* keys */
1873     BOOL bMenuVisible;
1874     BOOL _todo_wine;
1875 } menu_tests[] = {
1876     /* for each test, send keys or clicks and check for menu visibility */
1877     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 0}, TRUE, FALSE }, /* test 0 */
1878     { INPUT_KEYBOARD, {{0}}, {VK_ESCAPE, 0}, FALSE, FALSE },
1879     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 0}, TRUE, FALSE },
1880     { INPUT_KEYBOARD, {{0}}, {'D', 0}, FALSE, FALSE },
1881     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 0}, TRUE, FALSE },
1882     { INPUT_KEYBOARD, {{0}}, {'E', 0}, FALSE, FALSE },
1883     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 0}, TRUE, FALSE },
1884     { INPUT_KEYBOARD, {{0}}, {VK_ESCAPE, VK_ESCAPE, 0}, FALSE, FALSE },
1885     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', VK_ESCAPE, 0}, TRUE, FALSE },
1886     { INPUT_KEYBOARD, {{0}}, {VK_ESCAPE, 0}, FALSE, FALSE },
1887     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 0}, TRUE, FALSE },
1888     { INPUT_KEYBOARD, {{0}}, {'D', 0}, FALSE, FALSE },
1889     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 0}, TRUE, FALSE },
1890     { INPUT_KEYBOARD, {{0}}, {'E', 0}, FALSE, FALSE },
1891     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 'P', 0}, TRUE, FALSE },
1892     { INPUT_KEYBOARD, {{0}}, {'D', 0}, FALSE, FALSE },
1893     { INPUT_KEYBOARD, {{0}}, {VK_MENU, 'M', 'P', 0}, TRUE, FALSE },
1894     { INPUT_KEYBOARD, {{0}}, {'E', 0}, FALSE, FALSE },
1895     { INPUT_KEYBOARD, {{0}}, {VK_F10, 0}, TRUE, FALSE },
1896     { INPUT_KEYBOARD, {{0}}, {VK_F10, 0}, FALSE, FALSE },
1897
1898     { INPUT_MOUSE, {{1, 2}, {0}}, {0}, TRUE, TRUE }, /* test 20 */
1899     { INPUT_MOUSE, {{1, 1}, {0}}, {0}, FALSE, FALSE },
1900     { INPUT_MOUSE, {{1, 0}, {0}}, {0}, TRUE, TRUE },
1901     { INPUT_MOUSE, {{1, 1}, {0}}, {0}, FALSE, FALSE },
1902     { INPUT_MOUSE, {{1, 0}, {2, 2}, {0}}, {0}, TRUE, TRUE },
1903     { INPUT_MOUSE, {{2, 1}, {0}}, {0}, FALSE, FALSE },
1904     { INPUT_MOUSE, {{1, 0}, {2, 0}, {0}}, {0}, TRUE, TRUE },
1905     { INPUT_MOUSE, {{3, 0}, {0}}, {0}, FALSE, FALSE },
1906     { INPUT_MOUSE, {{1, 0}, {2, 0}, {0}}, {0}, TRUE, TRUE },
1907     { INPUT_MOUSE, {{3, 1}, {0}}, {0}, TRUE, TRUE },
1908     { INPUT_MOUSE, {{1, 1}, {0}}, {0}, FALSE, FALSE },
1909     { -1 }
1910 };
1911
1912 static void send_key(WORD wVk)
1913 {
1914     TEST_INPUT i[2];
1915     memset(i, 0, sizeof(i));
1916     i[0].type = i[1].type = INPUT_KEYBOARD;
1917     i[0].u.ki.wVk = i[1].u.ki.wVk = wVk;
1918     i[1].u.ki.dwFlags = KEYEVENTF_KEYUP;
1919     pSendInput(2, (INPUT *) i, sizeof(INPUT));
1920 }
1921
1922 static BOOL click_menu(HANDLE hWnd, struct menu_item_pair_s *mi)
1923 {
1924     HMENU hMenu = hMenus[mi->uMenu];
1925     TEST_INPUT i[3];
1926     MSG msg;
1927     RECT r;
1928     int screen_w = GetSystemMetrics(SM_CXSCREEN);
1929     int screen_h = GetSystemMetrics(SM_CYSCREEN);
1930     BOOL ret = GetMenuItemRect(mi->uMenu > 2 ? NULL : hWnd, hMenu, mi->uItem, &r);
1931     if(!ret) return FALSE;
1932
1933     memset(i, 0, sizeof(i));
1934     i[0].type = i[1].type = i[2].type = INPUT_MOUSE;
1935     i[0].u.mi.dx = i[1].u.mi.dx = i[2].u.mi.dx
1936             = ((r.left + 5) * 65535) / screen_w;
1937     i[0].u.mi.dy = i[1].u.mi.dy = i[2].u.mi.dy
1938             = ((r.top + 5) * 65535) / screen_h;
1939     i[0].u.mi.dwFlags = i[1].u.mi.dwFlags = i[2].u.mi.dwFlags
1940             = MOUSEEVENTF_ABSOLUTE;
1941     i[0].u.mi.dwFlags |= MOUSEEVENTF_MOVE;
1942     i[1].u.mi.dwFlags |= MOUSEEVENTF_LEFTDOWN;
1943     i[2].u.mi.dwFlags |= MOUSEEVENTF_LEFTUP;
1944     ret = pSendInput(3, (INPUT *) i, sizeof(INPUT));
1945
1946     /* hack to prevent mouse message buildup in Wine */
1947     while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessageA( &msg );
1948     return ret;
1949 }
1950
1951 static DWORD WINAPI test_menu_input_thread(LPVOID lpParameter)
1952 {
1953     int i, j;
1954     HANDLE hWnd = lpParameter;
1955
1956     Sleep(500);
1957     /* mixed keyboard/mouse test */
1958     for (i = 0; menu_tests[i].type != -1; i++)
1959     {
1960         int ret = TRUE, elapsed = 0;
1961
1962         got_input = i && menu_tests[i-1].bMenuVisible;
1963
1964         if (menu_tests[i].type == INPUT_KEYBOARD)
1965             for (j = 0; menu_tests[i].wVk[j] != 0; j++)
1966                 send_key(menu_tests[i].wVk[j]);
1967         else
1968             for (j = 0; menu_tests[i].menu_item_pairs[j].uMenu != 0; j++)
1969                 if (!(ret = click_menu(hWnd, &menu_tests[i].menu_item_pairs[j]))) break;
1970
1971         if (!ret)
1972         {
1973             skip( "test %u: failed to send input\n", i );
1974             PostMessage( hWnd, WM_CANCELMODE, 0, 0 );
1975             return 0;
1976         }
1977         while (menu_tests[i].bMenuVisible != bMenuVisible)
1978         {
1979             if (elapsed > 200)
1980                 break;
1981             elapsed += 20;
1982             Sleep(20);
1983         }
1984
1985         if (!got_input)
1986         {
1987             skip( "test %u: didn't receive input\n", i );
1988             PostMessage( hWnd, WM_CANCELMODE, 0, 0 );
1989             return 0;
1990         }
1991
1992         if (menu_tests[i]._todo_wine)
1993         {
1994             todo_wine {
1995                 ok(menu_tests[i].bMenuVisible == bMenuVisible, "test %d\n", i);
1996             }
1997         }
1998         else
1999             ok(menu_tests[i].bMenuVisible == bMenuVisible, "test %d\n", i);
2000     }
2001     return 0;
2002 }
2003
2004 static LRESULT CALLBACK WndProc(HWND hWnd, UINT msg, WPARAM wParam,
2005         LPARAM lParam)
2006 {
2007     switch (msg) {
2008         case WM_ENTERMENULOOP:
2009             bMenuVisible = TRUE;
2010             break;
2011         case WM_EXITMENULOOP:
2012             bMenuVisible = FALSE;
2013             break;
2014         case WM_KEYDOWN:
2015         case WM_SYSKEYDOWN:
2016         case WM_MOUSEMOVE:
2017         case WM_LBUTTONDOWN:
2018         case WM_LBUTTONUP:
2019         case WM_NCMOUSEMOVE:
2020         case WM_NCLBUTTONDOWN:
2021         case WM_NCLBUTTONUP:
2022             got_input = TRUE;
2023             /* fall through */
2024         default:
2025             return( DefWindowProcA( hWnd, msg, wParam, lParam ) );
2026     }
2027     return 0;
2028 }
2029
2030 static void test_menu_input(void) {
2031     MSG msg;
2032     WNDCLASSA  wclass;
2033     HINSTANCE hInstance = GetModuleHandleA( NULL );
2034     HANDLE hThread, hWnd;
2035     DWORD tid;
2036
2037     if (!pSendInput)
2038     {
2039         win_skip("SendInput is not available\n");
2040         return;
2041     }
2042
2043     wclass.lpszClassName = "MenuTestClass";
2044     wclass.style         = CS_HREDRAW | CS_VREDRAW;
2045     wclass.lpfnWndProc   = WndProc;
2046     wclass.hInstance     = hInstance;
2047     wclass.hIcon         = LoadIconA( 0, IDI_APPLICATION );
2048     wclass.hCursor       = LoadCursorA( NULL, IDC_ARROW );
2049     wclass.hbrBackground = (HBRUSH)( COLOR_WINDOW + 1 );
2050     wclass.lpszMenuName  = 0;
2051     wclass.cbClsExtra    = 0;
2052     wclass.cbWndExtra    = 0;
2053     assert (RegisterClassA( &wclass ));
2054     assert (hWnd = CreateWindowA( wclass.lpszClassName, "MenuTest",
2055                                   WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, 0,
2056                                   400, 200, NULL, NULL, hInstance, NULL) );
2057
2058     /* fixed menus */
2059     hMenus[3] = CreatePopupMenu();
2060     AppendMenu(hMenus[3], MF_STRING, 0, "&Enabled");
2061     AppendMenu(hMenus[3], MF_STRING|MF_DISABLED, 0, "&Disabled");
2062
2063     hMenus[2] = CreatePopupMenu();
2064     AppendMenu(hMenus[2], MF_STRING|MF_POPUP, (UINT_PTR) hMenus[3], "&Popup");
2065     AppendMenu(hMenus[2], MF_STRING, 0, "&Enabled");
2066     AppendMenu(hMenus[2], MF_STRING|MF_DISABLED, 0, "&Disabled");
2067
2068     hMenus[1] = CreateMenu();
2069     AppendMenu(hMenus[1], MF_STRING|MF_POPUP, (UINT_PTR) hMenus[2], "&Menu");
2070     AppendMenu(hMenus[1], MF_STRING, 0, "&Enabled");
2071     AppendMenu(hMenus[1], MF_STRING|MF_DISABLED, 0, "&Disabled");
2072
2073     SetMenu(hWnd, hMenus[1]);
2074     ShowWindow(hWnd, SW_SHOW);
2075     UpdateWindow(hWnd);
2076
2077     hThread = CreateThread(NULL, 0, test_menu_input_thread, hWnd, 0, &tid);
2078     while(1)
2079     {
2080         if (WAIT_TIMEOUT != WaitForSingleObject(hThread, 50))
2081             break;
2082         while (PeekMessage(&msg, 0, 0, 0, PM_REMOVE)) DispatchMessageA(&msg);
2083     }
2084     DestroyWindow(hWnd);
2085 }
2086
2087 static void test_menu_flags( void )
2088 {
2089     HMENU hMenu, hPopupMenu;
2090
2091     hMenu = CreateMenu();
2092     hPopupMenu = CreatePopupMenu();
2093
2094     AppendMenu(hMenu, MF_POPUP | MF_STRING, (UINT_PTR)hPopupMenu, "Popup");
2095
2096     AppendMenu(hPopupMenu, MF_STRING | MF_HILITE | MF_DEFAULT, 101, "Item 1");
2097     InsertMenu(hPopupMenu, 1, MF_BYPOSITION | MF_STRING | MF_HILITE | MF_DEFAULT, 102, "Item 2");
2098     AppendMenu(hPopupMenu, MF_STRING, 103, "Item 3");
2099     ModifyMenu(hPopupMenu, 2, MF_BYPOSITION | MF_STRING | MF_HILITE | MF_DEFAULT, 103, "Item 3");
2100
2101     ok(GetMenuState(hPopupMenu, 0, MF_BYPOSITION) & MF_HILITE,
2102       "AppendMenu should accept MF_HILITE\n");
2103     ok(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE,
2104       "InsertMenu should accept MF_HILITE\n");
2105     ok(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_HILITE,
2106       "ModifyMenu should accept MF_HILITE\n");
2107
2108     ok(!(GetMenuState(hPopupMenu, 0, MF_BYPOSITION) & MF_DEFAULT),
2109       "AppendMenu must not accept MF_DEFAULT\n");
2110     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_DEFAULT),
2111       "InsertMenu must not accept MF_DEFAULT\n");
2112     ok(!(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_DEFAULT),
2113       "ModifyMenu must not accept MF_DEFAULT\n");
2114
2115     DestroyMenu(hMenu);
2116 }
2117
2118 static void test_menu_hilitemenuitem( void )
2119 {
2120     HMENU hMenu, hPopupMenu;
2121     WNDCLASSA wclass;
2122     HWND hWnd;
2123
2124     wclass.lpszClassName = "HiliteMenuTestClass";
2125     wclass.style         = CS_HREDRAW | CS_VREDRAW;
2126     wclass.lpfnWndProc   = WndProc;
2127     wclass.hInstance     = GetModuleHandleA( NULL );
2128     wclass.hIcon         = LoadIconA( 0, IDI_APPLICATION );
2129     wclass.hCursor       = LoadCursorA( NULL, IDC_ARROW );
2130     wclass.hbrBackground = (HBRUSH)( COLOR_WINDOW + 1 );
2131     wclass.lpszMenuName  = 0;
2132     wclass.cbClsExtra    = 0;
2133     wclass.cbWndExtra    = 0;
2134     assert (RegisterClassA( &wclass ));
2135     assert (hWnd = CreateWindowA( wclass.lpszClassName, "HiliteMenuTest",
2136                                   WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, 0,
2137                                   400, 200, NULL, NULL, wclass.hInstance, NULL) );
2138
2139     hMenu = CreateMenu();
2140     hPopupMenu = CreatePopupMenu();
2141
2142     AppendMenu(hMenu, MF_POPUP | MF_STRING, (UINT_PTR)hPopupMenu, "Popup");
2143
2144     AppendMenu(hPopupMenu, MF_STRING, 101, "Item 1");
2145     AppendMenu(hPopupMenu, MF_STRING, 102, "Item 2");
2146     AppendMenu(hPopupMenu, MF_STRING, 103, "Item 3");
2147
2148     SetMenu(hWnd, hMenu);
2149
2150     /* test invalid arguments */
2151
2152     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2153       "HiliteMenuItem: Item 2 is hilited\n");
2154
2155     SetLastError(0xdeadbeef);
2156     todo_wine
2157     {
2158     ok(!HiliteMenuItem(NULL, hPopupMenu, 1, MF_HILITE | MF_BYPOSITION),
2159       "HiliteMenuItem: call should have failed.\n");
2160     }
2161     ok(GetLastError() == 0xdeadbeef || /* 9x */
2162        GetLastError() == ERROR_INVALID_WINDOW_HANDLE /* NT */,
2163       "HiliteMenuItem: expected error ERROR_INVALID_WINDOW_HANDLE, got: %d\n", GetLastError());
2164
2165     SetLastError(0xdeadbeef);
2166     ok(!HiliteMenuItem(hWnd, NULL, 1, MF_HILITE | MF_BYPOSITION),
2167       "HiliteMenuItem: call should have failed.\n");
2168     ok(GetLastError() == 0xdeadbeef || /* 9x */
2169        GetLastError() == ERROR_INVALID_MENU_HANDLE /* NT */,
2170       "HiliteMenuItem: expected error ERROR_INVALID_MENU_HANDLE, got: %d\n", GetLastError());
2171
2172     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2173       "HiliteMenuItem: Item 2 is hilited\n");
2174
2175     /* either MF_HILITE or MF_UNHILITE *and* MF_BYCOMMAND or MF_BYPOSITION need to be set */
2176
2177     SetLastError(0xdeadbeef);
2178     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_BYPOSITION),
2179       "HiliteMenuItem: call should have succeeded.\n");
2180     ok(GetLastError() == 0xdeadbeef,
2181       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2182
2183     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2184       "HiliteMenuItem: Item 2 is hilited\n");
2185
2186     SetLastError(0xdeadbeef);
2187     todo_wine
2188     {
2189     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_HILITE),
2190       "HiliteMenuItem: call should have succeeded.\n");
2191     }
2192     ok(GetLastError() == 0xdeadbeef,
2193       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2194
2195     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2196       "HiliteMenuItem: Item 2 is hilited\n");
2197
2198     /* hilite a menu item (by position) */
2199
2200     SetLastError(0xdeadbeef);
2201     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_HILITE | MF_BYPOSITION),
2202       "HiliteMenuItem: call should not have failed.\n");
2203     ok(GetLastError() == 0xdeadbeef,
2204       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2205
2206     todo_wine
2207     {
2208     ok(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE,
2209       "HiliteMenuItem: Item 2 is not hilited\n");
2210     }
2211
2212     /* unhilite a menu item (by position) */
2213
2214     SetLastError(0xdeadbeef);
2215     ok(HiliteMenuItem(hWnd, hPopupMenu, 1, MF_UNHILITE | MF_BYPOSITION),
2216       "HiliteMenuItem: call should not have failed.\n");
2217     ok(GetLastError() == 0xdeadbeef,
2218       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2219
2220     ok(!(GetMenuState(hPopupMenu, 1, MF_BYPOSITION) & MF_HILITE),
2221       "HiliteMenuItem: Item 2 is hilited\n");
2222
2223     /* hilite a menu item (by command) */
2224
2225     SetLastError(0xdeadbeef);
2226     ok(HiliteMenuItem(hWnd, hPopupMenu, 103, MF_HILITE | MF_BYCOMMAND),
2227       "HiliteMenuItem: call should not have failed.\n");
2228     ok(GetLastError() == 0xdeadbeef,
2229       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2230
2231     todo_wine
2232     {
2233     ok(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_HILITE,
2234       "HiliteMenuItem: Item 3 is not hilited\n");
2235     }
2236
2237     /* unhilite a menu item (by command) */
2238
2239     SetLastError(0xdeadbeef);
2240     ok(HiliteMenuItem(hWnd, hPopupMenu, 103, MF_UNHILITE | MF_BYCOMMAND),
2241       "HiliteMenuItem: call should not have failed.\n");
2242     ok(GetLastError() == 0xdeadbeef,
2243       "HiliteMenuItem: expected error 0xdeadbeef, got: %d\n", GetLastError());
2244
2245     ok(!(GetMenuState(hPopupMenu, 2, MF_BYPOSITION) & MF_HILITE),
2246       "HiliteMenuItem: Item 3 is hilited\n");
2247
2248     DestroyWindow(hWnd);
2249 }
2250
2251 static void check_menu_items(HMENU hmenu, UINT checked_cmd, UINT checked_type,
2252                              UINT checked_state)
2253 {
2254     INT i, count;
2255
2256     count = GetMenuItemCount(hmenu);
2257     ok (count != -1, "GetMenuItemCount returned -1\n");
2258
2259     for (i = 0; i < count; i++)
2260     {
2261         BOOL ret;
2262         MENUITEMINFO mii;
2263
2264         memset(&mii, 0, sizeof(mii));
2265         mii.cbSize = sizeof(mii);
2266         mii.fMask  = MIIM_FTYPE | MIIM_STATE | MIIM_ID | MIIM_SUBMENU;
2267         ret = GetMenuItemInfo(hmenu, i, TRUE, &mii);
2268         ok(ret, "GetMenuItemInfo(%u) failed\n", i);
2269 #if 0
2270         trace("item #%u: fType %04x, fState %04x, wID %u, hSubMenu %p\n",
2271                i, mii.fType, mii.fState, mii.wID, mii.hSubMenu);
2272 #endif
2273         if (mii.hSubMenu)
2274         {
2275             ok(mii.wID == (UINT_PTR)mii.hSubMenu, "id %u: wID should be equal to hSubMenu\n", checked_cmd);
2276             check_menu_items(mii.hSubMenu, checked_cmd, checked_type, checked_state);
2277         }
2278         else
2279         {
2280             if (mii.wID == checked_cmd)
2281             {
2282                 ok(mii.fType == checked_type, "id %u: expected fType %04x, got %04x\n", checked_cmd, checked_type, mii.fType);
2283                 ok(mii.fState == checked_state, "id %u: expected fState %04x, got %04x\n", checked_cmd, checked_state, mii.fState);
2284                 ok(mii.wID != 0, "id %u: not expected wID 0\n", checked_cmd);
2285             }
2286             else
2287             {
2288                 ok(mii.fType != MFT_RADIOCHECK, "id %u: not expected fType MFT_RADIOCHECK on cmd %u\n", checked_cmd, mii.wID);
2289
2290                 if (mii.fType == MFT_SEPARATOR)
2291                 {
2292                     ok(mii.fState == MFS_GRAYED, "id %u: expected fState MFS_GRAYED, got %04x\n", checked_cmd, mii.fState);
2293                     ok(mii.wID == 0, "id %u: expected wID 0, got %u\n", checked_cmd, mii.wID);
2294                 }
2295                 else
2296                 {
2297                     ok(mii.fState == 0, "id %u: expected fState 0, got %04x\n", checked_cmd, mii.fState);
2298                     ok(mii.wID != 0, "id %u: not expected wID 0\n", checked_cmd);
2299                 }
2300             }
2301         }
2302     }
2303 }
2304
2305 static void clear_ftype_and_state(HMENU hmenu, UINT id, UINT flags)
2306 {
2307     BOOL ret;
2308     MENUITEMINFO mii;
2309
2310     memset(&mii, 0, sizeof(mii));
2311     mii.cbSize = sizeof(mii);
2312     mii.fMask  = MIIM_FTYPE | MIIM_STATE;
2313     ret = SetMenuItemInfo(hmenu, id, (flags & MF_BYPOSITION) != 0, &mii);
2314     ok(ret, "SetMenuItemInfo(%u) failed\n", id);
2315 }
2316
2317 static void test_CheckMenuRadioItem(void)
2318 {
2319     BOOL ret;
2320     HMENU hmenu;
2321
2322     hmenu = LoadMenu(GetModuleHandle(0), MAKEINTRESOURCE(1));
2323     assert(hmenu != 0);
2324
2325     check_menu_items(hmenu, -1, 0, 0);
2326
2327     ret = CheckMenuRadioItem(hmenu, 100, 100, 100, MF_BYCOMMAND);
2328     ok(ret, "CheckMenuRadioItem failed\n");
2329     check_menu_items(hmenu, 100, MFT_RADIOCHECK, MFS_CHECKED);
2330
2331     /* MSDN is wrong, Windows does not remove MFT_RADIOCHECK */
2332     ret = CheckMenuRadioItem(hmenu, 100, 100, -1, MF_BYCOMMAND);
2333     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2334     check_menu_items(hmenu, 100, MFT_RADIOCHECK, 0);
2335
2336     /* clear check */
2337     clear_ftype_and_state(hmenu, 100, MF_BYCOMMAND);
2338     check_menu_items(hmenu, -1, 0, 0);
2339
2340     /* first and checked items are on different menus */
2341     ret = CheckMenuRadioItem(hmenu, 0, 300, 202, MF_BYCOMMAND);
2342     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2343     check_menu_items(hmenu, -1, 0, 0);
2344
2345     ret = CheckMenuRadioItem(hmenu, 200, 300, 202, MF_BYCOMMAND);
2346     ok(ret, "CheckMenuRadioItem failed\n");
2347     check_menu_items(hmenu, 202, MFT_RADIOCHECK, MFS_CHECKED);
2348
2349     /* MSDN is wrong, Windows does not remove MFT_RADIOCHECK */
2350     ret = CheckMenuRadioItem(hmenu, 202, 202, -1, MF_BYCOMMAND);
2351     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2352     check_menu_items(hmenu, 202, MFT_RADIOCHECK, 0);
2353
2354     /* clear check */
2355     clear_ftype_and_state(hmenu, 202, MF_BYCOMMAND);
2356     check_menu_items(hmenu, -1, 0, 0);
2357
2358     /* just for fun, try to check separator */
2359     ret = CheckMenuRadioItem(hmenu, 0, 300, 0, MF_BYCOMMAND);
2360     ok(!ret, "CheckMenuRadioItem should return FALSE\n");
2361     check_menu_items(hmenu, -1, 0, 0);
2362 }
2363
2364 static void test_menu_resource_layout(void)
2365 {
2366     static const struct
2367     {
2368         MENUITEMTEMPLATEHEADER mith;
2369         WORD data[14];
2370     } menu_template =
2371     {
2372         { 0, 0 }, /* versionNumber, offset */
2373         {
2374             /* mtOption, mtID, mtString[] '\0' terminated */
2375             MF_STRING, 1, 'F', 0,
2376             MF_STRING, 2, 0,
2377             MF_SEPARATOR, 3, 0,
2378             /* MF_SEPARATOR, 4, 'S', 0, FIXME: Wine ignores 'S' */
2379             MF_STRING|MF_GRAYED|MF_END, 5, 'E', 0
2380         }
2381     };
2382     static const struct
2383     {
2384         UINT type, state, id;
2385         const char *str;
2386     } menu_data[] =
2387     {
2388         { MF_STRING, MF_ENABLED, 1, "F" },
2389         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 2, "" },
2390         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 3, "" },
2391         /*{ MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 4, "S" }, FIXME: Wine ignores 'S'*/
2392         { MF_STRING, MF_GRAYED, 5, "E" },
2393         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 6, "" },
2394         { MF_STRING, MF_ENABLED, 7, "" },
2395         { MF_SEPARATOR, MF_GRAYED|MF_DISABLED, 8, "" }
2396     };
2397     HMENU hmenu;
2398     INT count, i;
2399     BOOL ret;
2400
2401     hmenu = LoadMenuIndirect(&menu_template);
2402     ok(hmenu != 0, "LoadMenuIndirect error %u\n", GetLastError());
2403
2404     ret = AppendMenu(hmenu, MF_STRING, 6, NULL);
2405     ok(ret, "AppendMenu failed\n");
2406     ret = AppendMenu(hmenu, MF_STRING, 7, "\0");
2407     ok(ret, "AppendMenu failed\n");
2408     ret = AppendMenu(hmenu, MF_SEPARATOR, 8, "separator");
2409     ok(ret, "AppendMenu failed\n");
2410
2411     count = GetMenuItemCount(hmenu);
2412     ok(count == sizeof(menu_data)/sizeof(menu_data[0]),
2413        "expected %u menu items, got %u\n",
2414        (UINT)(sizeof(menu_data)/sizeof(menu_data[0])), count);
2415
2416     for (i = 0; i < count; i++)
2417     {
2418         char buf[20];
2419         MENUITEMINFO mii;
2420
2421         memset(&mii, 0, sizeof(mii));
2422         mii.cbSize = sizeof(mii);
2423         mii.dwTypeData = buf;
2424         mii.cch = sizeof(buf);
2425         mii.fMask  = MIIM_FTYPE | MIIM_STATE | MIIM_ID | MIIM_STRING;
2426         ret = GetMenuItemInfo(hmenu, i, TRUE, &mii);
2427         ok(ret, "GetMenuItemInfo(%u) failed\n", i);
2428 #if 0
2429         trace("item #%u: fType %04x, fState %04x, wID %u, dwTypeData %s\n",
2430                i, mii.fType, mii.fState, mii.wID, (LPCSTR)mii.dwTypeData);
2431 #endif
2432         ok(mii.fType == menu_data[i].type,
2433            "%u: expected fType %04x, got %04x\n", i, menu_data[i].type, mii.fType);
2434         ok(mii.fState == menu_data[i].state,
2435            "%u: expected fState %04x, got %04x\n", i, menu_data[i].state, mii.fState);
2436         ok(mii.wID == menu_data[i].id,
2437            "%u: expected wID %04x, got %04x\n", i, menu_data[i].id, mii.wID);
2438         ok(mii.cch == strlen(menu_data[i].str),
2439            "%u: expected cch %u, got %u\n", i, (UINT)strlen(menu_data[i].str), mii.cch);
2440         ok(!strcmp(mii.dwTypeData, menu_data[i].str),
2441            "%u: expected dwTypeData %s, got %s\n", i, menu_data[i].str, (LPCSTR)mii.dwTypeData);
2442     }
2443
2444     DestroyMenu(hmenu);
2445 }
2446
2447 struct menu_data
2448 {
2449     UINT type, id;
2450     const char *str;
2451 };
2452
2453 static HMENU create_menu_from_data(const struct menu_data *item, INT item_count)
2454 {
2455     HMENU hmenu;
2456     INT i;
2457     BOOL ret;
2458
2459     hmenu = CreateMenu();
2460     assert(hmenu != 0);
2461
2462     for (i = 0; i < item_count; i++)
2463     {
2464         SetLastError(0xdeadbeef);
2465         ret = AppendMenu(hmenu, item[i].type, item[i].id, item[i].str);
2466         ok(ret, "%d: AppendMenu(%04x, %04x, %p) error %u\n",
2467            i, item[i].type, item[i].id, item[i].str, GetLastError());
2468     }
2469     return hmenu;
2470 }
2471
2472 /* use InsertMenuItem: does not set the MFT_BITMAP flag,
2473  * and does not accept non-magic bitmaps with invalid
2474  * bitmap handles */
2475 static HMENU create_menuitem_from_data(const struct menu_data *item, INT item_count)
2476 {
2477     HMENU hmenu;
2478     INT i;
2479     BOOL ret;
2480     MENUITEMINFO mii = { sizeof( MENUITEMINFO)};
2481
2482     hmenu = CreateMenu();
2483     assert(hmenu != 0);
2484
2485     for (i = 0; i < item_count; i++)
2486     {
2487         SetLastError(0xdeadbeef);
2488
2489         mii.fMask = MIIM_FTYPE | MIIM_ID | MIIM_STATE;
2490         mii.fType = 0;
2491         if(  item[i].type & MFT_BITMAP)
2492         {
2493             mii.fMask |= MIIM_BITMAP;
2494             mii.hbmpItem =  (HBITMAP)item[i].str;
2495         }
2496         else if(  item[i].type & MFT_SEPARATOR)
2497             mii.fType = MFT_SEPARATOR;
2498         else
2499         {
2500             mii.fMask |= MIIM_STRING;
2501             mii.dwTypeData =  (LPSTR)item[i].str;
2502             mii.cch = strlen(  item[i].str);
2503         }
2504         mii.fState = 0;
2505         if(  item[i].type & MF_HELP) mii.fType |= MF_HELP;
2506         mii.wID = item[i].id;
2507         ret = InsertMenuItem( hmenu, -1, TRUE, &mii);
2508         ok(ret, "%d: InsertMenuItem(%04x, %04x, %p) error %u\n",
2509            i, item[i].type, item[i].id, item[i].str, GetLastError());
2510     }
2511     return hmenu;
2512 }
2513
2514 static void compare_menu_data(HMENU hmenu, const struct menu_data *item, INT item_count)
2515 {
2516     INT count, i;
2517     BOOL ret;
2518
2519     count = GetMenuItemCount(hmenu);
2520     ok(count == item_count, "expected %d, got %d menu items\n", count, item_count);
2521
2522     for (i = 0; i < count; i++)
2523     {
2524         char buf[20];
2525         MENUITEMINFO mii;
2526
2527         memset(&mii, 0, sizeof(mii));
2528         mii.cbSize = sizeof(mii);
2529         mii.dwTypeData = buf;
2530         mii.cch = sizeof(buf);
2531         mii.fMask  = MIIM_FTYPE | MIIM_ID | MIIM_STRING | MIIM_BITMAP;
2532         ret = GetMenuItemInfo(hmenu, i, TRUE, &mii);
2533         ok(ret, "GetMenuItemInfo(%u) failed\n", i);
2534 #if 0
2535         trace("item #%u: fType %04x, fState %04x, wID %04x, hbmp %p\n",
2536                i, mii.fType, mii.fState, mii.wID, mii.hbmpItem);
2537 #endif
2538         ok(mii.fType == item[i].type,
2539            "%u: expected fType %04x, got %04x\n", i, item[i].type, mii.fType);
2540         ok(mii.wID == item[i].id,
2541            "%u: expected wID %04x, got %04x\n", i, item[i].id, mii.wID);
2542         if (mii.hbmpItem || !item[i].str)
2543             /* For some reason Windows sets high word to not 0 for
2544              * not "magic" ids.
2545              */
2546             ok(LOWORD(mii.hbmpItem) == LOWORD(item[i].str),
2547                "%u: expected hbmpItem %p, got %p\n", i, item[i].str, mii.hbmpItem);
2548         else
2549         {
2550             ok(mii.cch == strlen(item[i].str),
2551                "%u: expected cch %u, got %u\n", i, (UINT)strlen(item[i].str), mii.cch);
2552             ok(!strcmp(mii.dwTypeData, item[i].str),
2553                "%u: expected dwTypeData %s, got %s\n", i, item[i].str, (LPCSTR)mii.dwTypeData);
2554         }
2555     }
2556 }
2557
2558 static void test_InsertMenu(void)
2559 {
2560     HBITMAP hbm = CreateBitmap(1,1,1,1,NULL);
2561     /* Note: XP treats only bitmap handles 1 - 6 as "magic" ones
2562      * regardless of their id.
2563      */
2564     static const struct menu_data in1[] =
2565     {
2566         { MF_STRING, 1, "File" },
2567         { MF_BITMAP|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) },
2568         { MF_STRING|MF_HELP, 2, "Help" }
2569     };
2570     static const struct menu_data out1[] =
2571     {
2572         { MF_STRING, 1, "File" },
2573         { MF_STRING|MF_HELP, 2, "Help" },
2574         { MF_BITMAP|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) }
2575     };
2576     static const struct menu_data out1a[] =
2577     {
2578         { MF_STRING, 1, "File" },
2579         { MF_STRING|MF_HELP, 2, "Help" },
2580         { MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) }
2581     };
2582     const struct menu_data in2[] =
2583     {
2584         { MF_STRING, 1, "File" },
2585         { MF_BITMAP|MF_HELP, SC_CLOSE, (char*)hbm },
2586         { MF_STRING|MF_HELP, 2, "Help" }
2587     };
2588     const struct menu_data out2[] =
2589     {
2590         { MF_STRING, 1, "File" },
2591         { MF_BITMAP|MF_HELP, SC_CLOSE, (char*)hbm },
2592         { MF_STRING|MF_HELP, 2, "Help" }
2593     };
2594     const struct menu_data out2a[] =
2595     {
2596         { MF_STRING, 1, "File" },
2597         { MF_HELP, SC_CLOSE, (char*)hbm },
2598         { MF_STRING|MF_HELP, 2, "Help" }
2599     };
2600     static const struct menu_data in3[] =
2601     {
2602         { MF_STRING, 1, "File" },
2603         { MF_SEPARATOR|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(1) },
2604         { MF_STRING|MF_HELP, 2, "Help" }
2605     };
2606     static const struct menu_data out3[] =
2607     {
2608         { MF_STRING, 1, "File" },
2609         { MF_SEPARATOR|MF_HELP, SC_CLOSE, MAKEINTRESOURCE(0) },
2610         { MF_STRING|MF_HELP, 2, "Help" },
2611     };
2612     static const struct menu_data in4[] =
2613     {
2614         { MF_STRING, 1, "File" },
2615         { MF_BITMAP|MF_HELP, 1, MAKEINTRESOURCE(1) },
2616         { MF_STRING|MF_HELP, 2, "Help" }
2617     };
2618     static const struct menu_data out4[] =
2619     {
2620         { MF_STRING, 1, "File" },
2621         { MF_STRING|MF_HELP, 2, "Help" },
2622         { MF_BITMAP|MF_HELP, 1, MAKEINTRESOURCE(1) }
2623     };
2624     static const struct menu_data out4a[] =
2625     {
2626         { MF_STRING, 1, "File" },
2627         { MF_STRING|MF_HELP, 2, "Help" },
2628         { MF_HELP, 1, MAKEINTRESOURCE(1) }
2629     };
2630     HMENU hmenu;
2631
2632 #define create_menu(a) create_menu_from_data((a), sizeof(a)/sizeof((a)[0]))
2633 #define create_menuitem(a) create_menuitem_from_data((a), sizeof(a)/sizeof((a)[0]))
2634 #define compare_menu(h, a) compare_menu_data((h), (a), sizeof(a)/sizeof((a)[0]))
2635
2636     hmenu = create_menu(in1);
2637     compare_menu(hmenu, out1);
2638     DestroyMenu(hmenu);
2639
2640     hmenu = create_menu(in2);
2641     compare_menu(hmenu, out2);
2642     DestroyMenu(hmenu);
2643
2644     hmenu = create_menu(in3);
2645     compare_menu(hmenu, out3);
2646     DestroyMenu(hmenu);
2647
2648     hmenu = create_menu(in4);
2649     compare_menu(hmenu, out4);
2650     DestroyMenu(hmenu);
2651
2652     /* now using InsertMenuItemInfo */
2653     hmenu = create_menuitem(in1);
2654     compare_menu(hmenu, out1a);
2655     DestroyMenu(hmenu);
2656
2657     hmenu = create_menuitem(in2);
2658     compare_menu(hmenu, out2a);
2659     DestroyMenu(hmenu);
2660
2661     hmenu = create_menuitem(in3);
2662     compare_menu(hmenu, out3);
2663     DestroyMenu(hmenu);
2664
2665     hmenu = create_menuitem(in4);
2666     compare_menu(hmenu, out4a);
2667     DestroyMenu(hmenu);
2668
2669 #undef create_menu
2670 #undef create_menuitem
2671 #undef compare_menu
2672 }
2673
2674 static void test_menu_getmenuinfo(void)
2675 {
2676     HMENU hmenu;
2677     MENUINFO mi = {0};
2678     BOOL ret;
2679     DWORD gle;
2680
2681     if (!pGetMenuInfo)
2682     {
2683         win_skip("GetMenuInfo is not available\n");
2684         return;
2685     }
2686
2687     /* create a menu */
2688     hmenu = CreateMenu();
2689     assert( hmenu);
2690     /* test some parameter errors */
2691     SetLastError(0xdeadbeef);
2692     ret = pGetMenuInfo( hmenu, NULL);
2693     gle= GetLastError();
2694     ok( !ret, "GetMenuInfo() should have failed\n");
2695     ok( gle == ERROR_INVALID_PARAMETER ||
2696         broken(gle == 0xdeadbeef), /* Win98, WinME */
2697         "GetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2698     SetLastError(0xdeadbeef);
2699     mi.cbSize = 0;
2700     ret = pGetMenuInfo( hmenu, &mi);
2701     gle= GetLastError();
2702     ok( !ret, "GetMenuInfo() should have failed\n");
2703     ok( gle == ERROR_INVALID_PARAMETER ||
2704         broken(gle == 0xdeadbeef), /* Win98, WinME */
2705         "GetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2706     SetLastError(0xdeadbeef);
2707     mi.cbSize = sizeof( MENUINFO);
2708     ret = pGetMenuInfo( hmenu, &mi);
2709     gle= GetLastError();
2710     ok( ret, "GetMenuInfo() should have succeeded\n");
2711     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2712     SetLastError(0xdeadbeef);
2713     mi.cbSize = 0;
2714     ret = pGetMenuInfo( NULL, &mi);
2715     gle= GetLastError();
2716     ok( !ret, "GetMenuInfo() should have failed\n");
2717     ok( gle == ERROR_INVALID_PARAMETER ||
2718         broken(gle == 0xdeadbeef), /* Win98, WinME */
2719         "GetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2720     /* clean up */
2721     DestroyMenu( hmenu);
2722     return;
2723 }
2724
2725 static void test_menu_setmenuinfo(void)
2726 {
2727     HMENU hmenu, hsubmenu;
2728     MENUINFO mi = {0};
2729     MENUITEMINFOA mii = {sizeof( MENUITEMINFOA)};
2730     BOOL ret;
2731     DWORD gle;
2732
2733     if (!pGetMenuInfo || !pSetMenuInfo)
2734     {
2735         win_skip("Get/SetMenuInfo are not available\n");
2736         return;
2737     }
2738
2739     /* create a menu with a submenu */
2740     hmenu = CreateMenu();
2741     hsubmenu = CreateMenu();
2742     assert( hmenu && hsubmenu);
2743     mii.fMask = MIIM_SUBMENU;
2744     mii.hSubMenu = hsubmenu;
2745     ret = InsertMenuItem( hmenu, 0, FALSE, &mii);
2746     ok( ret, "InsertMenuItem failed with error %d\n", GetLastError());
2747     /* test some parameter errors */
2748     SetLastError(0xdeadbeef);
2749     ret = pSetMenuInfo( hmenu, NULL);
2750     gle= GetLastError();
2751     ok( !ret, "SetMenuInfo() should have failed\n");
2752     ok( gle == ERROR_INVALID_PARAMETER ||
2753         broken(gle == 0xdeadbeef), /* Win98, WinME */
2754         "SetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2755     SetLastError(0xdeadbeef);
2756     mi.cbSize = 0;
2757     ret = pSetMenuInfo( hmenu, &mi);
2758     gle= GetLastError();
2759     ok( !ret, "SetMenuInfo() should have failed\n");
2760     ok( gle == ERROR_INVALID_PARAMETER ||
2761         broken(gle == 0xdeadbeef), /* Win98, WinME */
2762         "SetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2763     SetLastError(0xdeadbeef);
2764     mi.cbSize = sizeof( MENUINFO);
2765     ret = pSetMenuInfo( hmenu, &mi);
2766     gle= GetLastError();
2767     ok( ret, "SetMenuInfo() should have succeeded\n");
2768     ok( gle == 0xdeadbeef, "SetMenuInfo() error got %u\n", gle);
2769     SetLastError(0xdeadbeef);
2770     mi.cbSize = 0;
2771     ret = pSetMenuInfo( NULL, &mi);
2772     gle= GetLastError();
2773     ok( !ret, "SetMenuInfo() should have failed\n");
2774     ok( gle == ERROR_INVALID_PARAMETER ||
2775         broken(gle == 0xdeadbeef), /* Win98, WinME */
2776         "SetMenuInfo() error got %u expected %u\n", gle, ERROR_INVALID_PARAMETER);
2777     /* functional tests */
2778     /* menu and submenu should have the CHECKORBMP style bit cleared */
2779     SetLastError(0xdeadbeef);
2780     mi.cbSize = sizeof( MENUINFO);
2781     mi.fMask = MIM_STYLE;
2782     ret = pGetMenuInfo( hmenu, &mi);
2783     gle= GetLastError();
2784     ok( ret, "GetMenuInfo() should have succeeded\n");
2785     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2786     ok( !(mi.dwStyle & MNS_CHECKORBMP), "menustyle was not expected to have the MNS_CHECKORBMP flag\n");
2787     SetLastError(0xdeadbeef);
2788     mi.cbSize = sizeof( MENUINFO);
2789     mi.fMask = MIM_STYLE;
2790     ret = pGetMenuInfo( hsubmenu, &mi);
2791     gle= GetLastError();
2792     ok( ret, "GetMenuInfo() should have succeeded\n");
2793     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2794     ok( !(mi.dwStyle & MNS_CHECKORBMP), "menustyle was not expected to have the MNS_CHECKORBMP flag\n");
2795     /* SetMenuInfo() */
2796     SetLastError(0xdeadbeef);
2797     mi.cbSize = sizeof( MENUINFO);
2798     mi.fMask = MIM_STYLE | MIM_APPLYTOSUBMENUS;
2799     mi.dwStyle = MNS_CHECKORBMP;
2800     ret = pSetMenuInfo( hmenu, &mi);
2801     gle= GetLastError();
2802     ok( ret, "SetMenuInfo() should have succeeded\n");
2803     ok( gle == 0xdeadbeef, "SetMenuInfo() error got %u\n", gle);
2804     /* Now both menus should have the MNS_CHECKORBMP style bit set */
2805     SetLastError(0xdeadbeef);
2806     mi.cbSize = sizeof( MENUINFO);
2807     mi.fMask = MIM_STYLE;
2808     ret = pGetMenuInfo( hmenu, &mi);
2809     gle= GetLastError();
2810     ok( ret, "GetMenuInfo() should have succeeded\n");
2811     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2812     ok( mi.dwStyle & MNS_CHECKORBMP, "menustyle was expected to have the MNS_CHECKORBMP flag\n");
2813     SetLastError(0xdeadbeef);
2814     mi.cbSize = sizeof( MENUINFO);
2815     mi.fMask = MIM_STYLE;
2816     ret = pGetMenuInfo( hsubmenu, &mi);
2817     gle= GetLastError();
2818     ok( ret, "GetMenuInfo() should have succeeded\n");
2819     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2820     ok( mi.dwStyle & MNS_CHECKORBMP, "menustyle was expected to have the MNS_CHECKORBMP flag\n");
2821     /* now repeat that without the APPLYTOSUBMENUS flag and another style bit */
2822     SetLastError(0xdeadbeef);
2823     mi.cbSize = sizeof( MENUINFO);
2824     mi.fMask = MIM_STYLE ;
2825     mi.dwStyle = MNS_NOCHECK;
2826     ret = pSetMenuInfo( hmenu, &mi);
2827     gle= GetLastError();
2828     ok( ret, "SetMenuInfo() should have succeeded\n");
2829     ok( gle == 0xdeadbeef, "SetMenuInfo() error got %u\n", gle);
2830     /* Now only the top menu should have the MNS_NOCHECK style bit set */
2831     SetLastError(0xdeadbeef);
2832     mi.cbSize = sizeof( MENUINFO);
2833     mi.fMask = MIM_STYLE;
2834     ret = pGetMenuInfo( hmenu, &mi);
2835     gle= GetLastError();
2836     ok( ret, "GetMenuInfo() should have succeeded\n");
2837     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2838     ok( mi.dwStyle & MNS_NOCHECK, "menustyle was expected to have the MNS_NOCHECK flag\n");
2839     SetLastError(0xdeadbeef);
2840     mi.cbSize = sizeof( MENUINFO);
2841     mi.fMask = MIM_STYLE;
2842     ret = pGetMenuInfo( hsubmenu, &mi);
2843     gle= GetLastError();
2844     ok( ret, "GetMenuInfo() should have succeeded\n");
2845     ok( gle == 0xdeadbeef, "GetMenuInfo() error got %u\n", gle);
2846     ok( !(mi.dwStyle & MNS_NOCHECK), "menustyle was not expected to have the MNS_NOCHECK flag\n");
2847     /* clean up */
2848     DestroyMenu( hsubmenu);
2849     DestroyMenu( hmenu);
2850     return;
2851 }
2852
2853 /* little func to easy switch either TrackPopupMenu() or TrackPopupMenuEx() */
2854 static DWORD MyTrackPopupMenu( int ex, HMENU hmenu, UINT flags, INT x, INT y, HWND hwnd, LPTPMPARAMS ptpm)
2855 {
2856     return ex
2857         ? TrackPopupMenuEx( hmenu, flags, x, y, hwnd, ptpm)
2858         : TrackPopupMenu( hmenu, flags, x, y, 0, hwnd, NULL);
2859 }
2860
2861 /* some TrackPopupMenu and TrackPopupMenuEx tests */
2862 /* the LastError values differ between NO_ERROR and invalid handle */
2863 /* between all windows versions tested. The first value is that valid on XP  */
2864 /* Vista was the only that made returned different error values */
2865 /* between the TrackPopupMenu and TrackPopupMenuEx functions */
2866 static void test_menu_trackpopupmenu(void)
2867 {
2868     BOOL ret;
2869     HMENU hmenu;
2870     DWORD gle;
2871     int Ex;
2872     HWND hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
2873             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
2874             NULL, NULL, NULL, NULL);
2875     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
2876     if (!hwnd) return;
2877     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_ownerdraw_wnd_proc);
2878     for( Ex = 0; Ex < 2; Ex++)
2879     {
2880         hmenu = CreatePopupMenu();
2881         ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
2882         if (!hmenu)
2883         {
2884             DestroyWindow(hwnd);
2885             return;
2886         }
2887         /* display the menu */
2888         /* start with an invalid menu handle */
2889         gle = 0xdeadbeef;
2890         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2891         ret = MyTrackPopupMenu( Ex, NULL, TPM_RETURNCMD, 100,100, hwnd, NULL);
2892         gle = GetLastError();
2893         ok( !ret, "TrackPopupMenu%s should have failed\n", Ex ? "Ex" : "");
2894         ok( gle == ERROR_INVALID_MENU_HANDLE
2895             || broken (gle == 0xdeadbeef) /* win95 */
2896             || broken (gle == NO_ERROR) /* win98/ME */
2897             ,"TrackPopupMenu%s error got %u expected %u\n",
2898             Ex ? "Ex" : "", gle, ERROR_INVALID_MENU_HANDLE);
2899         ok( !(gflag_initmenupopup || gflag_entermenuloop || gflag_initmenu),
2900                 "got unexpected message(s)%s%s%s\n",
2901                 gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2902                 gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2903                 gflag_initmenu ? "WM_INITMENU": "");
2904         /* another one but not NULL */
2905         gle = 0xdeadbeef;
2906         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2907         ret = MyTrackPopupMenu( Ex, (HMENU)hwnd, TPM_RETURNCMD, 100,100, hwnd, NULL);
2908         gle = GetLastError();
2909         ok( !ret, "TrackPopupMenu%s should have failed\n", Ex ? "Ex" : "");
2910         ok( gle == ERROR_INVALID_MENU_HANDLE
2911             || broken (gle == 0xdeadbeef) /* win95 */
2912             || broken (gle == NO_ERROR) /* win98/ME */
2913             ,"TrackPopupMenu%s error got %u expected %u\n",
2914             Ex ? "Ex" : "", gle, ERROR_INVALID_MENU_HANDLE);
2915         ok( !(gflag_initmenupopup || gflag_entermenuloop || gflag_initmenu),
2916                 "got unexpected message(s)%s%s%s\n",
2917                 gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2918                 gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2919                 gflag_initmenu ? "WM_INITMENU": "");
2920         /* now a somewhat successful call */
2921         gle = 0xdeadbeef;
2922         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2923         ret = MyTrackPopupMenu( Ex, hmenu, TPM_RETURNCMD, 100,100, hwnd, NULL);
2924         gle = GetLastError();
2925         ok( ret == 0, "TrackPopupMenu%s returned %d expected zero\n", Ex ? "Ex" : "", ret);
2926         ok( gle == NO_ERROR
2927             || gle == ERROR_INVALID_MENU_HANDLE /* NT4, win2k */
2928             || broken (gle == 0xdeadbeef) /* win95 */
2929             ,"TrackPopupMenu%s error got %u expected %u or %u\n",
2930             Ex ? "Ex" : "", gle, NO_ERROR, ERROR_INVALID_MENU_HANDLE);
2931         ok( gflag_initmenupopup && gflag_entermenuloop && gflag_initmenu,
2932                 "missed expected message(s)%s%s%s\n",
2933                 !gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2934                 !gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2935                 !gflag_initmenu ? "WM_INITMENU": "");
2936         /* and another */
2937         ret = AppendMenuA( hmenu, MF_STRING, 1, "winetest");
2938         ok( ret, "AppendMenA has failed!\n");
2939         gle = 0xdeadbeef;
2940         gflag_initmenupopup = gflag_entermenuloop = gflag_initmenu = 0;
2941         ret = MyTrackPopupMenu( Ex, hmenu, TPM_RETURNCMD, 100,100, hwnd, NULL);
2942         gle = GetLastError();
2943         ok( ret == 0, "TrackPopupMenu%s returned %d expected zero\n", Ex ? "Ex" : "", ret);
2944         ok( gle == NO_ERROR
2945             || gle == ERROR_INVALID_MENU_HANDLE /* NT4, win2k and Vista in the TrackPopupMenuEx case */
2946             || broken (gle == 0xdeadbeef) /* win95 */
2947             ,"TrackPopupMenu%s error got %u expected %u or %u\n",
2948             Ex ? "Ex" : "", gle, NO_ERROR, ERROR_INVALID_MENU_HANDLE);
2949         ok( gflag_initmenupopup && gflag_entermenuloop && gflag_initmenu,
2950                 "missed expected message(s)%s%s%s\n",
2951                 !gflag_initmenupopup ? " WM_INITMENUPOPUP ": " ",
2952                 !gflag_entermenuloop ? "WM_INITMENULOOP ": "",
2953                 !gflag_initmenu ? "WM_INITMENU": "");
2954         DestroyMenu(hmenu);
2955     }
2956     /* clean up */
2957     DestroyWindow(hwnd);
2958 }
2959
2960 static HMENU g_hmenu;
2961
2962 static LRESULT WINAPI menu_track_again_wnd_proc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
2963 {
2964     switch (msg)
2965     {
2966     case WM_ENTERMENULOOP:
2967     {
2968         BOOL ret;
2969
2970         /* try a recursive call */
2971         SetLastError(0xdeadbeef);
2972         ret = TrackPopupMenu(g_hmenu, 0, 100, 100, 0, hwnd, NULL);
2973         ok(ret == FALSE, "got %d\n", ret);
2974         ok(GetLastError() == ERROR_POPUP_ALREADY_ACTIVE ||
2975            broken(GetLastError() == 0xdeadbeef) /* W9x */, "got %d\n", GetLastError());
2976
2977         /* exit menu modal loop
2978          * ( A SendMessage does not work on NT3.51 here ) */
2979         return PostMessage(hwnd, WM_CANCELMODE, 0, 0);
2980     }
2981     }
2982     return DefWindowProc(hwnd, msg, wparam, lparam);
2983 }
2984
2985 static void test_menu_trackagain(void)
2986 {
2987     HWND hwnd;
2988     BOOL ret;
2989
2990     hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL,
2991             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
2992             NULL, NULL, NULL, NULL);
2993     ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError());
2994     if (!hwnd) return;
2995     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_track_again_wnd_proc);
2996
2997     g_hmenu = CreatePopupMenu();
2998     ok(g_hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError());
2999
3000     ret = TrackPopupMenu( g_hmenu, 0, 100, 100, 0, hwnd, NULL);
3001     todo_wine ok(ret == FALSE, "got %d\n", ret);
3002
3003     DestroyMenu(g_hmenu);
3004     DestroyWindow(hwnd);
3005 }
3006
3007 /* test handling of WM_CANCELMODE messages */
3008 static int g_got_enteridle;
3009 static HWND g_hwndtosend;
3010 static LRESULT WINAPI menu_cancelmode_wnd_proc(HWND hwnd, UINT msg,
3011         WPARAM wparam, LPARAM lparam)
3012 {
3013     switch (msg)
3014     {
3015         case WM_ENTERMENULOOP:
3016             g_got_enteridle = 0;
3017             return SendMessage( g_hwndtosend, WM_CANCELMODE, 0, 0);
3018         case WM_ENTERIDLE:
3019             {
3020                 if( g_got_enteridle++ == 0) {
3021                     /* little hack to get another WM_ENTERIDLE message */
3022                     PostMessage( hwnd, WM_MOUSEMOVE, 0, 0);
3023                     return SendMessage( g_hwndtosend, WM_CANCELMODE, 0, 0);
3024                 }
3025                 pEndMenu();
3026                 return TRUE;
3027             }
3028     }
3029     return DefWindowProc( hwnd, msg, wparam, lparam);
3030 }
3031
3032 static void test_menu_cancelmode(void)
3033 {
3034     DWORD ret;
3035     HWND hwnd, hwndchild;
3036     HMENU menu;
3037     if( !pEndMenu) { /* win95 */
3038         win_skip( "EndMenu is not available\n");
3039         return;
3040     }
3041     hwnd = CreateWindowEx( 0, MAKEINTATOM(atomMenuCheckClass), NULL,
3042             WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200,
3043             NULL, NULL, NULL, NULL);
3044     hwndchild = CreateWindowEx( 0, MAKEINTATOM(atomMenuCheckClass), NULL,
3045             WS_VISIBLE | WS_CHILD, 10, 10, 20, 20,
3046             hwnd, NULL, NULL, NULL);
3047     ok( hwnd != NULL && hwndchild != NULL,
3048             "CreateWindowEx failed with error %d\n", GetLastError());
3049     g_hwndtosend = hwnd;
3050     SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR)menu_cancelmode_wnd_proc);
3051     SetWindowLongPtr( hwndchild, GWLP_WNDPROC, (LONG_PTR)menu_cancelmode_wnd_proc);
3052     menu = CreatePopupMenu();
3053     ok( menu != NULL, "CreatePopupMenu failed with error %d\n", GetLastError());
3054     ret = AppendMenuA( menu, MF_STRING, 1, "winetest");
3055     ok( ret, "Functie failed lasterror is %u\n", GetLastError());
3056     /* seems to be needed only on wine :( */
3057     {MSG msg;   while (PeekMessage(&msg, 0, 0, 0, PM_REMOVE)) DispatchMessage(&msg);}
3058     /* test the effect of sending a WM_CANCELMODE message in the WM_INITMENULOOP
3059      * handler of the menu owner */
3060     /* test results is exctracted from variable g_got_enteridle. Possible values:
3061      * 0 : complete conformance. Sending WM_CANCELMODE cancels a menu initializing tracking
3062      * 1 : Sending WM_CANCELMODE cancels a menu that is in tracking state
3063      * 2 : Sending WM_CANCELMODE does not work
3064      */
3065     /* menu owner is top level window */
3066     g_hwndtosend = hwnd;
3067     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 100,100, 0, hwnd, NULL);
3068     todo_wine {
3069         ok( g_got_enteridle == 0, "received %d WM_ENTERIDLE messages, none expected\n", g_got_enteridle);
3070     }
3071     ok( g_got_enteridle < 2, "received %d WM_ENTERIDLE messages, should be less than 2\n", g_got_enteridle);
3072     /* menu owner is child window */
3073     g_hwndtosend = hwndchild;
3074     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 100,100, 0, hwndchild, NULL);
3075     todo_wine {
3076         ok(g_got_enteridle == 0, "received %d WM_ENTERIDLE messages, none expected\n", g_got_enteridle);
3077     }
3078     ok(g_got_enteridle < 2, "received %d WM_ENTERIDLE messages, should be less than 2\n", g_got_enteridle);
3079     /* now send the WM_CANCELMODE messages to the WRONG window */
3080     /* those should fail ( to have any effect) */
3081     g_hwndtosend = hwnd;
3082     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 100,100, 0, hwndchild, NULL);
3083     ok( g_got_enteridle == 2, "received %d WM_ENTERIDLE messages, should be 2\n", g_got_enteridle);
3084     /* cleanup */
3085     DestroyMenu( menu);
3086     DestroyWindow( hwndchild);
3087     DestroyWindow( hwnd);
3088 }
3089
3090 /* show menu trees have a maximum depth */
3091 static void test_menu_maxdepth(void)
3092 {
3093 #define NR_MENUS 100
3094     HMENU hmenus[ NR_MENUS];
3095     int i;
3096     DWORD ret;
3097
3098     SetLastError(12345678);
3099     for( i = 0; i < NR_MENUS; i++) {
3100         hmenus[i] = CreatePopupMenu();
3101         if( !hmenus[i]) break;
3102     }
3103     ok( i == NR_MENUS, "could not create more than %d menu's\n", i);
3104     for( i = 1; i < NR_MENUS; i++) {
3105         ret = AppendMenuA( hmenus[i], MF_POPUP, (UINT_PTR)hmenus[i-1],"test");
3106         if( !ret) break;
3107     }
3108     trace("Maximum depth is %d\n", i);
3109     ok( GetLastError() == 12345678, "unexpected error %d\n",  GetLastError());
3110     ok( i < NR_MENUS ||
3111            broken( i == NR_MENUS), /* win98, NT */
3112            "no ( or very large) limit on menu depth!\n");
3113
3114     for( i = 0; i < NR_MENUS; i++)
3115         DestroyMenu( hmenus[i]);
3116 }
3117
3118 /* bug #12171 */
3119 static void test_menu_circref(void)
3120 {
3121     HMENU menu1, menu2;
3122     DWORD ret;
3123
3124     menu1 = CreatePopupMenu();
3125     menu2 = CreatePopupMenu();
3126     ok( menu1 && menu2, "error creating menus.\n");
3127     ret = AppendMenuA( menu1, MF_POPUP, (UINT_PTR)menu2, "winetest");
3128     ok( ret, "AppendMenu failed, error is %d\n", GetLastError());
3129     ret = AppendMenuA( menu1, MF_STRING | MF_HILITE, 123, "winetest");
3130     ok( ret, "AppendMenu failed, error is %d\n", GetLastError());
3131     /* app chooses an id that happens to clash with its own hmenu */
3132     ret = AppendMenuA( menu2, MF_STRING, (UINT_PTR)menu2, "winetest");
3133     ok( ret, "AppendMenu failed, error is %d\n", GetLastError());
3134     /* now attempt to change the string of the first item of menu1 */
3135     ret = ModifyMenuA( menu1, (UINT_PTR)menu2, MF_POPUP, (UINT_PTR)menu2, "menu 2");
3136     ok( !ret ||
3137             broken( ret), /* win98, NT */
3138             "ModifyMenu should have failed.\n");
3139     if( !ret) { /* will probably stack fault if the ModifyMenu succeeded */
3140         ret = GetMenuState( menu1, 123, 0);
3141         ok( ret == MF_HILITE, "GetMenuState returned %x\n",ret);
3142     }
3143     DestroyMenu( menu2);
3144     DestroyMenu( menu1);
3145 }
3146
3147 /* test how the menu texts are aligned when the menu items have
3148  * different combinations of text and bitmaps (bug #13350) */
3149 static void test_menualign(void)
3150 {
3151     BYTE bmfill[300];
3152     HMENU menu;
3153     HBITMAP hbm1, hbm2, hbm3;
3154     MENUITEMINFO mii = { sizeof(MENUITEMINFO)};
3155     DWORD ret;
3156     HWND hwnd;
3157     MENUINFO mi = { sizeof( MENUINFO)};
3158
3159     if( !winetest_interactive) {
3160         skip( "interactive alignment tests.\n");
3161         return;
3162     }
3163     hwnd = CreateWindowEx(0,
3164             "STATIC",
3165             "Menu text alignment Test\nPlease make a selection.",
3166             WS_OVERLAPPEDWINDOW,
3167             100, 100,
3168             300, 300,
3169             NULL, NULL, 0, NULL);
3170     ShowWindow( hwnd, SW_SHOW);
3171     /* create bitmaps */
3172     memset( bmfill, 0xcc, sizeof( bmfill));
3173     hbm1 = CreateBitmap( 10,10,1,1,bmfill);
3174     hbm2 = CreateBitmap( 20,20,1,1,bmfill);
3175     hbm3 = CreateBitmap( 50,6,1,1,bmfill);
3176     ok( hbm1 && hbm2 && hbm3, "Creating bitmaps failed\n");
3177     menu = CreatePopupMenu();
3178     ok( menu != NULL, "CreatePopupMenu() failed\n");
3179     if( pGetMenuInfo) {
3180         mi.fMask = MIM_STYLE;
3181         ret = pGetMenuInfo( menu, &mi);
3182         ok( menu != NULL, "GetMenuInfo() failed\n");
3183         ok( 0 == mi.dwStyle, "menuinfo style is %x\n", mi.dwStyle);
3184     }
3185     /* test 1 */
3186     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3187     mii.wID = 1;
3188     mii.hbmpItem = hbm1;
3189     mii.dwTypeData = (LPSTR) " OK: menu texts are correctly left-aligned.";
3190     ret = InsertMenuItem( menu, -1, TRUE, &mii);
3191     ok( ret, "InsertMenuItem() failed\n");
3192     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID ;
3193     mii.wID = 2;
3194     mii.hbmpItem = hbm2;
3195     mii.dwTypeData = (LPSTR) " FAIL: menu texts are NOT left-aligned.";
3196     ret = InsertMenuItem( menu, -1, TRUE, &mii);
3197     ok( ret, "InsertMenuItem() failed\n");
3198     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 110, 200, 0, hwnd, NULL);
3199     ok( ret != 2, "User indicated that menu text alignment test 1 failed %d\n", ret);
3200     /* test 2*/
3201     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3202     mii.wID = 3;
3203     mii.hbmpItem = hbm3;
3204     mii.dwTypeData = NULL;
3205     ret = InsertMenuItem( menu, 0, TRUE, &mii);
3206     ok( ret, "InsertMenuItem() failed\n");
3207     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3208     mii.wID = 1;
3209     mii.hbmpItem = hbm1;
3210     /* make the text a bit longer, to keep it readable */
3211     /* this bug is on winXP and reproduced on wine */
3212     mii.dwTypeData = (LPSTR) " OK: menu texts are to the right of the bitmaps........";
3213     ret = SetMenuItemInfo( menu, 1, TRUE, &mii);
3214     ok( ret, "SetMenuItemInfo() failed\n");
3215     mii.wID = 2;
3216     mii.hbmpItem = hbm2;
3217     mii.dwTypeData = (LPSTR) " FAIL: menu texts are below the first bitmap.  ";
3218     ret = SetMenuItemInfo( menu, 2, TRUE, &mii);
3219     ok( ret, "SetMenuItemInfo() failed\n");
3220     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 110, 200, 0, hwnd, NULL);
3221     ok( ret != 2, "User indicated that menu text alignment test 2 failed %d\n", ret);
3222     /* test 3 */
3223     mii.fMask = MIIM_TYPE | MIIM_ID;
3224     mii.wID = 3;
3225     mii.fType = MFT_BITMAP;
3226     mii.dwTypeData = (LPSTR) hbm3;
3227     ret = SetMenuItemInfo( menu, 0, TRUE, &mii);
3228     ok( ret, "SetMenuItemInfo() failed\n");
3229     mii.fMask = MIIM_BITMAP | MIIM_STRING | MIIM_ID;
3230     mii.wID = 1;
3231     mii.hbmpItem = NULL;
3232     mii.dwTypeData = (LPSTR) " OK: menu texts are below the bitmap.";
3233     ret = SetMenuItemInfo( menu, 1, TRUE, &mii);
3234     ok( ret, "SetMenuItemInfo() failed\n");
3235     mii.wID = 2;
3236     mii.hbmpItem = NULL;
3237     mii.dwTypeData = (LPSTR) " FAIL: menu texts are NOT below the bitmap.";
3238     ret = SetMenuItemInfo( menu, 2, TRUE, &mii);
3239     ok( ret, "SetMenuItemInfo() failed\n");
3240     ret = TrackPopupMenu( menu, TPM_RETURNCMD, 110, 200, 0, hwnd, NULL);
3241     ok( ret != 2, "User indicated that menu text alignment test 3 failed %d\n", ret);
3242     /* cleanup */
3243     DeleteObject( hbm1);
3244     DeleteObject( hbm2);
3245     DeleteObject( hbm3);
3246     DestroyMenu( menu);
3247     DestroyWindow( hwnd);
3248 }
3249
3250 START_TEST(menu)
3251 {
3252     init_function_pointers();
3253
3254     /* Wine defines MENUITEMINFO for W2K and above. NT4 and below can't
3255      * handle that.
3256      */
3257     if (correct_behavior())
3258     {
3259         test_menu_add_string();
3260         test_menu_iteminfo();
3261         test_menu_search_bycommand();
3262         test_CheckMenuRadioItem();
3263         test_menu_resource_layout();
3264         test_InsertMenu();
3265         test_menualign();
3266     }
3267
3268     register_menu_check_class();
3269
3270     test_menu_locked_by_window();
3271     test_subpopup_locked_by_menu();
3272     test_menu_ownerdraw();
3273     test_menu_bmp_and_string();
3274     test_menu_getmenuinfo();
3275     test_menu_setmenuinfo();
3276     test_menu_input();
3277     test_menu_flags();
3278
3279     test_menu_hilitemenuitem();
3280     test_menu_trackpopupmenu();
3281     test_menu_trackagain();
3282     test_menu_cancelmode();
3283     test_menu_maxdepth();
3284     test_menu_circref();
3285 }