Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1696)

Unified Diff: chrome/browser/ui/panels/panel_manager.cc

Issue 8774013: Add panel overflow logic to panel strip. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/panels/panel_manager.cc
diff --git a/chrome/browser/ui/panels/panel_manager.cc b/chrome/browser/ui/panels/panel_manager.cc
index ae5bbe22801b42eaf74ecd6ded6ad688ff0ced65..93eb80ad9df1e13b3c6fbe6b46ecb31a04ccbd50 100644
--- a/chrome/browser/ui/panels/panel_manager.cc
+++ b/chrome/browser/ui/panels/panel_manager.cc
@@ -76,7 +76,7 @@ void PanelManager::Layout() {
panel_strip_bounds.set_width(adjusted_work_area_.width() -
kPanelStripLeftMargin - kPanelStripRightMargin);
panel_strip_bounds.set_height(height);
- panel_strip_->SetBounds(panel_strip_bounds);
+ panel_strip_->SetDisplayArea(panel_strip_bounds);
}
Panel* PanelManager::CreatePanel(Browser* browser) {
@@ -103,13 +103,6 @@ void PanelManager::Remove(Panel* panel) {
// TODO(jianli): else try removing from overflow strip
}
-void PanelManager::OnPanelRemoved(Panel* panel) {
- content::NotificationService::current()->Notify(
- chrome::NOTIFICATION_PANEL_REMOVED,
- content::Source<Panel>(panel),
- content::NotificationService::NoDetails());
-}
-
void PanelManager::StartDragging(Panel* panel) {
panel_strip_->StartDragging(panel);
}

Powered by Google App Engine
This is Rietveld 408576698