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

views/widget/native_widget_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/widget/native_widget_gtk.h"
 
 #include <gdk/gdk.h>
 #include <gdk/gdkx.h>
 #include <X11/extensions/shape.h>
 #include <X11/Xatom.h>
@@ skipping 1178 matching lines @@
 
 void NativeWidgetGtk::EnableClose(bool enable) {
   gtk_window_set_deletable(GetNativeWindow(), enable);
 }
 
 void NativeWidgetGtk::Show() {
   if (widget_) {
     gtk_widget_show(widget_);
     if (widget_->window)
       gdk_window_raise(widget_->window);
-    // See Hide() for the reason why we're not calling
-    // OnNativeWidgetVisibilityChange.
   }
 }
 
 void NativeWidgetGtk::Hide() {
   if (widget_) {
     gtk_widget_hide(widget_);
     if (widget_->window)
       gdk_window_lower(widget_->window);
-    // We're not calling OnNativeWidgetVisibilityChanged because it
-    // breaks the ability to refocus to FindBar. NativeControlGtk
-    // detaches the underlying gtk widget for optimization purpose
-    // when it becomes invisible, which in turn breaks SetNativeFocus
-    // because there is no gtk attached to NativeControlGtk. I'm not
-    // fixing that part because
-    // a) This is views/gtk only issue, which will be gone soon.
-    // b) Alternative fix, which we can modify animator to show it
-    // immediately, won't be necessary for non gtk implementation.
   }
 }
 
 void NativeWidgetGtk::ShowMaximizedWithBounds(
     const gfx::Rect& restored_bounds) {
   // TODO: when we add maximization support update this.
   Show();
 }
 
 void NativeWidgetGtk::ShowWithWindowState(ui::WindowShowState show_state) {
@@ skipping 590 matching lines @@
   widget_ = window_contents_ = NULL;
 }
 
 void NativeWidgetGtk::OnDestroyed(GObject *where_the_object_was) {
   delegate_->OnNativeWidgetDestroyed();
   if (ownership_ == Widget::InitParams::NATIVE_WIDGET_OWNS_WIDGET)
     delete this;
 }
 
 void NativeWidgetGtk::OnShow(GtkWidget* widget) {
+  delegate_->OnNativeWidgetVisibilityChanged(true);
 }
 
 void NativeWidgetGtk::OnMap(GtkWidget* widget) {
 #if defined(TOUCH_UI)
   // Force an expose event to trigger OnPaint for touch. This is
   // a workaround for a bug that X Expose event does not trigger
   // Gdk's expose signal. This happens when you try to open views menu
   // while a virtual keyboard gets kicked in or out. This seems to be
   // a bug in message_pump_x.cc as we do get X Expose event but
   // it doesn't trigger gtk's expose signal. We're not going to fix this
   // as we're removing gtk and migrating to new compositor.
   gdk_window_process_all_updates();
 #endif
 }
 
 void NativeWidgetGtk::OnHide(GtkWidget* widget) {
+  delegate_->OnNativeWidgetVisibilityChanged(false);
 }
 
 gboolean NativeWidgetGtk::OnWindowStateEvent(GtkWidget* widget,
                                              GdkEventWindowState* event) {
   if (GetWidget()->non_client_view() &&
       !(event->new_window_state & GDK_WINDOW_STATE_WITHDRAWN)) {
     SaveWindowPosition();
   }
   window_state_ = event->new_window_state;
   return FALSE;
@@ skipping 460 matching lines @@
     button_pressed = event->type == GDK_BUTTON_PRESS ||
         event->type == GDK_2BUTTON_PRESS ||
         event->type == GDK_3BUTTON_PRESS;
     gdk_event_free(event);
   }
   return button_pressed;
 }
 
 }  // namespace internal
 }  // namespace views