views: Some more layer change, and add/update some tests for layers.

views/controls/native_control_gtk.cc

Index: views/controls/native_control_gtk.cc
diff --git a/views/controls/native_control_gtk.cc b/views/controls/native_control_gtk.cc
index 965d34738c194215329ca08a25fb57a8b23bc847..1e3c0b6d0badd66f697bcf593c7d9580054fda45 100644
--- a/views/controls/native_control_gtk.cc
+++ b/views/controls/native_control_gtk.cc
@@ -172,6 +172,8 @@ void NativeControlGtk::ViewHierarchyChanged(bool is_add, View* parent,
 }
 
 void NativeControlGtk::VisibilityChanged(View* starting_from, bool is_visible) {
+  if (starting_from != this)
+    return;

[oshima, 2011/10/10 20:48:43]

I discussed this with dave and we concluded that the correct way to fix this is
simply remove 1st if block. Since we no longer detach it's not necessary, and
Layout can show/hide native widget based on the visibility.

[sadrul, 2011/10/10 22:29:58]

Done. trybots seem happy with the change.

   if (!is_visible) {
     if (native_view()) {
       // We destroy the child widget when we become invisible because of the