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

Unified Diff: mojo/application_manager/application_manager.cc

Issue 489493004: Update view manager to support content handling. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: blah to the blizzah Created 6 years, 4 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: mojo/application_manager/application_manager.cc
diff --git a/mojo/application_manager/application_manager.cc b/mojo/application_manager/application_manager.cc
index c9b12dedc47f0d61044dcedf9e67e07d098a91b5..207b95001bf87830d86a865eca1bb2030d1e0df5 100644
--- a/mojo/application_manager/application_manager.cc
+++ b/mojo/application_manager/application_manager.cc
@@ -65,13 +65,16 @@ class ApplicationManager::LoadCallbacksImpl
return shell_handle.Pass();
}
- virtual void LoadWithContentHandler(const GURL& content_handler_url,
- URLResponsePtr content) OVERRIDE {
+ virtual void LoadWithContentHandler(
+ const GURL& content_handler_url,
+ URLResponsePtr url_response,
+ URLLoaderPtr url_loader) OVERRIDE {
if (manager_) {
manager_->LoadWithContentHandler(requested_url_,
requestor_url_,
content_handler_url,
- content.Pass(),
+ url_response.Pass(),
+ url_loader.Pass(),
service_provider_.Pass());
}
}
@@ -232,7 +235,8 @@ void ApplicationManager::LoadWithContentHandler(
const GURL& content_url,
const GURL& requestor_url,
const GURL& content_handler_url,
- URLResponsePtr content,
+ URLResponsePtr url_response,
+ URLLoaderPtr url_loader,
ServiceProviderPtr service_provider) {
ContentHandlerConnection* connection = NULL;
URLToContentHandlerMap::iterator iter =
@@ -243,8 +247,17 @@ void ApplicationManager::LoadWithContentHandler(
connection = new ContentHandlerConnection(this, content_handler_url);
url_to_content_handler_[content_handler_url] = connection;
}
+
+ InterfaceRequest<ServiceProvider> spir;
+ spir.Bind(service_provider.PassMessagePipe());
+
+ ContentHandlerResponsePtr content_handler_response(
+ ContentHandlerResponse::New());
+ content_handler_response->response = url_response.Pass();
+ content_handler_response->loader = url_loader.Pass();
+
connection->content_handler->OnConnect(
- content_url.spec(), content.Pass(), service_provider.Pass());
+ content_url.spec(), content_handler_response.Pass(), spir.Pass());
}
void ApplicationManager::SetLoaderForURL(scoped_ptr<ApplicationLoader> loader,
« no previous file with comments | « mojo/application_manager/application_manager.h ('k') | mojo/examples/content_handler_demo/content_handler_demo.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698