Closed GoogleCodeExporter closed 9 years ago
I've added a patch containing a quick fix to this issue.
Original comment by mihai.o....@gmail.com
on 29 Mar 2012 at 4:15
Attachments:
The OnSystemSetFocus() method proposed in issue 563 can be used to solve this
issue as well.
Original comment by magreenb...@gmail.com
on 2 Apr 2012 at 3:15
OnSystemSetFocus function, which is at a higher level, is not visible for
webviewhost, where the MouseDown and, consequently, WM_SETFOCUS events occur.
Solution for this issue was to send focus to the parent window, which will fire
the event to the application, and after that will set the focus back to the
webviewhost.
Original comment by iohan...@gmail.com
on 2 Apr 2012 at 4:35
@comment #3: I'd rather avoid bouncing the focus between parent and child
windows after receiving a mouse click in the child window. It's better to just
set the focus to the child window directly. See the fix in issue #563 (revision
568).
Original comment by magreenb...@gmail.com
on 2 Apr 2012 at 4:44
Thank you. It should work now.
Original comment by iohan...@gmail.com
on 2 Apr 2012 at 4:51
@comment #4. I think we should check whether webviewhost already has focus
before calling SetFocus(true), otherwise OnSetFocus will be called for each
mouse click.
Original comment by iohan...@gmail.com
on 2 Apr 2012 at 5:28
@comment#6: Agreed, I'll fix it shortly.
Original comment by magreenb...@gmail.com
on 2 Apr 2012 at 5:41
Revision 569 avoids calling OnSetFocus multiple times.
Original comment by magreenb...@gmail.com
on 2 Apr 2012 at 7:40
Original issue reported on code.google.com by
mihai.o....@gmail.com
on 29 Mar 2012 at 2:40