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

Unified Diff: extensions/components/javascript_dialog_extensions_client/javascript_dialog_extension_client_impl.cc

Issue 666533007: Move JavaScriptDialogManager, JavascriptAppModalDialogViews to components/app_modal_dialogs (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: extensions/components/javascript_dialog_extensions_client/javascript_dialog_extension_client_impl.cc
diff --git a/extensions/components/javascript_dialog_extensions_client/javascript_dialog_extension_client_impl.cc b/extensions/components/javascript_dialog_extensions_client/javascript_dialog_extension_client_impl.cc
new file mode 100644
index 0000000000000000000000000000000000000000..5522b54dbe7384c784ff961b3ca3c3bbf12fd2ef
--- /dev/null
+++ b/extensions/components/javascript_dialog_extensions_client/javascript_dialog_extension_client_impl.cc
@@ -0,0 +1,80 @@
+// 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 "extensions/components/javascript_dialog_extensions_client/javascript_dialog_extension_client_impl.h"
+
+#include "components/app_modal_dialogs/javascript_dialog_extensions_client.h"
+#include "components/app_modal_dialogs/javascript_dialog_manager.h"
+#include "content/public/browser/web_contents.h"
+#include "extensions/browser/process_manager.h"
+#include "extensions/common/extension.h"
+#include "ui/gfx/native_widget_types.h"
+
+namespace {
+
+using extensions::Extension;
+
+// Returns the ProcessManager for the browser context from |web_contents|.
+extensions::ProcessManager* GetProcessManager(
+ content::WebContents* web_contents) {
+ return extensions::ProcessManager::Get(web_contents->GetBrowserContext());
+}
+
+// Returns the extension associated with |web_contents| or NULL if there is no
+// associated extension (or extensions are not supported).
+const Extension* GetExtensionForWebContents(
+ content::WebContents* web_contents) {
+ extensions::ProcessManager* pm = GetProcessManager(web_contents);
+ return pm->GetExtensionForRenderViewHost(web_contents->GetRenderViewHost());
+}
+
+class JavaScriptDialogExtensionsClientImpl
+ : public JavaScriptDialogExtensionsClient {
+ public:
+ JavaScriptDialogExtensionsClientImpl() {}
+ ~JavaScriptDialogExtensionsClientImpl() override {}
+
+ // JavaScriptDialogExtensionsClient:
+ void OnDialogOpened(content::WebContents* web_contents) override {
+ const Extension* extension = GetExtensionForWebContents(web_contents);
+ if (extension == nullptr)
+ return;
+
+ DCHECK(web_contents);
+ extensions::ProcessManager* pm = GetProcessManager(web_contents);
+ if (pm)
+ pm->IncrementLazyKeepaliveCount(extension);
+ }
+ void OnDialogClosed(content::WebContents* web_contents) override {
+ const Extension* extension = GetExtensionForWebContents(web_contents);
+ if (extension == nullptr)
+ return;
+
+ DCHECK(web_contents);
+ extensions::ProcessManager* pm = GetProcessManager(web_contents);
+ if (pm)
+ pm->DecrementLazyKeepaliveCount(extension);
+ }
+ bool GetExtensionName(content::WebContents* web_contents,
+ const GURL& origin_url,
+ std::string* name_out) override {
+ const Extension* extension = GetExtensionForWebContents(web_contents);
+ if (extension &&
+ web_contents->GetLastCommittedURL().GetOrigin() == origin_url) {
+ *name_out = extension->name();
+ return true;
+ }
+ return false;
+ }
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(JavaScriptDialogExtensionsClientImpl);
+};
+
+} // namespace
+
+void InstallJavaScriptDialogExtensionsClient() {
+ SetJavaScriptDialogExtensionsClient(
+ make_scoped_ptr(new JavaScriptDialogExtensionsClientImpl));
+}

Powered by Google App Engine
This is Rietveld 408576698