Differential D18955

[autotests] Check the configure request sent to popups
ClosedPublic
Actions

Authored by davidedmundson on Feb 12 2019, 12:09 PM.

Details

Reviewers
zzag
Group Reviewers
KWin
Commits
R108:dc7ea09e8d85: [autotests] Check the configure request sent to popups
Summary

We currently check the final position kwin knows about.
This patch also checks the position the popup is told that
it's in as well as acking configure requests correctly.

Diff Detail

Repository
R108 KWin
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 8211
Build 8229: arc lint + arc unit
Restricted Application added a project: KWin. · View Herald TranscriptFeb 12 2019, 12:09 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
davidedmundson requested review of this revision.Feb 12 2019, 12:09 PM
zzag added a subscriber: zzag.Feb 12 2019, 1:34 PM
zzag added inline comments.
autotests/integration/transient_placement.cpp
378

Surface::CommitFlags::None

zzag added inline comments.Feb 12 2019, 1:35 PM
autotests/integration/transient_placement.cpp
378

EDIT: CommitFlag

zzag accepted this revision.Feb 12 2019, 5:38 PM
This revision is now accepted and ready to land.Feb 12 2019, 5:38 PM
Closed by commit R108:dc7ea09e8d85: [autotests] Check the configure request sent to popups (authored by davidedmundson). · Explain WhyFeb 26 2019, 2:58 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 51527

View Options

autotests/integration/transient_placement.cpp

Loading...