Re: [PATCH v3 0/11] MR557: mciqtz32: Correct video window behavior by creating default window.

[Zebediah Figura (she/her)]

On 8/3/22 09:49, Akihiro Sagawa (@sgwaki) wrote:
> Not at all. Actually, video rect is (0,0)-(32,24).

Oops, misremembered it :-)
> > +
> > +        /* test default video destination size */
> > +        err = mciSendCommandW(wDeviceID, MCI_WHERE, MCI_DGV_WHERE_DESTINATION, (DWORD_PTR)&parm);
> > +        ok(!err,"mciCommand where destination returned %s\n", dbg_mcierr(err));
> > +        if (style & (WS_POPUP|WS_CHILD))
> > +            rc = src_rc;
> > +        else
> > +            GetClientRect(video_window, &rc);
> > +        todo_wine_if(style & (WS_POPUP|WS_CHILD))
> > +        ok(EqualRect(&parm.where.rc, &rc), "got %s, expected %s\n", wine_dbgstr_rect(&parm.where.rc), wine_dbgstr_rect(&rc));
> > And this should too. The test here is a bit awkward as such because the
> > destination and source rect are *always* equal, and separately, the
> > client rect will also be equal to the video size if the window is a
> > popup or child.
>
> Minimum width of video window is SM_CXMIN. So, if the video width less than SM_CXMIN approximately, destination rect isn't identical to the source one.

IIRC when I tested, MCI_DGV_WHERE_DESTINATION was always equal to the 
source rect by default, whereas the client rect might be bigger due to 
SM_CXMIN. But I might be misremembering that.