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

Unified Diff: content/public/test/mock_render_process_host.cc

Issue 2766263009: Convert content ConnectionFilter to OnBindInterface (Closed)
Patch Set: . Created 3 years, 8 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: content/public/test/mock_render_process_host.cc
diff --git a/content/public/test/mock_render_process_host.cc b/content/public/test/mock_render_process_host.cc
index c39d7a1caea00ab77071546ac2afa240d28f8cb6..232282d5086962ac833b1e1a2b3b3cfaad7e5414 100644
--- a/content/public/test/mock_render_process_host.cc
+++ b/content/public/test/mock_render_process_host.cc
@@ -99,7 +99,6 @@ void MockRenderProcessHost::SimulateCrash() {
bool MockRenderProcessHost::Init() {
has_connection_ = true;
- remote_interfaces_.reset(new service_manager::InterfaceProvider);
return true;
}
@@ -264,9 +263,11 @@ base::TimeDelta MockRenderProcessHost::GetChildProcessIdleTime() const {
return base::TimeDelta::FromMilliseconds(0);
}
-service_manager::InterfaceProvider*
-MockRenderProcessHost::GetRemoteInterfaces() {
- return remote_interfaces_.get();
+void MockRenderProcessHost::BindInterface(
+ const std::string& interface_name,
+ mojo::ScopedMessagePipeHandle interface_pipe) {
+ if (binder_overrides_.count(interface_name) > 0)
+ binder_overrides_[interface_name].Run(std::move(interface_pipe));
}
std::unique_ptr<base::SharedPersistentMemoryAllocator>
@@ -379,6 +380,12 @@ bool MockRenderProcessHost::OnMessageReceived(const IPC::Message& msg) {
void MockRenderProcessHost::OnChannelConnected(int32_t peer_pid) {}
+void MockRenderProcessHost::OverrideBinderForTesting(
+ const std::string& interface_name,
+ const InterfaceBinder& binder) {
+ binder_overrides_[interface_name] = binder;
+}
+
MockRenderProcessHostFactory::MockRenderProcessHostFactory() {}
MockRenderProcessHostFactory::~MockRenderProcessHostFactory() {
« no previous file with comments | « content/public/test/mock_render_process_host.h ('k') | content/renderer/mus/render_widget_window_tree_client_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698