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

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

Issue 10736028: Refactor browser window zoom handling and enable zoom icon on all platforms. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 4 months 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_browser_window.cc
diff --git a/chrome/browser/ui/panels/panel_browser_window.cc b/chrome/browser/ui/panels/panel_browser_window.cc
index ddac72c464489ca43eae958db51302214a3eb90f..57163fca24a2d994efd32fe6727362f8e8a1601a 100644
--- a/chrome/browser/ui/panels/panel_browser_window.cc
+++ b/chrome/browser/ui/panels/panel_browser_window.cc
@@ -154,17 +154,9 @@ void PanelBrowserWindow::SetStarredState(bool is_starred) {
// have starred state.
}
-void PanelBrowserWindow::SetZoomIconState(ZoomController::ZoomIconState state) {
- // Since panels don't have an Omnibox, they don't have a zoom icon.
-}
-
-void PanelBrowserWindow::SetZoomIconTooltipPercent(int zoom_percent) {
- // Since panels don't have an Omnibox, they don't have a zoom icon.
-}
-
-void PanelBrowserWindow::ShowZoomBubble(int zoom_percent) {
+void PanelBrowserWindow::ZoomChangedForActiveTab(bool can_show_bubble) {
// Since panels don't have an Omnibox, they don't have a zoom icon, so no
- // bubble will appear from it.
+ // view icon nor bubble will appear for this action.
}
void PanelBrowserWindow::EnterFullscreen(

Powered by Google App Engine
This is Rietveld 408576698