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

Unified Diff: chrome/browser/ui/fullscreen_controller.cc

Issue 10067033: RefCounted types should not have public destructors, chrome/browser/ui (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge fix Created 8 years, 8 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
« no previous file with comments | « chrome/browser/ui/fullscreen_controller.h ('k') | chrome/browser/ui/login/login_prompt.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/fullscreen_controller.cc
diff --git a/chrome/browser/ui/fullscreen_controller.cc b/chrome/browser/ui/fullscreen_controller.cc
index 24f363a476d1efec23817b2ce2aaf4a0c5af496a..f0e3514b5fca94014353dde3307c42de4d0925e6 100644
--- a/chrome/browser/ui/fullscreen_controller.cc
+++ b/chrome/browser/ui/fullscreen_controller.cc
@@ -37,8 +37,6 @@ FullscreenController::FullscreenController(BrowserWindow* window,
mouse_lock_state_(MOUSELOCK_NOT_REQUESTED) {
}
-FullscreenController::~FullscreenController() {}
-
bool FullscreenController::IsFullscreenForBrowser() const {
return window_->IsFullscreen() && !tab_caused_fullscreen_;
}
@@ -271,6 +269,8 @@ bool FullscreenController::HandleUserPressedEscape() {
return true;
}
+FullscreenController::~FullscreenController() {}
+
void FullscreenController::NotifyTabOfFullscreenExitIfNecessary() {
if (fullscreened_tab_) {
RenderViewHost* rvh =
« no previous file with comments | « chrome/browser/ui/fullscreen_controller.h ('k') | chrome/browser/ui/login/login_prompt.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698