wxToolBar theme issue

[Harukalover]

wxToolBar in trunk seems to partially use the classic Windows theme even with the resource file properly included and a manifest being created. I say partially since the dropdown control in the sample displays part of the Vista theme.

The pictures attached to this ticket are from the toolbar sample. One is from the 2.9.0 branch which displays correctly, while the other is from trunk where it is broken.

[Harukalover]

2.9.0 Branch

[Harukalover]

Trunk (r62777)

[vadz]

I can see this too (under Windows 7).

No relevant changes seem to have been done in src/msw/toolbar.cpp itself since 2.9.0 so it was probably broken by either r61859 or r61885.

Could you please test if you can find which change broke it exactly? If it's neither of the above, it would be necessary to bisect the history to find it (the last common revision for 2.9.0 and trunk is r60038 so it could be used as the known good point).

TIA!

[Harukalover]

Did some checking and found it's r61086 that caused the regression.

[MoonKid]

*watch*

[VZ]

(In [62805]) Generate erase background events for native controls too under MSW.

Remove the test which prevented the generation of erase background events from
wxMSW::wxWindow::HandleEraseBkgnd(). Although it is true that native controls
mostly erase background on their own, there are exceptions: we must erase the
background of wxToolBar ourselves, for example. More importantly, there is no
reason to prevent the user code from defining wxEVT_ERASE_BACKGROUND handlers
for the native controls as this works just fine under MSW (although it doesn't
under GTK nor probably other ports...). So also add a test erase background
handler to the toolbar sample to verify that this does work.

Closes #11514.