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

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

Issue 9428018: Create BaseWindow and ExtensionWindowWrapper for extension API access to Panels (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 10 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_wrapper.cc
diff --git a/chrome/browser/extensions/browser_extension_window_wrapper.cc b/chrome/browser/extensions/browser_extension_window_wrapper.cc
new file mode 100644
index 0000000000000000000000000000000000000000..323c409d680f2f0ff74bda85cd0750b5381516fe
--- /dev/null
+++ b/chrome/browser/extensions/browser_extension_window_wrapper.cc
@@ -0,0 +1,63 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/extensions/browser_extension_window_wrapper.h"
+
+#include "chrome/browser/extensions/extension_tab_util.h"
+#include "chrome/browser/extensions/extension_tabs_module_constants.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/sessions/session_id.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_window.h"
+
+BrowserExtensionWindowWrapper::BrowserExtensionWindowWrapper(Browser* browser)
+ : ExtensionWindowWrapper(browser->window(), browser->profile()),
+ browser_(browser) {
+}
+
+const SessionID& BrowserExtensionWindowWrapper::GetSessionId() const {
+ return browser_->session_id();
+}
+
+namespace keys = extension_tabs_module_constants;
+
+base::DictionaryValue* BrowserExtensionWindowWrapper::CreateWindowValue(
+ bool populate_tabs) const {
+ DictionaryValue* result = new DictionaryValue();
sky 2012/02/22 21:46:25 There's a lot of duplicate code between this and t
stevenjb 2012/02/23 02:19:15 Done.
+ result->SetInteger(keys::kIdKey, browser_->session_id().id());
+ result->SetBoolean(keys::kIncognitoKey,
+ browser_->profile()->IsOffTheRecord());
+ result->SetBoolean(keys::kFocusedKey, window()->IsActive());
+
+ gfx::Rect bounds;
+ if (window()->IsMaximized() || browser_->window()->IsFullscreen())
+ bounds = window()->GetBounds();
+ else
+ bounds = window()->GetRestoredBounds();
+ result->SetInteger(keys::kLeftKey, bounds.x());
+ result->SetInteger(keys::kTopKey, bounds.y());
+ result->SetInteger(keys::kWidthKey, bounds.width());
+ result->SetInteger(keys::kHeightKey, bounds.height());
+
+ result->SetString(keys::kWindowTypeKey,
+ ExtensionTabUtil::GetWindowTypeText(browser_));
+ result->SetString(keys::kShowStateKey,
+ ExtensionTabUtil::GetWindowShowStateText(browser_));
+
+ if (populate_tabs)
+ result->Set(keys::kTabsKey, ExtensionTabUtil::CreateTabList(browser_));
+
+ return result;
+}
+
+bool BrowserExtensionWindowWrapper::CanClose(
+ ExtensionWindowWrapper::Reason* reason) const {
+ // Don't let an extension remove the window if the user is dragging tabs
+ // in that window.
+ if (!browser_->IsTabStripEditable()) {
+ *reason = ExtensionWindowWrapper::REASON_TAB_STRIP_NOT_EDITABLE;
+ return false;
+ }
+ return true;
+}

Powered by Google App Engine
This is Rietveld 408576698