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

chrome/browser/ui/panels/panel_manager.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_manager.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
+#include "chrome/browser/fullscreen.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/panels/panel_mouse_watcher.h"
 #include "chrome/browser/ui/window_sizer.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 
 namespace {
 // Invalid panel index.
@@ skipping 39 matching lines @@
 
 PanelManager::PanelManager()
     : minimized_panel_count_(0),
       are_titlebars_up_(false),
       dragging_panel_index_(kInvalidPanelIndex),
       dragging_panel_original_x_(0),
       delayed_titlebar_action_(NO_ACTION),
       remove_delays_for_testing_(false),
       titlebar_action_factory_(this),
       auto_sizing_enabled_(true),
-      mouse_watching_disabled_(false) {
+      mouse_watching_disabled_(false),
+      is_full_screen_mode_on_(false) {
   panel_mouse_watcher_.reset(PanelMouseWatcher::Create());
   auto_hiding_desktop_bar_ = AutoHidingDesktopBar::Create(this);
   OnDisplayChanged();
 }
 
 PanelManager::~PanelManager() {
   DCHECK(panels_.empty());
   DCHECK(panels_pending_to_remove_.empty());
   DCHECK_EQ(0, minimized_panel_count_);
 }
@@ skipping 62 matching lines @@
   int x = GetRightMostAvailablePosition() - width;
   panel->Initialize(gfx::Rect(x, y, width, height));
 
   panels_.push_back(panel);
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_PANEL_ADDED,
       content::Source<Panel>(panel),
       content::NotificationService::NoDetails());
 
+  if (num_panels() == 1) {

[dcheng, 2011/11/30 01:07:28]

Use panels_.size() -- it makes the connection more obvious--I have to read the
.h to know that num_panels() is actually panels_.size().

[jennb, 2011/11/30 02:06:27]

panels_ is going away. use num_panels().

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

Done.

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

See Jenn's comment.

+    full_screen_mode_timer_.Start(FROM_HERE,
+        base::TimeDelta::FromMilliseconds(kFullScreenModeCheckInterval),
+        this, &PanelManager::CheckFullScreenMode);
+  }
+
   return panel;
 }
 
 int PanelManager::GetMaxPanelWidth() const {
   return static_cast<int>(adjusted_work_area_.width() * kPanelMaxWidthFactor);
 }
 
 int PanelManager::GetMaxPanelHeight() const {
   return static_cast<int>(adjusted_work_area_.height() * kPanelMaxHeightFactor);
 }
 
 int PanelManager::StartingRightPosition() const {
   return adjusted_work_area_.right() - kRightScreenEdgeSpacingWidth;
 }
 
 int PanelManager::GetRightMostAvailablePosition() const {
   return panels_.empty() ? StartingRightPosition() :
       (panels_.back()->GetBounds().x() - kPanelsHorizontalSpacing);
 }
 
+void PanelManager::CheckFullScreenMode() {
+  bool is_full_screen_mode_on_new = IsFullScreenMode();
+  if (is_full_screen_mode_on_ == is_full_screen_mode_on_new)
+    return;
+
+  is_full_screen_mode_on_ = is_full_screen_mode_on_new;
+  for (size_t i = 0; i < panels_.size(); ++i)
+    panels_[i]->FullScreenModeChanged(is_full_screen_mode_on_);

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

if you do panel->native_panel()->FooBar() here, you don't need a re-translating
method on Panel.

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

Currently PanelManager does not access native_panel directly. Is there value in
keeping that separation?

[Dmitry Titov, 2011/12/01 04:04:26]

What would be the reason for such separation? NativePanel is an abstract
interface specifically introduced to insulate the caller from platform-dependent
implementation and extend BrowserWindow which is not always enough for our
purposes. Interface itself is very platform-independent so I don't see any
reason to avoid using it in PanelManager.

There is a reason to keep it separate from Panel and BrowserWindow, but that's
an orthogonal story (mostly because it is implemented separately, esp on Mac).

Don't see any reason to add more translating methods.

[prasadt, 2011/12/01 04:20:48]

Done.

NativePanel interface is currently protected and Panel/PanelBrowserTest are
friends. Made the interface public so I can use it from PanelStrip and removed
friends. It seems an odd pattern to make it protected and then add friends.

[jennb, 2011/12/01 21:22:09]

Another reason for the separation is that the Panel often performs some
platform-independent logic first, then calls the NativePanel layer to do some
platform-dependent logic. Without the separation, the caller has to know when
it's safe to go straight to the platform-dependent layer.

[prasadt, 2011/12/01 21:35:16]

Now that I think about it, this makes sense to me.

One way to think of this could be to Panel+NavitePanelImpl as one class that's
separated for implementation reasons. If we had a single platform to support,
this would be one class. An alternate but messy implementation would've been one
class with a bunch of #ifdefs.

+}
+
 void PanelManager::Remove(Panel* panel) {
   // If we're in the process of dragging, delay the removal.
   if (dragging_panel_index_ != kInvalidPanelIndex) {
     panels_pending_to_remove_.push_back(panel);
     return;
   }
 
   DoRemove(panel);
 }
 
@@ skipping 11 matching lines @@
   if (panel->expansion_state() != Panel::EXPANDED)
     DecrementMinimizedPanels();
 
   gfx::Rect bounds = (*iter)->GetBounds();
   Rearrange(panels_.erase(iter), bounds.right());
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_PANEL_REMOVED,
       content::Source<Panel>(panel),
       content::NotificationService::NoDetails());
+
+  if (num_panels() == 0)

[dcheng, 2011/11/30 01:07:28]

Use panels_.empty().

[jennb, 2011/11/30 02:06:27]

Ditto.

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

Done.

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

See Jenn's comment.

+    full_screen_mode_timer_.Stop();
 }
 
 void PanelManager::StartDragging(Panel* panel) {
   for (size_t i = 0; i < panels_.size(); ++i) {
     if (panels_[i] == panel) {
       dragging_panel_index_ = i;
       dragging_panel_bounds_ = panel->GetBounds();
       dragging_panel_original_x_ = dragging_panel_bounds_.x();
       break;
     }
@@ skipping 428 matching lines @@
 
   // Start from the bottom to avoid reshuffling.
   for (Panels::reverse_iterator iter = panels_copy.rbegin();
        iter != panels_copy.rend(); ++iter)
     (*iter)->Close();
 }
 
 bool PanelManager::is_dragging_panel() const {
   return dragging_panel_index_ != kInvalidPanelIndex;
 }