Renamed local BindInterface since it clashed with template

There is a template named BindInterface in
content/public/common/bind_interface_helpers.h so to avoid
compilation failures when that header is included directly
or indirectly (as in jumbo builds), it's better to call
local functions something else than BindInterface.

This renames BindInterface -> BindNamedInterface.

Bug: 746953
Change-Id: Ib595232b8c6c9ae58b5321ff58ad1c2927787631
Reviewed-on: https://chromium-review.googlesource.com/822433Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#523721}

content/renderer/mojo/blink_interface_provider_impl.cc

@@ -17,9 +17,9 @@ namespace content {
 
 namespace {
 
-void BindInterface(base::WeakPtr<service_manager::Connector> connector,
-                   const std::string& name,
-                   mojo::ScopedMessagePipeHandle handle) {
+void BindNamedInterface(base::WeakPtr<service_manager::Connector> connector,
+                        const std::string& name,
+                        mojo::ScopedMessagePipeHandle handle) {
   if (!connector)
     return;
 
@@ -43,7 +43,7 @@ void BlinkInterfaceProviderImpl::GetInterface(
     mojo::ScopedMessagePipeHandle handle) {
   // Construct a closure that can safely be passed across threads if necessary.
   base::OnceClosure closure = base::BindOnce(
-      &BindInterface, connector_, std::string(name), std::move(handle));
+      &BindNamedInterface, connector_, std::string(name), std::move(handle));
 
   if (main_thread_task_runner_->BelongsToCurrentThread()) {
     std::move(closure).Run();