Re: [PATCH 7/9] ieframe: Forward more DocHost::Exec calls to embedders

2012-11-12 Thread Alexandre Julliard
Francois Gouget writes: > We can potentially change the criteria that TestBot uses to decide > whether to skip. But what should these be here? Also, do they really > belong in WineTestBot, which would likely require duplicating that code > to WineTest too, or should the skipping really be done

Re: [PATCH 7/9] ieframe: Forward more DocHost::Exec calls to embedders

2012-11-12 Thread Francois Gouget
On Mon, 12 Nov 2012, Jacek Caban wrote: > On 11/12/12 13:21, Marvin wrote: [...] > > === W2KPROSP4 (32 bit webbrowser) === > > webbrowser.c:2596: Test failed: expected GetOverridesKeyPath > > webbrowser.c:2601: Test failed: expected Invoke_SETSECURELOCKICON > > webbrowser.c:2602: Test failed: expe

Re: [PATCH 7/9] ieframe: Forward more DocHost::Exec calls to embedders

2012-11-12 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=22897 Your paranoid android

Re: [PATCH 7/9] ieframe: Forward more DocHost::Exec calls to embedders

2012-11-12 Thread Jacek Caban
On 11/12/12 13:21, Marvin wrote: > Hi, > > While running your changed tests on Windows, I think I found new failures. > Being a bot and all I'm not very good at pattern recognition, so I might be > wrong, but could you please double-check? > Full results can be found at > http://testbot.winehq.org/