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

views/controls/native_control_gtk.cc

 // Copyright (c) 2011 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 "views/controls/native_control_gtk.h"
 
 #include <gtk/gtk.h>
 
 #include "base/logging.h"
 #include "ui/base/accessibility/accessibility_types.h"
@@ skipping 154 matching lines @@
   if (!is_add && child == this && native_view()) {
     Detach();
   } else if (is_add && GetWidget() && !native_view()) {
     // Create the widget when we're added to a valid Widget. Many
     // controls need a parent widget to function properly.
     CreateNativeControl();
   }
 }
 
 void NativeControlGtk::VisibilityChanged(View* starting_from, bool is_visible) {
+  if (starting_from != this)
+    return;

[sadrul, 2011/10/08 23:21:08]

This change is to make sure chromium-os:20545 remains fixed (more in
http://codereview.chromium.org/7978015)

   if (!is_visible) {
     if (native_view()) {
       // We destroy the child widget when we become invisible because of the
       // performance cost of maintaining widgets that aren't currently needed.
       Detach();
       // Make sure that Detach destroyed the widget.
       DCHECK(!native_view());
     }
   } else if (!native_view()) {
     if (GetWidget())
@@ skipping 57 matching lines @@
     // options page is only based on views.
     // NOTREACHED();
     NOTIMPLEMENTED();
     return false;
   }
   focus_manager->SetFocusedView(control->focus_view());
   return false;
 }
 
 }  // namespace views