Same as MR !2394
Signed-off-by: Zhao Yi zhaoyi(a)uniontech.com
--
v6: comctl32/listview: Reset bNoItemMetrics to TRUE in LISTVIEW_DeleteAllItems() to arrange listview items.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3392
On Wed Jul 26 19:17:48 2023 +0000, Nikolay Sivov wrote:
> I had to learn what's going on in there once again. I think what we need
> to do is basically stop using 'presenter->cs' in the streaming thread
> entirely, MFVP_MESSAGE_ENDSTREAMING handling will remain unchanged.
> This means two things:
> - queue itself should be protected with its own lock;
> - video_presenter_process_input() should not be called in the streaming
> thread all. Instead it could be called right in
> video_presenter_allocator_cb_NotifyRelease(), with appropriate locking.
It's slightly more complicated, I can see now why EVRM_PROCESS_INPUT was added - it was probably to avoid interacting with tracking thread. The point still stands, I think we'll need to remove remaining presenter lock from streaming thread, one way or another.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3319#note_40394
This get us pass the "Update your browser" blocker in adobe's sign-in page. The page itself doesn't make use of `window.MutationObserver`.
However the sign-in page is still broken.
--
v16: mshtml: add stubs for MutationObserver methods
mshtml: implement window.MutationObserver with MutationObserver stub
https://gitlab.winehq.org/wine/wine/-/merge_requests/3391
On Wed Jul 26 17:26:57 2023 +0000, Santino Mazza wrote:
> Oh I just saw that we have `PRESENTER_MIXER_HAS_INPUT` flag and is set
> when it receives the `MFVP_MESSAGE_PROCESSINPUTNOTIFY` message, but when
> the streaming thread receives the `EVRM_PROCESS_INPUT` message it
> doesn't set that flag, shouldn't we set that flag and check for it
> before waiting for the thread?
I had to learn what's going on in there once again. I think what we need to do is basically stop using 'presenter->cs' in the streaming thread entirely, MFVP_MESSAGE_ENDSTREAMING handling will remain unchanged.
This means two things:
- queue itself should be protected with its own lock;
- video_presenter_process_input() should not be called in the streaming thread all. Instead it could be called right in video_presenter_allocator_cb_NotifyRelease(), with appropriate locking.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3319#note_40382
This get us pass the "Update your browser" blocker in adobe's sign-in page. The page itself doesn't make use of `window.MutationObserver`.
However the sign-in page is still broken.
--
v15: mshtml: add stubs for MutationObserver methods
mshtml: implement window.MutationObserver with MutationObserver stub
https://gitlab.winehq.org/wine/wine/-/merge_requests/3391