Re: [PATCH v2 0/1] MR6075: Add chevron visibility test to check the case of restoring band width to ideal value

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/rebar.c:
> #undef X
>  }
>
> +static void test_chevron(void)
> +{
> - HWND hRebar;
> - REBARBANDINFOA rbi;
> -
> - hRebar = create_rebar_control(0);
> - rbi.cbSize = REBARBANDINFOA_V6_SIZE;
> - rbi.fMask = RBBIM_CHILD | RBBIM_CHILDSIZE | RBBIM_IDEALSIZE | RBBIM_SIZE | RBBIM_STYLE;
> - rbi.cxIdeal = 128;
> - rbi.cx = rbi.cxIdeal >> 1;

Why do you use >> 1 and >> 2 to adjust width. You should check that the chevron appears when cx is less than cxIdeal. And the chevron disappears when cx equals to or is greater than cxIdeal. So cx should be cxIdeal - 1.