[PATCH 1/3] include/mshtml.idl: Add more interface declarations.

Dmitry Timoshkov dmitry at baikal.ru
Mon Jun 3 06:56:08 CDT 2019


Alexandre Julliard <julliard at winehq.org> wrote:

> Dmitry Timoshkov <dmitry at baikal.ru> writes:
> 
> > Signed-off-by: Dmitry Timoshkov <dmitry at baikal.ru>
> > ---
> >  include/mshtmdid.h |  32 ++++-
> >  include/mshtml.idl | 342 ++++++++++++++++++++++++++++++++++++++++++---
> >  2 files changed, 356 insertions(+), 18 deletions(-)
> 
> This breaks the tests, sorry for not catching this earlier:
> 
> ../../../tools/runtest -q -P wine -T ../../.. -M mshtml.dll -p mshtml_test.exe script && touch script.ok
> script.c:645: Test failed: L"Got exception Object doesn't support this property or method"

Something is wrong with new generated mshtml.tlb: somebody tries to get
an id of the identifier with name "getComputedStyle" however mshtml.tlb
doesn't (and shouldn't) have it, instead "GetComputedStyle" is returned,
and that leads to the failure later on. After looking at dlls/mshtml/dispex.c,
get_builtin_id() I wonder whether case sensitive identifier search is
broken there, perhaps Jacek has a better idea what is going on.

-- 
Dmitry.



More information about the wine-devel mailing list