Have panels respond to changes in work area on Linux.

chrome/browser/ui/panels/panel_browser_window_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 "chrome/browser/ui/panels/panel_browser_window_gtk.h"
 
 #include "base/bind.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
@@ skipping 52 matching lines @@
 
 PanelBrowserWindowGtk::~PanelBrowserWindowGtk() {
   if (drag_widget_) {
     // Terminate the grab if we have it. We could do this using any widget,
     // |drag_widget_| is just convenient.
     gtk_grab_add(drag_widget_);
     gtk_grab_remove(drag_widget_);
     DestroyDragWidget();
   }
   panel_->OnNativePanelClosed();
+  ui::WorkAreaWatcherX::RemoveObserver(this);
 }
 
 void PanelBrowserWindowGtk::Init() {
   BrowserWindowGtk::Init();
 
   // Keep the window docked to the bottom of the screen on resizes.
   gtk_window_set_gravity(window(), GDK_GRAVITY_SOUTH_EAST);
 
   // Keep the window always on top.
   gtk_window_set_keep_above(window(), TRUE);
 
   // Show the window on all the virtual desktops.
   gtk_window_stick(window());
 
   // Do not show an icon in the task bar.  Window operations such as close,
   // minimize etc. can only be done from the panel UI.
   gtk_window_set_skip_taskbar_hint(window(), TRUE);
 
   g_signal_connect(titlebar_widget(), "button-press-event",
                    G_CALLBACK(OnTitlebarButtonPressEventThunk), this);
   g_signal_connect(titlebar_widget(), "button-release-event",
                    G_CALLBACK(OnTitlebarButtonReleaseEventThunk), this);
+
+  // Strictly speaking we should have a single observer for work area changed

[Dmitry Titov, 2011/11/18 04:56:37]

Not sure that long comment has to be here. If it's not a TODO, I don't see what
it adds. Remove?

[prasadt, 2011/11/18 22:31:42]

Just a clarification in case someone wonders why its done this way. Removed it
as it doesn't seem to help.

+  // notification. There are a few reasons for doing it this way:
+  // 1) Keep it consistent with the way we do it on other platforms.
+  // 2) This is a rare enough event to create a separate class with
+  //    implementations for all platforms.
+  // 3) It may not be possible to listen to this event without assoicating it
+  //    with a window on all platforms.
+  ui::WorkAreaWatcherX::AddObserver(this);
 }
 
 bool PanelBrowserWindowGtk::GetWindowEdge(int x, int y, GdkWindowEdge* edge) {
   // Since panels are not resizable or movable by the user, we should not
   // detect the window edge for behavioral purposes.  The edge, if any,
   // is present only for visual aspects.
   return FALSE;
 }
 
 bool PanelBrowserWindowGtk::HandleTitleBarLeftMousePress(
@@ skipping 113 matching lines @@
                                            NULL);
     *title = title_with_markup;
     g_free(title_escaped);
     g_free(title_with_markup);
     return BrowserWindowGtk::PANGO_MARKUP;
   } else {
     return BrowserWindowGtk::GetWindowTitle(title);
   }
 }
 
+void PanelBrowserWindowGtk::WorkAreaChanged() {
+  panel_->manager()->OnDisplayChanged();
+}
+
 void PanelBrowserWindowGtk::ShowPanel() {
   Show();
 }
 
 void PanelBrowserWindowGtk::ShowPanelInactive() {
   ShowInactive();
 }
 
 gfx::Rect PanelBrowserWindowGtk::GetPanelBounds() const {
   return bounds_;
@@ skipping 490 matching lines @@
   }
 }
 
 bool NativePanelTestingGtk::IsWindowSizeKnown() const {
   return panel_browser_window_gtk_->window_size_known_;
 }
 
 bool NativePanelTestingGtk::IsAnimatingBounds() const {
   return panel_browser_window_gtk_->IsAnimatingBounds();
 }