/srv/irclogs.ubuntu.com/2017/03/20/#ubuntu-mir.txt

=== Guest63440 is now known as RAOF
anpokalan_g: one final question on the dnd MP - so a client identifies a drop event by makeing a != nulptr test on the dnd handle of a pointer button up event?09:01
anpoknm - there is a test for just that09:03
alan_ganpok: yes09:03
alan_gkdub: is mcl::ErrorChain obsolete - I don't see it being used.11:09
kdubeh, guess we have check that mir_presentation_chain_get_error_msg is working11:11
kdubcould have been overlooked when MirRenderSurface error handling was introduced I suppose11:12
alan_gThanks. (I can delete it without screwing up WIP.)11:16
dandraderanpok, around?11:49
anpokyes11:49
dandraderanpok, http://pastebin.ubuntu.com/24214922/11:50
dandraderanpok, can this input validator alter keyboard modifiers?11:50
dandraderanpok, eg: add a Ctrl keyboard modifier to a event that originally had none?11:50
anpokwell it did not..11:55
anpokwe changed that bit in lp:mir11:56
anpokiirc it would only copy the input event modifiers..11:56
greybackalan_g: hey, qtmir ftbfs against miral 1.3.1: miral/set_command_line_hander.h: No such file or directory11:59
alan_ggreyback: Sorry, a spelling correction: miral/set_command_line_handler.h12:01
greybackalan_g: yep I know. Just thought you should know, in case you'd consider it an api break12:01
alan_ggreyback: thanks. But as the ABI is unbroken (which I care about more), I'm not too worried.12:04
greybackok12:04
dandraderanpok, I don't get it. can the input validator in the currently released mir add keyboard modifiers to the kdb input event passed to Surface::consume()?12:04
anpokdandrader: in 0.26 it only affected touch events12:06
anpokand usually I hope it never did anything at all12:06
dandraderanpok, I'm puzzled. I'm investigating a bug and currently it seems like qtmir is sending a keyboard event in surface->consume() without modifiers but it's arriving with a modifer on the client side (qtubuntu)12:08
anpokhm12:08
dandraderanpok, would that be possible?12:08
anpokwith unity8 every client gets a keymap ..12:08
anpokthat enables client side keystate tracking12:09
dandraderanpok, hmm, so you mean mir client lib could be adding that?12:10
anpokso if the client has seen a ctrl key down..12:11
anpokand never seen a ctrl key up..  xkb common will assume that it is stil down12:11
greybackalan_g: you missed fixing the typo in set_window_managment_policy.h12:16
alan_ggreyback: are you sure?12:18
alan_gRats! you're right12:18
anpokdandrader: that key state is reset on focus change .. and if you need to filter out events... we could try to add a machinery to trigger the input device state events which would sync clients and the server again13:03
dandraderanpok, I don't need to filter out events. I need to fix a bug where a client is creating and focusing a new child surface in response to a ctrl+O. once that child surface closes, the top-level surface is stuck with a ctrl modifier until ctrl is pressed+released again13:05
anpokthis is odd13:05
anpokis there no focus change involved?13:06
anpokoh hold on the new child is focused and gets focused again?13:06
dandraderanpok, there are many bugs involved in that use case. in miral (just fixed), in qtmir and unity8.13:06
dandraderanpok, and now I'm thinking there must be some issue in mir as well. miral just released its fix. will try with that13:07
dandraderanpok, the new child surface does get focused13:08
dandraderanpok, once that child window is closed and focuse is assigned back to the top-level surface. it gets a sticky ctrl modifier13:08
dandraderanpok, even though, afaict, qtmir is sending key events without any modifiers.13:09
dandraderanpok, will proceed my investigations further and if I can't pinpoint another bug elsewhere will add mir to the bug report13:09
anpokok..13:10
anpokdandrader: MIR_CLIENT_INPUT_RECEIVER_REPORT=log would be helpful13:11
=== JanC is now known as Guest30327
=== JanC_ is now known as JanC
=== dandrader is now known as dandrader|afk
=== hikiko is now known as hikiko|bbl
=== dandrader|afk is now known as dandrader
=== JanC_ is now known as JanC
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== hikiko|bbl is now known as hikiko
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== mapreri_ is now known as mapreri
=== Saviq_ is now known as Saviq

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!