ozone/wayland: use wl_output_set_user_data instead of own containers [chromium/src : master]
Maksim Sisov (Gerrit)
Attention is currently required from: Robert Kroeger.
Maksim Sisov would like Robert Kroeger to review this change.
ozone/wayland: use wl_output_set_user_data instead of own containers
Having own container and searching for an output adds additional
overhead, which can be avoided with using Wayland's *_set_user_data.
This way, existing Wayland utility is used to associated clients'
objects with Wayland create objects.
Bug: 578890
Change-Id: I8bcd11f9fb13c4802a966e553e5282729aceb9e2
---
M ui/ozone/platform/wayland/host/wayland_output_manager.cc
M ui/ozone/platform/wayland/host/wayland_output_manager.h
M ui/ozone/platform/wayland/host/wayland_window.cc
3 files changed, 11 insertions(+), 17 deletions(-)
To view, visit change 2772993. To unsubscribe, or for help writing mail filters, visit settings.
Maksim Sisov (Gerrit)
Attention is currently required from: Robert Kroeger.
Patch set 1:Code-Review +1Commit-Queue +1
1 comment:
Patchset:
ptal!
To view, visit change 2772993. To unsubscribe, or for help writing mail filters, visit settings.
Maksim Sisov (Gerrit)
Attention is currently required from: Robert Kroeger.
Maksim Sisov uploaded patch set #2 to this change.
ozone/wayland: use wl_output_set_user_data instead of own containers
Having own container and searching for an output adds additional
overhead, which can be avoided with using Wayland's *_set_user_data.
This way, existing Wayland utility methods are used to associate
clients' objects with Wayland create objects.
Bug: 578890
Change-Id: I8bcd11f9fb13c4802a966e553e5282729aceb9e2
---
M ui/ozone/platform/wayland/host/wayland_output_manager.cc
M ui/ozone/platform/wayland/host/wayland_output_manager.h
M ui/ozone/platform/wayland/host/wayland_window.cc
3 files changed, 11 insertions(+), 17 deletions(-)
To view, visit change 2772993. To unsubscribe, or for help writing mail filters, visit settings.
Maksim Sisov (Gerrit)
Attention is currently required from: Robert Kroeger.
Maksim Sisov uploaded patch set #3 to this change.
ozone/wayland: use wl_output_set_user_data instead of own containers
Having own container and searching for an output adds additional
overhead, which can be avoided with using Wayland's *_set_user_data.
This way, existing Wayland utility methods are used to associate
clients' objects with Wayland create objects.
Bug: 1190141
Change-Id: I8bcd11f9fb13c4802a966e553e5282729aceb9e2
---
M ui/ozone/platform/wayland/host/wayland_output_manager.cc
M ui/ozone/platform/wayland/host/wayland_output_manager.h
M ui/ozone/platform/wayland/host/wayland_window.cc
3 files changed, 11 insertions(+), 17 deletions(-)
To view, visit change 2772993. To unsubscribe, or for help writing mail filters, visit settings.
Robert Kroeger (Gerrit)
Attention is currently required from: Maksim Sisov.
Patch set 4:Code-Review +1
Maksim Sisov (Gerrit)
Patch set 4:Commit-Queue +2
Chromium LUCI CQ (Gerrit)
Chromium LUCI CQ submitted this change.
ozone/wayland: use wl_output_set_user_data instead of own containers
Having own container and searching for an output adds additional
overhead, which can be avoided with using Wayland's *_set_user_data.
This way, existing Wayland utility methods are used to associate
clients' objects with Wayland create objects.
Bug: 1190141
Change-Id: I8bcd11f9fb13c4802a966e553e5282729aceb9e2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2772993
Reviewed-by: Maksim Sisov <msi...@igalia.com>
Reviewed-by: Robert Kroeger <rjkr...@chromium.org>
Commit-Queue: Maksim Sisov <msi...@igalia.com>
Cr-Commit-Position: refs/heads/master@{#865063}
---
M ui/ozone/platform/wayland/host/wayland_output.cc
M ui/ozone/platform/wayland/host/wayland_output_manager.cc
M ui/ozone/platform/wayland/host/wayland_output_manager.h
M ui/ozone/platform/wayland/host/wayland_window.cc
4 files changed, 17 insertions(+), 19 deletions(-)