Chromium Code Reviews| Index: chrome/browser/guest_view/app_view/app_view_guest.cc |
| diff --git a/chrome/browser/guest_view/app_view/app_view_guest.cc b/chrome/browser/guest_view/app_view/app_view_guest.cc |
| index 2eeda8407768c07b5db0b644b466df59bff6fe9d..456f74ec0046c9efa726348e2bf092ee94019b80 100644 |
| --- a/chrome/browser/guest_view/app_view/app_view_guest.cc |
| +++ b/chrome/browser/guest_view/app_view/app_view_guest.cc |
| @@ -4,25 +4,173 @@ |
| #include "chrome/browser/guest_view/app_view/app_view_guest.h" |
| -// static |
| +#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/guest_view/guest_view_manager.h" |
| +#include "chrome/browser/profiles/profile.h" |
| +#include "chrome/browser/renderer_context_menu/context_menu_delegate.h" |
| +#include "chrome/browser/renderer_context_menu/render_view_context_menu.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/event_router.h" |
| +#include "extensions/browser/extension_host.h" |
| +#include "extensions/browser/extension_system.h" |
| +#include "extensions/browser/lazy_background_task_queue.h" |
| +#include "extensions/browser/view_type_utils.h" |
| +#include "extensions/common/api/app_runtime.h" |
| +#include "extensions/common/extension_messages.h" |
| +#include "ipc/ipc_message_macros.h" |
| + |
| +namespace app_runtime = extensions::core_api::app_runtime; |
| + |
| +using content::RenderFrameHost; |
| +using content::WebContents; |
| +using extensions::ExtensionHost; |
| + |
| +namespace { |
| + |
| +struct ResponseInfo { |
| + scoped_refptr<const extensions::Extension> guest_extension; |
| + base::WeakPtr<AppViewGuest> app_view_guest; |
| + GuestViewBase::WebContentsCreatedCallback callback; |
| + |
| + ResponseInfo(const extensions::Extension* guest_extension, |
| + const base::WeakPtr<AppViewGuest>& app_view_guest, |
| + const GuestViewBase::WebContentsCreatedCallback& callback) |
| + : guest_extension(guest_extension), |
| + app_view_guest(app_view_guest), |
| + callback(callback) {} |
| + |
| + ~ResponseInfo() {} |
| +}; |
| + |
| +typedef std::map<int, linked_ptr<ResponseInfo> > PendingResponseMap; |
| +static base::LazyInstance<PendingResponseMap> pending_response_map = |
| + LAZY_INSTANCE_INITIALIZER; |
| + |
| +} // namespace |
| + |
| +// static. |
| const char AppViewGuest::Type[] = "appview"; |
| +// static. |
| +bool AppViewGuest::CompletePendingRequest( |
| + const GURL& url, |
| + int guest_instance_id, |
| + content::BrowserContext* browser_context, |
| + const std::string& guest_extension_id) { |
| + PendingResponseMap* response_map = pending_response_map.Pointer(); |
| + PendingResponseMap::iterator it = response_map->find(guest_instance_id); |
| + if (it == response_map->end()) { |
| + // An app is sending invalid responses. We should probably kill it. |
|
lazyboy
2014/07/08 18:24:33
Why "probably" kill?
If we're not sure about killi
Fady Samuel
2014/07/08 20:23:46
Done.
|
| + return false; |
| + } |
| + |
| + linked_ptr<ResponseInfo> response_info = it->second; |
| + if (!response_info->app_view_guest || |
| + (response_info->guest_extension->id() != guest_extension_id)) { |
| + // An app is trying to respond to an <appview> that didn't initiate |
| + // communication with it. We should kill the app here. |
| + return false; |
| + } |
| + |
| + response_info->app_view_guest-> |
| + CompleteCreateWebContents(url, |
| + response_info->guest_extension, |
| + response_info->callback); |
| + |
| + response_map->erase(guest_instance_id); |
| + return true; |
| +} |
| + |
| AppViewGuest::AppViewGuest(content::BrowserContext* browser_context, |
| int guest_instance_id) |
| - : GuestView<AppViewGuest>(browser_context, guest_instance_id) { |
| + : GuestView<AppViewGuest>(browser_context, guest_instance_id), |
| + weak_ptr_factory_(this) { |
| } |
| AppViewGuest::~AppViewGuest() { |
| } |
| +extensions::WindowController* AppViewGuest::GetExtensionWindowController() |
| + const { |
| + return NULL; |
| +} |
| + |
| +content::WebContents* AppViewGuest::GetAssociatedWebContents() const { |
| + return guest_web_contents(); |
| +} |
| + |
| +bool AppViewGuest::OnMessageReceived(const IPC::Message& message) { |
| + bool handled = true; |
| + IPC_BEGIN_MESSAGE_MAP(AppViewGuest, message) |
| + IPC_MESSAGE_HANDLER(ExtensionHostMsg_Request, OnRequest) |
| + IPC_MESSAGE_UNHANDLED(handled = false) |
| + IPC_END_MESSAGE_MAP() |
| + return handled; |
| +} |
| + |
| +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; |
| +} |
| + |
| void AppViewGuest::CreateWebContents( |
| const std::string& embedder_extension_id, |
| int embedder_render_process_id, |
| const base::DictionaryValue& create_params, |
| const WebContentsCreatedCallback& callback) { |
| - // TODO(fsamuel): Create a WebContents with the appropriate SiteInstance here. |
| - // After the WebContents has been created, call the |callback|. |
| - // callback.Run(new_web_contents); |
| + std::string app_id; |
| + if (!create_params.GetString(appview::kAppID, &app_id)) { |
| + callback.Run(NULL); |
| + return; |
| + } |
| + |
| + Profile* profile = Profile::FromBrowserContext(browser_context()); |
| + ExtensionService* service = |
| + extensions::ExtensionSystem::Get(profile)->extension_service(); |
| + const extensions::Extension* guest_extension = |
| + service->GetExtensionById(app_id, false); |
| + const extensions::Extension* embedder_extension = |
| + service->GetExtensionById(embedder_extension_id, false); |
| + |
| + if (!guest_extension || !guest_extension->is_platform_app() || |
| + !embedder_extension | !embedder_extension->is_platform_app()) { |
| + callback.Run(NULL); |
| + return; |
| + } |
| + |
| + pending_response_map.Get().insert( |
| + std::make_pair(GetGuestInstanceID(), |
| + new ResponseInfo(guest_extension, |
| + weak_ptr_factory_.GetWeakPtr(), |
| + callback))); |
| + |
| + extensions::LazyBackgroundTaskQueue* queue = |
| + extensions::ExtensionSystem::Get(profile)->lazy_background_task_queue(); |
| + if (queue->ShouldEnqueueTask(profile, guest_extension)) { |
| + queue->AddPendingTask(profile, |
| + guest_extension->id(), |
| + base::Bind(&AppViewGuest::LaunchAppAndFireEvent, |
| + weak_ptr_factory_.GetWeakPtr(), |
| + callback)); |
| + return; |
| + } |
| + |
| + extensions::ProcessManager* process_manager = |
| + extensions::ExtensionSystem::Get(profile)->process_manager(); |
| + ExtensionHost* host = |
| + process_manager->GetBackgroundHostForExtension(guest_extension->id()); |
| + DCHECK(host); |
| + LaunchAppAndFireEvent(callback, host); |
| } |
| void AppViewGuest::DidAttachToEmbedder() { |
| @@ -30,5 +178,58 @@ void AppViewGuest::DidAttachToEmbedder() { |
| // element. This means that the host element knows how to route input |
| // events to the guest, and the guest knows how to get frames to the |
| // embedder. |
| - // TODO(fsamuel): Perform the initial navigation here. |
| + guest_web_contents()->GetController().LoadURL( |
| + url_, content::Referrer(), content::PAGE_TRANSITION_LINK, std::string()); |
| +} |
| + |
| +void AppViewGuest::DidInitialize() { |
| + extension_function_dispatcher_.reset( |
| + new extensions::ExtensionFunctionDispatcher(browser_context(), this)); |
| } |
| + |
| +void AppViewGuest::OnRequest(const ExtensionHostMsg_Request_Params& params) { |
| + extension_function_dispatcher_->Dispatch( |
| + params, guest_web_contents()->GetRenderViewHost()); |
| +} |
| + |
| +void AppViewGuest::CompleteCreateWebContents( |
| + const GURL& url, |
| + const extensions::Extension* guest_extension, |
| + const WebContentsCreatedCallback& callback) { |
| + if (!url.is_valid()) { |
| + callback.Run(NULL); |
| + return; |
| + } |
| + url_ = url; |
| + guest_extension_id_ = guest_extension->id(); |
| + |
| + WebContents::CreateParams params( |
| + browser_context(), |
| + content::SiteInstance::CreateForURL(browser_context(), |
| + guest_extension->url())); |
| + params.guest_delegate = this; |
| + callback.Run(WebContents::Create(params)); |
| +} |
| + |
| +void AppViewGuest::LaunchAppAndFireEvent( |
| + const WebContentsCreatedCallback& callback, |
| + ExtensionHost* extension_host) { |
| + Profile* profile = Profile::FromBrowserContext(browser_context()); |
| + extensions::ExtensionSystem* system = |
| + extensions::ExtensionSystem::Get(browser_context()); |
| + bool has_event_listener = system->event_router()->ExtensionHasEventListener( |
| + extension_host->extension()->id(), |
| + app_runtime::OnAppEmbeddingRequest::kEventName); |
| + if (!has_event_listener) { |
| + callback.Run(NULL); |
| + return; |
| + } |
| + |
| + scoped_ptr<base::DictionaryValue> embed_app_data(new base::DictionaryValue()); |
| + embed_app_data->SetInteger(appview::kGuestInstanceID, GetGuestInstanceID()); |
| + embed_app_data->SetString(appview::kEmbedderExtensionID, |
| + embedder_extension_id()); |
| + extensions::AppRuntimeEventRouter::DispatchOnAppEmbeddingRequestEvent( |
| + profile, embed_app_data.Pass(), extension_host->extension()); |
| +} |
| + |