Make panels not show on top when there is an app running in full screen mode.

chrome/browser/ui/panels/panel_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_
 #define CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_
 #pragma once
 
 #include <vector>
 #include "base/basictypes.h"
 #include "base/lazy_instance.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "base/timer.h"
 #include "chrome/browser/ui/panels/auto_hiding_desktop_bar.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_mouse_watcher_observer.h"
 #include "ui/gfx/rect.h"
 
 class Browser;
 class Panel;
 class PanelMouseWatcher;
 
 // This class manages a set of panels.
@@ skipping 59 matching lines @@
   // Moves a panel to the panel strip. The panel does not currently
   // belong in any other strip. This may cause other panels to be
   // bumped out of the panel strip.
   void MoveToPanelStrip(Panel* panel);
 
   // Moves a panel to the overflow strip. The panel does not currently
   // belong in any other strip.
   // |is_new| is true if the panel was just created.
   void MoveToOverflowStrip(Panel* panel, bool is_new);
 
-  // Overridden from PanelMouseWatcherObserver:
+  // Overridden from PanelMouseWatcher::Observer:

[Dmitry Titov, 2011/11/30 01:35:53]

Isn't it now PanelMouseWatcherObserver?

[prasadt, 2011/11/30 02:38:47]

Done.

   virtual void OnMouseMove(const gfx::Point& mouse_position) OVERRIDE;
 
+  bool is_full_screen_mode_on()  const { return is_full_screen_mode_on_; }
+
 #ifdef UNIT_TEST
   static int horizontal_spacing() { return kPanelsHorizontalSpacing; }
 
   const gfx::Rect& work_area() const {
     return work_area_;
   }
 
   void set_auto_hiding_desktop_bar(
       AutoHidingDesktopBar* auto_hiding_desktop_bar) {
     auto_hiding_desktop_bar_ = auto_hiding_desktop_bar;
@@ skipping 118 matching lines @@
   // Owned by MessageLoop after posting.
   base::WeakPtrFactory<PanelManager> titlebar_action_factory_;
 
   // Whether or not bounds will be updated when the preferred content size is
   // changed. The testing code could set this flag to false so that other tests
   // will not be affected.
   bool auto_sizing_enabled_;
 
   bool mouse_watching_disabled_;  // For tests to simulate mouse movements.
 
+  // Timer used to track if the current active app is in full screen mode.
+  base::RepeatingTimer<PanelManager> full_screen_mode_timer_;
+  void CheckFullScreenMode();
+  bool is_full_screen_mode_on_;
+
   static const int kPanelsHorizontalSpacing = 4;
 
   // Absolute minimum width and height for panels, including non-client area.
   // Should only be big enough to accomodate a close button on the reasonably
   // recognisable titlebar.
   static const int kPanelMinWidth;
   static const int kPanelMinHeight;
 
+  static const int kFullScreenModeCheckInterval = 1000;

[Dmitry Titov, 2011/11/30 01:35:53]

this is implementation detail, does not have to be in .h file. It is tricky to
have initialized statics in .h file, not recommended in fact. There was a thread
on chromium-dev about it recently.

[jianli, 2011/11/30 01:53:15]

Can this constant be moved to .cc file?

[prasadt, 2011/11/30 02:38:47]

Done.

[prasadt, 2011/11/30 02:38:47]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(PanelManager);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_