From: Jacek Caban <jacek@codeweavers.com>
Subject: Re: [PATCH] ieframe: IOleObject::SetExtent() should resize the Shell Embedding Window.
Message-Id: <11b81a54-92e7-e5ac-18c1-08ac0acaf3ce@codeweavers.com>
Date: Tue, 23 Jun 2020 17:09:02 +0200
In-Reply-To: <20200622174048.f5423b3fb14f2e34e55b43bd@baikal.ru>
References: <20200619111907.66db11b2ab2298b224f52f8c@baikal.ru> <56ea653b-c5f4-7a9b-a800-197b099fd19a@codeweavers.com> <20200622174048.f5423b3fb14f2e34e55b43bd@baikal.ru>

On 22.06.2020 11:40, Dmitry Timoshkov wrote:
> Hi Jacek,
>
> Jacek Caban <jacek@codeweavers.com> wrote:
>
>> Could you please add a test? We already have some SetExtent() tests and
>> you can use GetWindow() to get the window handle there.
> Thanks for the review. I've added some tests and they show that ::SetExtent()
> should not immediately resize the Shell embedding window, it looks like that
> the window gets resized during UI activation. However, the application that
> I have here calls ::SetExtent() (and nothing else) from its main window WM_SIZE
> handler, and under Windows the contents size follows main window size, but
> under Wine contents size keeps original (wrong) values. Do you have an idea
> what might be wrong, and how this behaviour could be tested?


I don't know, it will need more experimenting. Maybe we'd need to pump 
messages in tests for this call to take place? Or maybe something else 
specific to embedder (eg. its IOleInPlaceSite implementation) changes 
the behaviour.


Thanks,

Jacek