Index: ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc |
diff --git a/ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc b/ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc |
index 112a741c6140a5888daac8ad044c02efe21e06cc..355093016bf2cd14744d9aef6181f2a35bb2e521 100644 |
--- a/ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc |
+++ b/ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc |
@@ -26,7 +26,6 @@ |
#include "ui/events/platform/x11/x11_event_source.h" |
#include "ui/gfx/path.h" |
#include "ui/gfx/path_x11.h" |
-#include "ui/gfx/x/x11_atom_cache.h" |
#include "ui/views/test/views_interactive_ui_test_base.h" |
#include "ui/views/test/x11_property_change_waiter.h" |
#include "ui/views/widget/desktop_aura/desktop_native_widget_aura.h" |
@@ -41,10 +40,7 @@ namespace { |
class MinimizeWaiter : public X11PropertyChangeWaiter { |
public: |
explicit MinimizeWaiter(XID window) |
- : X11PropertyChangeWaiter(window, "_NET_WM_STATE") { |
- const char* const kAtomsToCache[] = {"_NET_WM_STATE_HIDDEN", nullptr}; |
- atom_cache_.reset(new ui::X11AtomCache(gfx::GetXDisplay(), kAtomsToCache)); |
- } |
+ : X11PropertyChangeWaiter(window, "_NET_WM_STATE") {} |
~MinimizeWaiter() override {} |
@@ -54,14 +50,12 @@ class MinimizeWaiter : public X11PropertyChangeWaiter { |
std::vector<Atom> wm_states; |
if (ui::GetAtomArrayProperty(xwindow(), "_NET_WM_STATE", &wm_states)) { |
auto it = std::find(wm_states.cbegin(), wm_states.cend(), |
- atom_cache_->GetAtom("_NET_WM_STATE_HIDDEN")); |
+ ui::GetAtom("_NET_WM_STATE_HIDDEN")); |
return it == wm_states.cend(); |
} |
return true; |
} |
- std::unique_ptr<ui::X11AtomCache> atom_cache_; |
- |
DISALLOW_COPY_AND_ASSIGN(MinimizeWaiter); |
}; |
@@ -400,12 +394,8 @@ TEST_F(X11TopmostWindowFinderTest, Menu) { |
CWOverrideRedirect, |
&swa); |
{ |
- const char* const kAtomsToCache[] = {"_NET_WM_WINDOW_TYPE_MENU", nullptr}; |
- ui::X11AtomCache atom_cache(gfx::GetXDisplay(), kAtomsToCache); |
- ui::SetAtomProperty(menu_xid, |
- "_NET_WM_WINDOW_TYPE", |
- "ATOM", |
- atom_cache.GetAtom("_NET_WM_WINDOW_TYPE_MENU")); |
+ ui::SetAtomProperty(menu_xid, "_NET_WM_WINDOW_TYPE", "ATOM", |
+ ui::GetAtom("_NET_WM_WINDOW_TYPE_MENU")); |
} |
ui::SetUseOSWindowFrame(menu_xid, false); |
ShowAndSetXWindowBounds(menu_xid, gfx::Rect(140, 110, 100, 100)); |