summaryrefslogtreecommitdiffstats
path: root/chrome/browser/extensions/chrome_extension_function_details.cc
diff options
context:
space:
mode:
authorrockot <rockot@chromium.org>2014-09-05 13:02:08 -0700
committerCommit bot <commit-bot@chromium.org>2014-09-05 20:06:27 +0000
commit1e1bbdec0613dedaa9e43c84a9b5346a942f1841 (patch)
treeb3c6ea03e0f599e4162ba4c52a8e741435b9bb1f /chrome/browser/extensions/chrome_extension_function_details.cc
parent6a55df7b58ee45c958df5ad3e18f679d2c464106 (diff)
downloadchromium_src-1e1bbdec0613dedaa9e43c84a9b5346a942f1841.zip
chromium_src-1e1bbdec0613dedaa9e43c84a9b5346a942f1841.tar.gz
chromium_src-1e1bbdec0613dedaa9e43c84a9b5346a942f1841.tar.bz2
Add ChromeExtensionFunctionDetails
In the interest of phasing out the totally unnecessary *Chrome*ExtensionFunction base classes, this establishes a ChromeExtensionFunctionDetails object which can be created by extension function implementations in //chrome who need access to Chrome-specific details. This object can be very easily composed into such function implementations such that they can move away from using e.g. ChromeAsyncExtensionFunction or ChromeUIThreadExtensionFunction as a base class and instead rely on more universal and generic ExtensionFunction and its immediate derivatives. A conversion of the tabs API ExecuteCodeInTabFunction is included for demonstration purposes. BUG=None R=scheib@chromium.org CC=lfg@chromium.org Review URL: https://codereview.chromium.org/543873002 Cr-Commit-Position: refs/heads/master@{#293570}
Diffstat (limited to 'chrome/browser/extensions/chrome_extension_function_details.cc')
-rw-r--r--chrome/browser/extensions/chrome_extension_function_details.cc115
1 files changed, 115 insertions, 0 deletions
diff --git a/chrome/browser/extensions/chrome_extension_function_details.cc b/chrome/browser/extensions/chrome_extension_function_details.cc
new file mode 100644
index 0000000..d80c0812
--- /dev/null
+++ b/chrome/browser/extensions/chrome_extension_function_details.cc
@@ -0,0 +1,115 @@
+// Copyright 2014 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/chrome_extension_function_details.h"
+
+#include "chrome/browser/extensions/window_controller.h"
+#include "chrome/browser/extensions/window_controller_list.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/browser/render_view_host.h"
+#include "extensions/browser/extension_function.h"
+#include "extensions/browser/extension_function_dispatcher.h"
+
+using content::WebContents;
+using content::RenderViewHost;
+using extensions::WindowController;
+
+ChromeExtensionFunctionDetails::ChromeExtensionFunctionDetails(
+ UIThreadExtensionFunction* function)
+ : function_(function) {
+}
+
+ChromeExtensionFunctionDetails::~ChromeExtensionFunctionDetails() {
+}
+
+Profile* ChromeExtensionFunctionDetails::GetProfile() const {
+ return Profile::FromBrowserContext(function_->browser_context());
+}
+
+bool ChromeExtensionFunctionDetails::CanOperateOnWindow(
+ const extensions::WindowController* window_controller) const {
+ // |extension()| is NULL for unit tests only.
+ if (function_->extension() != NULL &&
+ !window_controller->IsVisibleToExtension(function_->extension())) {
+ return false;
+ }
+
+ if (GetProfile() == window_controller->profile())
+ return true;
+
+ if (!function_->include_incognito())
+ return false;
+
+ return GetProfile()->HasOffTheRecordProfile() &&
+ GetProfile()->GetOffTheRecordProfile() == window_controller->profile();
+}
+
+// TODO(stevenjb): Replace this with GetExtensionWindowController().
+Browser* ChromeExtensionFunctionDetails::GetCurrentBrowser() const {
+ // If the delegate has an associated browser, return it.
+ if (function_->dispatcher()) {
+ extensions::WindowController* window_controller =
+ function_->dispatcher()->delegate()->GetExtensionWindowController();
+ if (window_controller) {
+ Browser* browser = window_controller->GetBrowser();
+ if (browser)
+ return browser;
+ }
+ }
+
+ // Otherwise, try to default to a reasonable browser. If |include_incognito_|
+ // is true, we will also search browsers in the incognito version of this
+ // profile. Note that the profile may already be incognito, in which case
+ // we will search the incognito version only, regardless of the value of
+ // |include_incognito|. Look only for browsers on the active desktop as it is
+ // preferable to pretend no browser is open then to return a browser on
+ // another desktop.
+ if (function_->render_view_host()) {
+ Profile* profile = Profile::FromBrowserContext(
+ function_->render_view_host()->GetProcess()->GetBrowserContext());
+ Browser* browser = chrome::FindAnyBrowser(
+ profile, function_->include_incognito(), chrome::GetActiveDesktop());
+ if (browser)
+ return browser;
+ }
+
+ // NOTE(rafaelw): This can return NULL in some circumstances. In particular,
+ // a background_page onload chrome.tabs api call can make it into here
+ // before the browser is sufficiently initialized to return here, or
+ // all of this profile's browser windows may have been closed.
+ // A similar situation may arise during shutdown.
+ // TODO(rafaelw): Delay creation of background_page until the browser
+ // is available. http://code.google.com/p/chromium/issues/detail?id=13284
+ return NULL;
+}
+
+extensions::WindowController*
+ChromeExtensionFunctionDetails::GetExtensionWindowController() const {
+ // If the delegate has an associated window controller, return it.
+ if (function_->dispatcher()) {
+ extensions::WindowController* window_controller =
+ function_->dispatcher()->delegate()->GetExtensionWindowController();
+ if (window_controller)
+ return window_controller;
+ }
+
+ return extensions::WindowControllerList::GetInstance()
+ ->CurrentWindowForFunction(*this);
+}
+
+content::WebContents*
+ChromeExtensionFunctionDetails::GetAssociatedWebContents() {
+ content::WebContents* web_contents = function_->GetAssociatedWebContents();
+ if (web_contents)
+ return web_contents;
+
+ Browser* browser = GetCurrentBrowser();
+ if (!browser)
+ return NULL;
+ return browser->tab_strip_model()->GetActiveWebContents();
+}