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

Unified Diff: components/renderer_context_menu/context_menu_content_type.cc

Issue 1117893002: Moving extension code out of "components/" to avoid layering violations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressing nit. Created 5 years, 7 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: components/renderer_context_menu/context_menu_content_type.cc
diff --git a/components/renderer_context_menu/context_menu_content_type.cc b/components/renderer_context_menu/context_menu_content_type.cc
index 668789180453eb178e3a319174a243d0f101d323..7f33b00ab92deec7591c9f90d23ddcba6e96d6cf 100644
--- a/components/renderer_context_menu/context_menu_content_type.cc
+++ b/components/renderer_context_menu/context_menu_content_type.cc
@@ -9,19 +9,9 @@
#include "content/public/common/url_constants.h"
#include "third_party/WebKit/public/web/WebContextMenuData.h"
-#if defined(ENABLE_EXTENSIONS)
-#include "extensions/browser/process_manager.h"
-#include "extensions/common/extension.h"
-#endif
-
using blink::WebContextMenuData;
using content::WebContents;
-#if defined(ENABLE_EXTENSIONS)
-using extensions::Extension;
-using extensions::ProcessManager;
-#endif
-
namespace {
bool IsDevToolsURL(const GURL& url) {
@@ -48,15 +38,6 @@ ContextMenuContentType::ContextMenuContentType(
ContextMenuContentType::~ContextMenuContentType() {
}
-#if defined(ENABLE_EXTENSIONS)
-const Extension* ContextMenuContentType::GetExtension() const {
- ProcessManager* process_manager =
- ProcessManager::Get(source_web_contents_->GetBrowserContext());
- return process_manager->GetExtensionForWebContents(
- source_web_contents_);
-}
-#endif
-
bool ContextMenuContentType::SupportsGroup(int group) {
const bool has_selection = !params_.selection_text.empty();

Powered by Google App Engine
This is Rietveld 408576698