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

Unified Diff: mojo/apps/js/application_delegate_impl.h

Issue 665743003: Mojo JS Bindings: Simplify sharing services for content-provided JS applications (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « no previous file | mojo/apps/js/application_delegate_impl.cc » ('j') | mojo/apps/js/content_handler_impl.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/apps/js/application_delegate_impl.h
diff --git a/mojo/apps/js/application_delegate_impl.h b/mojo/apps/js/application_delegate_impl.h
index 81a481cd356bc3a76be815561d7a17358c8693af..c316655153130c57baae3876d175790985f9f1de 100644
--- a/mojo/apps/js/application_delegate_impl.h
+++ b/mojo/apps/js/application_delegate_impl.h
@@ -37,9 +37,9 @@ class ApplicationDelegateImpl : public ApplicationDelegate {
// Remove app from the AppVector; destroys the app.
void QuitJSApp(JSApp *app);
- void ConnectToService(ScopedMessagePipeHandle pipe_handle,
- const std::string& application_url,
- const std::string& interface_name);
+ // Use the shell to connect to a ServiceProvider for application_url.
+ void ConnectToApplication(const std::string& application_url,
+ ScopedMessagePipeHandle pipe_handle);
protected:
// ApplicationDelegate:
@@ -47,7 +47,7 @@ class ApplicationDelegateImpl : public ApplicationDelegate {
private:
typedef ScopedVector<JSApp> AppVector;
- ApplicationImpl* application_impl_;
+ ApplicationImpl* application_impl_; // Owns the shell used by all JSApps.
AppVector app_vector_;
};
« no previous file with comments | « no previous file | mojo/apps/js/application_delegate_impl.cc » ('j') | mojo/apps/js/content_handler_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698