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

Unified Diff: services/log/log_impl.cc

Issue 1975993002: Change InterfaceFactory<I>::Create() to take a ConnectionContext instead of an ApplicationConnectio… (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 7 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 | « services/log/log_impl.h ('k') | services/log/log_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/log/log_impl.cc
diff --git a/services/log/log_impl.cc b/services/log/log_impl.cc
index b02459fa500fc6633dd75db08d9d69f106d345df..1bf505e0a8fc153e8be586564e9b02f118422aa2 100644
--- a/services/log/log_impl.cc
+++ b/services/log/log_impl.cc
@@ -8,7 +8,7 @@
#include "base/logging.h"
#include "base/strings/stringprintf.h"
-#include "mojo/public/cpp/application/application_connection.h"
+#include "mojo/public/cpp/application/connection_context.h"
#include "mojo/services/log/interfaces/entry.mojom.h"
namespace mojo {
@@ -47,13 +47,12 @@ LogImpl::LogImpl(const std::string& remote_url,
LogImpl::~LogImpl() {}
// static
-void LogImpl::Create(ApplicationConnection* connection,
+void LogImpl::Create(const ConnectionContext& connection_context,
InterfaceRequest<Log> request,
PrintLogMessageFunction print_log_message_function) {
- DCHECK(connection);
DCHECK(print_log_message_function);
- const std::string& remote_url = connection->GetRemoteApplicationURL();
+ const std::string& remote_url = connection_context.remote_url;
if (remote_url.empty()) {
LOG(ERROR) << "No remote URL.";
return;
« no previous file with comments | « services/log/log_impl.h ('k') | services/log/log_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698