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

Side by Side Diff: chrome/browser/ui/panels/panel_browser_frame_view.h

Issue 8539035: Fix 3 panel related bugs. (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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_PANELS_PANEL_BROWSER_FRAME_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_PANELS_PANEL_BROWSER_FRAME_VIEW_H_
6 #define CHROME_BROWSER_UI_PANELS_PANEL_BROWSER_FRAME_VIEW_H_ 6 #define CHROME_BROWSER_UI_PANELS_PANEL_BROWSER_FRAME_VIEW_H_
7 #pragma once 7 #pragma once
8 8
9 #include "base/gtest_prod_util.h" 9 #include "base/gtest_prod_util.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
154 154
155 // Make settings button visible if either of the conditions is met: 155 // Make settings button visible if either of the conditions is met:
156 // 1) The panel is active, i.e. having focus. 156 // 1) The panel is active, i.e. having focus.
157 // 2) The mouse is over the panel. 157 // 2) The mouse is over the panel.
158 void UpdateSettingsButtonVisibility(bool active, bool cursor_in_view); 158 void UpdateSettingsButtonVisibility(bool active, bool cursor_in_view);
159 159
160 const Extension* GetExtension() const; 160 const Extension* GetExtension() const;
161 161
162 bool EnsureSettingsMenuCreated(); 162 bool EnsureSettingsMenuCreated();
163 163
164 string16 GetTitleText() const;
165
164 #ifdef UNIT_TEST 166 #ifdef UNIT_TEST
165 PanelSettingsMenuModel* settings_menu_model() const { 167 PanelSettingsMenuModel* settings_menu_model() const {
166 return settings_menu_model_.get(); 168 return settings_menu_model_.get();
167 } 169 }
168 170
169 void set_mouse_watcher(MouseWatcher* mouse_watcher) { 171 void set_mouse_watcher(MouseWatcher* mouse_watcher) {
170 mouse_watcher_.reset(mouse_watcher); 172 mouse_watcher_.reset(mouse_watcher);
171 } 173 }
172 #endif 174 #endif
173 175
(...skipping 17 matching lines...) Expand all
191 // Used to animate the visibility change of settings button. 193 // Used to animate the visibility change of settings button.
192 scoped_ptr<ui::SlideAnimation> settings_button_animator_; 194 scoped_ptr<ui::SlideAnimation> settings_button_animator_;
193 gfx::Rect settings_button_full_bounds_; 195 gfx::Rect settings_button_full_bounds_;
194 gfx::Rect settings_button_zero_bounds_; 196 gfx::Rect settings_button_zero_bounds_;
195 bool is_settings_button_visible_; 197 bool is_settings_button_visible_;
196 198
197 DISALLOW_COPY_AND_ASSIGN(PanelBrowserFrameView); 199 DISALLOW_COPY_AND_ASSIGN(PanelBrowserFrameView);
198 }; 200 };
199 201
200 #endif // CHROME_BROWSER_UI_PANELS_PANEL_BROWSER_FRAME_VIEW_H_ 202 #endif // CHROME_BROWSER_UI_PANELS_PANEL_BROWSER_FRAME_VIEW_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/ui/panels/panel_browser_frame_view.cc » ('j') | chrome/browser/ui/panels/panel_browser_view.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698