| Index: ui/display/manager/chromeos/x11/native_display_delegate_x11.cc
|
| diff --git a/ui/display/manager/chromeos/x11/native_display_delegate_x11.cc b/ui/display/manager/chromeos/x11/native_display_delegate_x11.cc
|
| index 56f71806ad60c9ed5195401fdfeacaba5b3712a5..d94e618ae00a2ec1ab549c1261c3b73fff807dde 100644
|
| --- a/ui/display/manager/chromeos/x11/native_display_delegate_x11.cc
|
| +++ b/ui/display/manager/chromeos/x11/native_display_delegate_x11.cc
|
| @@ -16,6 +16,7 @@
|
| #include "base/logging.h"
|
| #include "base/macros.h"
|
| #include "base/memory/ptr_util.h"
|
| +#include "ui/base/x/x11_util.h"
|
| #include "ui/display/manager/chromeos/x11/display_mode_x11.h"
|
| #include "ui/display/manager/chromeos/x11/display_snapshot_x11.h"
|
| #include "ui/display/manager/chromeos/x11/display_util_x11.h"
|
| @@ -402,8 +403,7 @@ bool NativeDisplayDelegateX11::GetHDCPState(const DisplaySnapshot& output,
|
| Atom actual_type = None;
|
| int success = 0;
|
| RROutput output_id = static_cast<const DisplaySnapshotX11&>(output).output();
|
| - // TODO(kcwu): Use X11AtomCache to save round trip time of XInternAtom.
|
| - Atom prop = XInternAtom(display_, kContentProtectionAtomName, False);
|
| + Atom prop = ui::GetAtom(kContentProtectionAtomName);
|
|
|
| // TODO(kcwu): Move this to x11_util (similar method calls in this file and
|
| // output_util.cc)
|
| @@ -430,13 +430,11 @@ bool NativeDisplayDelegateX11::GetHDCPState(const DisplaySnapshot& output,
|
| if (success == Success && actual_type == XA_ATOM && actual_format == 32 &&
|
| nitems == 1) {
|
| Atom value = reinterpret_cast<Atom*>(values)[0];
|
| - if (value == XInternAtom(display_, kProtectionUndesiredAtomName, False)) {
|
| + if (value == ui::GetAtom(kProtectionUndesiredAtomName)) {
|
| *state = HDCP_STATE_UNDESIRED;
|
| - } else if (value ==
|
| - XInternAtom(display_, kProtectionDesiredAtomName, False)) {
|
| + } else if (value == ui::GetAtom(kProtectionDesiredAtomName)) {
|
| *state = HDCP_STATE_DESIRED;
|
| - } else if (value ==
|
| - XInternAtom(display_, kProtectionEnabledAtomName, False)) {
|
| + } else if (value == ui::GetAtom(kProtectionEnabledAtomName)) {
|
| *state = HDCP_STATE_ENABLED;
|
| } else {
|
| LOG(ERROR) << "Unknown " << kContentProtectionAtomName
|
| @@ -461,14 +459,14 @@ void NativeDisplayDelegateX11::SetHDCPState(
|
|
|
| bool NativeDisplayDelegateX11::SetHDCPState(const DisplaySnapshot& output,
|
| HDCPState state) {
|
| - Atom name = XInternAtom(display_, kContentProtectionAtomName, False);
|
| + Atom name = ui::GetAtom(kContentProtectionAtomName);
|
| Atom value = None;
|
| switch (state) {
|
| case HDCP_STATE_UNDESIRED:
|
| - value = XInternAtom(display_, kProtectionUndesiredAtomName, False);
|
| + value = ui::GetAtom(kProtectionUndesiredAtomName);
|
| break;
|
| case HDCP_STATE_DESIRED:
|
| - value = XInternAtom(display_, kProtectionDesiredAtomName, False);
|
| + value = ui::GetAtom(kProtectionDesiredAtomName);
|
| break;
|
| default:
|
| NOTREACHED() << "Invalid HDCP state: " << state;
|
| @@ -545,8 +543,8 @@ void NativeDisplayDelegateX11::UpdateCrtcsForNewFramebuffer(
|
| }
|
|
|
| bool NativeDisplayDelegateX11::IsOutputAspectPreservingScaling(RROutput id) {
|
| - Atom scaling_prop = XInternAtom(display_, "scaling mode", False);
|
| - Atom full_aspect_atom = XInternAtom(display_, "Full aspect", False);
|
| + Atom scaling_prop = ui::GetAtom("scaling mode");
|
| + Atom full_aspect_atom = ui::GetAtom("Full aspect");
|
| if (scaling_prop == None || full_aspect_atom == None)
|
| return false;
|
|
|
|
|