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 87%
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..0405c9282fa7ae4b1116ba7ae06c1bbb4394903e 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,24 @@
 // 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_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,15 +91,20 @@ 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);
+  return new AppViewGuest(
+      ExtensionsAPIClient::Get()->CreateAppViewGuestDelegate(),
+      browser_context,
+      guest_instance_id);
 }
 
-AppViewGuest::AppViewGuest(content::BrowserContext* browser_context,
+AppViewGuest::AppViewGuest(AppViewGuestDelegate* app_view_guest_delegate,

[Xi Han, 2014/08/18 21:29:11]

The AppViewGuestDelegate doesn't need to be exposed as a parameter of the
constructor of AppViewGuest, you can initialize it within the constructor, like:

app_view_guest_delegate_.reset(ExtensionsAPIClient::Get()->CreateAppViewGuestDelegate());

+                           content::BrowserContext* browser_context,
                            int guest_instance_id)
     : GuestView<AppViewGuest>(browser_context, guest_instance_id),
+      app_view_guest_delegate_(app_view_guest_delegate),
       weak_ptr_factory_(this) {
 }
 
@@ -126,14 +129,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_->HandleContextMenu(guest_web_contents(),
+                                                     params);
 }
 
 const char* AppViewGuest::GetAPINamespace() {
@@ -157,11 +154,11 @@ void AppViewGuest::CreateWebContents(
     return;
   }
 
-  ExtensionService* service =
-      ExtensionSystem::Get(browser_context())->extension_service();
-  const Extension* guest_extension = service->GetExtensionById(app_id, false);
+  const Extension* guest_extension = app_view_guest_delegate_->GetExtensionById(

[Xi Han, 2014/08/18 21:29:11]

This part doesn't need to be moved to delegate, since the const Extensions*
embedder_extension can be obtained through ExtensionRegistry::GetExtensionById()
function, which exists in extensions. The code will look like:

ExtensionRegistry::Get()->GetExtensionById(id, ExtensionRegistry::ENABLED);

+      browser_context(), app_id, false);
   const Extension* embedder_extension =
-      service->GetExtensionById(embedder_extension_id, false);
+      app_view_guest_delegate_->GetExtensionById(
+          browser_context(), embedder_extension_id, false);
 
   if (!guest_extension || !guest_extension->is_platform_app() ||
       !embedder_extension | !embedder_extension->is_platform_app()) {