Revert of X11: Use CopyFromParent colormap when possible

ui/base/x/x11_util.cc

Index: ui/base/x/x11_util.cc
diff --git a/ui/base/x/x11_util.cc b/ui/base/x/x11_util.cc
index 321e594fbf87f629593abbe6073b751e6fbce213..ec6b5363d72403d266c487e9e625a49ff7751e1d 100644
--- a/ui/base/x/x11_util.cc
+++ b/ui/base/x/x11_util.cc
@@ -1420,7 +1420,6 @@
 
 XVisualManager::XVisualManager()
     : display_(gfx::GetXDisplay()),
-      default_visual_id_(0),
       system_visual_id_(0),
       transparent_visual_id_(0),
       using_software_rendering_(false),
@@ -1437,9 +1436,11 @@
   using_compositing_wm_ = XGetSelectionOwner(display_, NET_WM_CM_S0) != None;
 
   // Choose the opaque visual.
-  default_visual_id_ =
-      XVisualIDFromVisual(DefaultVisual(display_, DefaultScreen(display_)));
-  system_visual_id_ = default_visual_id_;
+  XWindowAttributes attribs;
+  Window root = XDefaultRootWindow(display_);
+  Status status = XGetWindowAttributes(display_, root, &attribs);
+  DCHECK_NE(0, status);
+  system_visual_id_ = attribs.visual->visualid;
   DCHECK(system_visual_id_);
   DCHECK(visuals_.find(system_visual_id_) != visuals_.end());
 
@@ -1469,20 +1470,15 @@
                                            bool* using_argb_visual) {
   bool use_argb = want_argb_visual && using_compositing_wm_ &&
                   (using_software_rendering_ || have_gpu_argb_visual_);
-  VisualID visual_id = use_argb && transparent_visual_id_
-                           ? transparent_visual_id_
-                           : system_visual_id_;
-  XVisualData& visual_data = *visuals_[visual_id];
-  const XVisualInfo& visual_info = visual_data.visual_info;
-
-  bool is_default_visual = visual_id == default_visual_id_;
-
+  XVisualData& visual_data =
+      *visuals_[use_argb && transparent_visual_id_ ? transparent_visual_id_
+                                                   : system_visual_id_];
   if (visual)
-    *visual = is_default_visual ? CopyFromParent : visual_info.visual;
+    *visual = visual_data.visual_info.visual;
   if (depth)
-    *depth = is_default_visual ? CopyFromParent : visual_info.depth;
+    *depth = visual_data.visual_info.depth;
   if (colormap)
-    *colormap = is_default_visual ? CopyFromParent : visual_data.GetColormap();
+    *colormap = visual_data.GetColormap();
   if (using_argb_visual)
     *using_argb_visual = use_argb;
 }