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

Unified Diff: apps/custom_launcher_page_contents.cc

Issue 1169223002: [Extensions] Clean up the handling of ExtensionHostMsg_Request (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Latest master Created 5 years, 6 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: apps/custom_launcher_page_contents.cc
diff --git a/apps/custom_launcher_page_contents.cc b/apps/custom_launcher_page_contents.cc
index 20744b33c26ecc2d517ec080c9b4a4d8836f14e0..8888993aac5c39e380dc2df0ecc7ff1d34ae93ab 100644
--- a/apps/custom_launcher_page_contents.cc
+++ b/apps/custom_launcher_page_contents.cc
@@ -30,14 +30,10 @@ CustomLauncherPageContents::~CustomLauncherPageContents() {
void CustomLauncherPageContents::Initialize(content::BrowserContext* context,
const GURL& url) {
- extension_function_dispatcher_.reset(
- new extensions::ExtensionFunctionDispatcher(context, this));
-
web_contents_.reset(
content::WebContents::Create(content::WebContents::CreateParams(
context, content::SiteInstance::CreateForURL(context, url))));
- Observe(web_contents());
web_contents_->GetMutableRendererPrefs()
->browser_handles_all_top_level_requests = true;
web_contents_->GetRenderViewHost()->SyncRendererPrefs();
@@ -135,30 +131,4 @@ bool CustomLauncherPageContents::CheckMediaAccessPermission(
return helper_->CheckMediaAccessPermission(security_origin, type);
}
-bool CustomLauncherPageContents::OnMessageReceived(
- const IPC::Message& message) {
- bool handled = true;
- IPC_BEGIN_MESSAGE_MAP(CustomLauncherPageContents, message)
- IPC_MESSAGE_HANDLER(ExtensionHostMsg_Request, OnRequest)
- IPC_MESSAGE_UNHANDLED(handled = false)
- IPC_END_MESSAGE_MAP()
- return handled;
-}
-
-extensions::WindowController*
-CustomLauncherPageContents::GetExtensionWindowController() const {
- return NULL;
-}
-
-content::WebContents* CustomLauncherPageContents::GetAssociatedWebContents()
- const {
- return web_contents();
-}
-
-void CustomLauncherPageContents::OnRequest(
- const ExtensionHostMsg_Request_Params& params) {
- extension_function_dispatcher_->Dispatch(params,
- web_contents_->GetRenderViewHost());
-}
-
} // namespace apps

Powered by Google App Engine
This is Rietveld 408576698