[wayland] Implement maximize rules
AcceptedPublic

Authored by zzag on Feb 28 2019, 12:22 PM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Summary

There is still one small issue that has to be addressed in the future:
xdg-toplevel doesn't have states like MAXIMIZED_VERT or MAXIMIZED_HORZ,
thus Window Rules KCM should display only single maximize rule(not two)
for wayland clients.

Test Plan

The new tests pass.

Diff Detail

Repository
R108 KWin
Branch
implement-size-position-rules
Lint
Lint ErrorsExcuse: fjkdsalfj;aj
SeverityLocationCodeMessage
Errorautotests/integration/shell_client_rules_test.cpp:406CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:407CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:485CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:486CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:545CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:546CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:686CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:687CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:1953CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:1954CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2006CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2007CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2063CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2064CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2181CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2182CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2264CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2265CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2310CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2311CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2358CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2359CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2452CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2453CppcheckdoubleFree
Errorautotests/integration/shell_client_rules_test.cpp:2532CppcheckdoubleFree
Unit
No Unit Test Coverage
Build Status
Buildable 8995
Build 9013: arc lint + arc unit
zzag created this revision.Feb 28 2019, 12:22 PM
Restricted Application added a project: KWin. · View Herald TranscriptFeb 28 2019, 12:22 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Feb 28 2019, 12:22 PM
zzag updated this revision to Diff 52843.Feb 28 2019, 4:28 PM

use enums in the tests

davidedmundson accepted this revision.Mar 8 2019, 12:18 AM
This revision is now accepted and ready to land.Mar 8 2019, 12:18 AM