linux/x11: Fix setting custom window shapes in High DPI.

ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc

 // 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.
 
 #include "ui/views/widget/desktop_aura/desktop_window_tree_host_x11.h"
 
 #include <X11/extensions/shape.h>
 #include <X11/extensions/XInput2.h>
 #include <X11/Xatom.h>
 #include <X11/Xregion.h>
@@ skipping 474 matching lines @@
   return ToDIPRect(GetWorkAreaBoundsInPixels());
 }
 
 void DesktopWindowTreeHostX11::SetShape(gfx::NativeRegion native_region) {
   if (window_shape_)
     XDestroyRegion(window_shape_);
   custom_window_shape_ = false;
   window_shape_ = NULL;
 
   if (native_region) {
+    gfx::Transform transform = GetRootTransform();
+    if (!transform.IsIdentity()) {
+      SkPath path_in_dip;
+      if (native_region->getBoundaryPath(&path_in_dip)) {
+        SkPath path_in_pixels;
+        path_in_dip.transform(transform.matrix(), &path_in_pixels);
+        window_shape_ = gfx::CreateRegionFromSkPath(path_in_pixels);
+      }

[pkotwicz, 2015/03/08 23:44:57]

I getBoundaryPath() returns false, you can just do "window_shape_ =
XCreateRegion()". Alternatively, you can check !native_region->isEmpty() on line
496

I think both options are more readable

[sadrul, 2015/03/09 17:20:10]

Done.

+    }

[pkotwicz, 2015/03/08 23:33:26]

Move "window_shape_ = gfx::CreateRegionFromSkRegion(*native_region);" to an else
statement

[sadrul, 2015/03/09 17:20:10]

Done.

+
     custom_window_shape_ = true;
-    window_shape_ = gfx::CreateRegionFromSkRegion(*native_region);
+    if (!window_shape_)
+      window_shape_ = gfx::CreateRegionFromSkRegion(*native_region);
     delete native_region;
   }
   ResetWindowRegion();
 }
 
 void DesktopWindowTreeHostX11::Activate() {
   if (!window_mapped_)
     return;
 
   X11DesktopHandler::get()->ActivateWindow(xwindow_);
@@ skipping 1479 matching lines @@
   if (linux_ui) {
     ui::NativeTheme* native_theme = linux_ui->GetNativeTheme(window);
     if (native_theme)
       return native_theme;
   }
 
   return ui::NativeTheme::instance();
 }
 
 }  // namespace views