[Fuchsia] Don't bind ComponentControllerImpl twice.

Previously ComponentControllerImpl::BindToRequest() was called twice,
so the Context process was crashing.

Bug: 850743
Change-Id: I7fe63558afcf586beb1380ce0d108458e1532f9b
Reviewed-on: https://chromium-review.googlesource.com/1147551
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577646}

webrunner/app/component_controller_impl.h

@@ -59,8 +59,6 @@ class ComponentControllerImpl : public fuchsia::sys::ComponentController,
       override;
 
  private:
-  friend class WebContentRunner;
-
   ComponentControllerImpl(WebContentRunner* runner);
 
   // Binds |this| to a Runner::StartComponent() call. Returns false on failure

webrunner/app/web_content_runner.cc

@@ -31,8 +31,6 @@ void WebContentRunner::StartComponent(
   auto controller = ComponentControllerImpl::CreateForRequest(
       this, std::move(package), std::move(startup_info),
       std::move(controller_request));
-  controller->BindToRequest(std::move(package), std::move(startup_info),
-                            std::move(controller_request));
   controllers_.insert(std::move(controller));
 }