Moving app_view to extensions.

extensions/browser/guest_view/app_view/app_view_guest.cc

Index: extensions/browser/guest_view/app_view/app_view_guest.cc
diff --git a/chrome/browser/guest_view/app_view/app_view_guest.cc b/extensions/browser/guest_view/app_view/app_view_guest.cc
similarity index 88%
rename from chrome/browser/guest_view/app_view/app_view_guest.cc
rename to extensions/browser/guest_view/app_view/app_view_guest.cc
index 6aadc186b428dc81ceb8acacb0ea2fc5daa6a24d..6d9edfc362e238f25012a863662b5e5b56c3fe61 100644
--- a/chrome/browser/guest_view/app_view/app_view_guest.cc
+++ b/extensions/browser/guest_view/app_view/app_view_guest.cc
@@ -2,26 +2,25 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/guest_view/app_view/app_view_guest.h"
+#include "extensions/browser/guest_view/app_view/app_view_guest.h"
 
 #include "base/command_line.h"
-#include "chrome/browser/extensions/chrome_extension_web_contents_observer.h"
-#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/guest_view/app_view/app_view_constants.h"
-#include "chrome/browser/renderer_context_menu/render_view_context_menu.h"
-#include "chrome/common/chrome_switches.h"
-#include "components/renderer_context_menu/context_menu_delegate.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/common/renderer_preferences.h"
 #include "extensions/browser/api/app_runtime/app_runtime_api.h"
+#include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_host.h"
+#include "extensions/browser/extension_registry.h"
 #include "extensions/browser/extension_system.h"
+#include "extensions/browser/guest_view/app_view/app_view_constants.h"
 #include "extensions/browser/guest_view/guest_view_manager.h"
 #include "extensions/browser/lazy_background_task_queue.h"
+#include "extensions/browser/process_manager.h"
 #include "extensions/browser/view_type_utils.h"
 #include "extensions/common/api/app_runtime.h"
 #include "extensions/common/extension_messages.h"
+#include "extensions/common/switches.h"
 #include "ipc/ipc_message_macros.h"
 
 namespace app_runtime = extensions::core_api::app_runtime;
@@ -93,7 +92,7 @@ bool AppViewGuest::CompletePendingRequest(
 GuestViewBase* AppViewGuest::Create(content::BrowserContext* browser_context,
                                     int guest_instance_id) {
   if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
-      switches::kEnableAppView)) {
+      extensions::switches::kEnableAppView)) {
     return NULL;
   }
   return new AppViewGuest(browser_context, guest_instance_id);
@@ -102,6 +101,8 @@ GuestViewBase* AppViewGuest::Create(content::BrowserContext* browser_context,
 AppViewGuest::AppViewGuest(content::BrowserContext* browser_context,
                            int guest_instance_id)
     : GuestView<AppViewGuest>(browser_context, guest_instance_id),
+      app_view_guest_delegate_(
+          ExtensionsAPIClient::Get()->CreateAppViewGuestDelegate()),
       weak_ptr_factory_(this) {
 }
 
@@ -126,14 +127,8 @@ bool AppViewGuest::OnMessageReceived(const IPC::Message& message) {
 }
 
 bool AppViewGuest::HandleContextMenu(const content::ContextMenuParams& params) {
-  ContextMenuDelegate* menu_delegate =
-      ContextMenuDelegate::FromWebContents(guest_web_contents());
-  DCHECK(menu_delegate);
-
-  scoped_ptr<RenderViewContextMenu> menu =
-      menu_delegate->BuildMenu(guest_web_contents(), params);
-  menu_delegate->ShowMenu(menu.Pass());
-  return true;
+  return app_view_guest_delegate_->OnHandleContextMenu(guest_web_contents(),

[Yoyo Zhou, 2014/08/19 21:55:08]

I agree that this is better called just HandleContextMenu (no On)

+                                                       params);
 }
 
 const char* AppViewGuest::GetAPINamespace() {
@@ -157,11 +152,11 @@ void AppViewGuest::CreateWebContents(
     return;
   }
 
-  ExtensionService* service =
-      ExtensionSystem::Get(browser_context())->extension_service();
-  const Extension* guest_extension = service->GetExtensionById(app_id, false);
-  const Extension* embedder_extension =
-      service->GetExtensionById(embedder_extension_id, false);
+  ExtensionRegistry* registry = ExtensionRegistry::Get(browser_context());
+  const Extension* guest_extension =
+      registry->GetExtensionById(app_id, ExtensionRegistry::ENABLED);

[not at google - send to devlin, 2014/08/19 20:19:02]

Just registry->enabled_extensions()->GetByID(app_id), and ditto below.

+  const Extension* embedder_extension = registry->GetExtensionById(
+      embedder_extension_id, ExtensionRegistry::ENABLED);
 
   if (!guest_extension || !guest_extension->is_platform_app() ||
       !embedder_extension | !embedder_extension->is_platform_app()) {