Panels back behind a flag.

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"
@@ skipping 17 matching lines @@
 class PanelStrip;
 
 // This class manages a set of panels.
 class PanelManager : public AutoHidingDesktopBar::Observer {
  public:
   typedef std::vector<Panel*> Panels;
 
   // Returns a single instance.
   static PanelManager* GetInstance();
 
+  // Returns true if panels should always be used.
+  static bool ShouldUsePanels(const std::string& id);

[Dmitry Titov, 2011/12/08 20:59:01]

could we name the param 'extension_id' to make it more descriptive?
Another variant would be to name the method EnablePanelsForExtension(string id);

[asargent_no_longer_on_chrome, 2011/12/08 21:21:26]

nit: it might make sense to either rename the parameter extension_id, or mention
in the comment something like "...used for the extension with |id|."

+
   // Called when the display is changed, i.e. work area is updated.
   void OnDisplayChanged();
 
   // Creates a panel and returns it. The panel might be queued for display
   // later.
   Panel* CreatePanel(Browser* browser);
 
   void Remove(Panel* panel);
   void RemoveAll();
 
@@ skipping 134 matching lines @@
   // Timer used to track if the current active app is in full screen mode.
   base::RepeatingTimer<PanelManager> full_screen_mode_timer_;
 
   // True if current active app is in full screen mode.
   bool is_full_screen_;
 
   DISALLOW_COPY_AND_ASSIGN(PanelManager);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_