Differential D18440

[client] Wrap wl_shell_surface_set_{class,title}
ClosedPublic
Actions

Authored by zzag on Jan 21 2019, 9:44 PM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Commits
R127:0a2c807ad21e: [client] Wrap wl_shell_surface_set_{class,title}
Summary

We have to wrap these two requests just for convenience.

Diff Detail

Repository
R127 KWayland
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Jan 21 2019, 9:44 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 21 2019, 9:44 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
zzag requested review of this revision.Jan 21 2019, 9:44 PM
davidedmundson accepted this revision.Jan 21 2019, 9:54 PM
davidedmundson added a subscriber: davidedmundson.

ShellSurface is super-duper deprecated. I wouldn't invest too much time on it.

This revision is now accepted and ready to land.Jan 21 2019, 9:54 PM
zzag added a comment.Jan 21 2019, 9:54 PM
In D18440#397756, @davidedmundson wrote:

ShellSurface is super-duper deprecated. I wouldn't invest too much time on it.

Yes, I know. I just need set_class in some KWin's tests. :-)

Closed by commit R127:0a2c807ad21e: [client] Wrap wl_shell_surface_set_{class,title} (authored by zzag). · Explain WhyJan 22 2019, 8:54 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 50045

View Options

autotests/client/test_wayland_shell.cpp

Loading...
View Options

src/client/shell.h

Loading...
View Options

src/client/shell.cpp

Loading...