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

Unified Diff: chrome/browser/extensions/browser_extension_window_controller.cc

Issue 10021071: Remove Browser dependency in ExtensionFunctionDispatcher (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase 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
Index: chrome/browser/extensions/browser_extension_window_controller.cc
diff --git a/chrome/browser/extensions/browser_extension_window_controller.cc b/chrome/browser/extensions/browser_extension_window_controller.cc
index e0d6a0b82e41821d0d3fe92d29adff88e92e7e2c..ed647bd3d8e9910b614824eeaea96ce579f3e160 100644
--- a/chrome/browser/extensions/browser_extension_window_controller.cc
+++ b/chrome/browser/extensions/browser_extension_window_controller.cc
@@ -48,12 +48,11 @@ BrowserExtensionWindowController::CreateWindowValueWithTabs() const {
return result;
}
-bool BrowserExtensionWindowController::CanClose(
- ExtensionWindowController::Reason* reason) const {
+bool BrowserExtensionWindowController::CanClose(Reason* reason) const {
// Don't let an extension remove the window if the user is dragging tabs
// in that window.
if (!browser_->IsTabStripEditable()) {
- *reason = ExtensionWindowController::REASON_TAB_STRIP_NOT_EDITABLE;
+ *reason = ExtensionWindowController::REASON_NOT_EDITABLE;
return false;
}
return true;
@@ -65,3 +64,7 @@ void BrowserExtensionWindowController::SetFullscreenMode(
if (browser_->window()->IsFullscreen() != is_fullscreen)
browser_->ToggleFullscreenModeWithExtension(extension_url);
}
+
+Browser* BrowserExtensionWindowController::GetBrowser() const {
+ return browser_;
+}

Powered by Google App Engine
This is Rietveld 408576698