Implement text_input_unstable_v3
Needs ReviewPublic

Authored by romangg on Nov 7 2018, 2:06 PM.

Details

Reviewers
None
Group Reviewers
KWin
Frameworks
Maniphest Tasks
T9815: Improve touch/virtual keyboard
Summary

Adds support for the text_input_unstable_v3 protocol to our current
implementations v0 and v2.

Protocol v3 is relatively new and features large changes relative to v2.
All state changes are now double buffered and key input functionality has been
removed, what makes the protocol leaner.

Test Plan

Autotest amended.

Diff Detail

Repository
R127 KWayland
Branch
textInputV3
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 4673
Build 4691: arc lint + arc unit
romangg created this revision.Nov 7 2018, 2:06 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptNov 7 2018, 2:06 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
romangg requested review of this revision.Nov 7 2018, 2:06 PM
zzag added a subscriber: zzag.Feb 8 2019, 2:29 PM
zzag added inline comments.
autotests/client/test_text_input.cpp
545–548

Personal opinion: tests have to be easy to follow, it's okay if we copy some code. I'd prefer to have a specialized test, e.g. testSurroundingTextV3.

1001

If there is only one row, we don't need _data().

src/client/textinput.h
122

**/

src/client/textinput_p.h
83

V3?

src/server/textinput_interface_v3.cpp
137–140

Please use single line comments instead.

428

We don't need this namespace because waylandCauseToKWayland is static.