Remove usages of XInternAtom

ui/gfx/x/x11_atom_cache.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_GFX_X_X11_ATOM_CACHE_H_
 #define UI_GFX_X_X11_ATOM_CACHE_H_
 
 #include <map>
 #include <string>
 
 #include "base/macros.h"
 #include "ui/gfx/gfx_export.h"
 #include "ui/gfx/x/x11_types.h"
 
+namespace base {
+template <typename T>
+struct DefaultSingletonTraits;
+}
+
 namespace ui {

[sadrul, 2017/06/01 20:45:31]

OK, I think my bad, but this should be gfx :(

Something we should fix. Not in this CL though.

[Tom Anderson, 2017/06/01 21:20:42]

Acknowledged.

 
 // Pre-caches all Atoms on first use to minimize roundtrips to the X11
 // server. By default, GetAtom() will CHECK() that atoms accessed through
 // GetAtom() were passed to the constructor, but this behaviour can be changed
 // with allow_uncached_atoms().
 class GFX_EXPORT X11AtomCache {
  public:
-  // Preinterns the NULL terminated list of string |to_cache| on |xdisplay|.
-  X11AtomCache(XDisplay* xdisplay, const char* const* to_cache);
-  ~X11AtomCache();
+  static X11AtomCache* GetInstance();
 
   // Returns the pre-interned Atom without having to go to the x server.
   XAtom GetAtom(const char*) const;
 
-  // When an Atom isn't in the list of items we've cached, we should look it
-  // up, cache it locally, and then return the result.
-  void allow_uncached_atoms() { uncached_atoms_allowed_ = true; }
+ private:
+  friend struct base::DefaultSingletonTraits<X11AtomCache>;
 
- private:
+  X11AtomCache();
+  ~X11AtomCache();
+
   XDisplay* xdisplay_;
 
-  bool uncached_atoms_allowed_;
-
   mutable std::map<std::string, XAtom> cached_atoms_;
 
   DISALLOW_COPY_AND_ASSIGN(X11AtomCache);
 };
 
 }  // namespace ui
 
 #endif  // UI_GFX_X_X11_ATOM_CACHE_H_