Fix SetShape (SetAlphaShape) to allow Null regions (+ tests).

ui/views/widget/native_widget_aura.cc

Index: ui/views/widget/native_widget_aura.cc
diff --git a/ui/views/widget/native_widget_aura.cc b/ui/views/widget/native_widget_aura.cc
index e57a4ff0bc0ecb56c12fba415058f0d60a13b85f..089872fe305f79b0356311ff0d87a2525227f8b6 100644
--- a/ui/views/widget/native_widget_aura.cc
+++ b/ui/views/widget/native_widget_aura.cc
@@ -438,8 +438,8 @@ void NativeWidgetAura::StackBelow(gfx::NativeView native_view) {
 }
 
 void NativeWidgetAura::SetShape(gfx::NativeRegion region) {
-  // No need for this. Just delete and ignore.
-  delete region;
+  if (window_ && window_->layer())

[sky, 2014/08/11 23:06:18]

no need for the window_->layer() part of the conditional. If window_ is valid,
so is it's layer.

[garykac, 2014/08/11 23:20:56]

Done.

+    window_->layer()->SetAlphaShape(make_scoped_ptr(region));

[sky, 2014/08/11 23:06:18]

else case to delete region?

[garykac, 2014/08/11 23:20:56]

Done.

 }
 
 void NativeWidgetAura::Close() {