Create a RegisterViewAssociate method in ViewManager

services/ui/view_manager/view_associate_table.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "services/ui/view_manager/view_associate_table.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "mojo/public/cpp/application/connect.h"
 #include "mojo/services/ui/views/cpp/formatting.h"
 
 namespace view_manager {
 
 template <typename T>
 static bool Contains(const mojo::Array<T>& array, const T& value) {
   return std::find(array.storage().cbegin(), array.storage().cend(), value) !=
          array.storage().cend();
 }
 
 ViewAssociateTable::ViewAssociateTable() {}
 
 ViewAssociateTable::~ViewAssociateTable() {}
 
-void ViewAssociateTable::ConnectAssociates(
-    mojo::ApplicationImpl* app_impl,
+void ViewAssociateTable::RegisterViewAssociate(
     mojo::ui::ViewInspector* inspector,
-    const std::vector<std::string>& urls,
-    const AssociateConnectionErrorCallback& connection_error_callback) {
-  DCHECK(app_impl);
+    mojo::ui::ViewAssociatePtr associate,
+    const AssociateConnectionErrorCallback& connection_error_callback,
+    const mojo::String& label) {
   DCHECK(inspector);
+  DCHECK(associate.is_bound());
 
-  for (auto& url : urls) {
-    DVLOG(1) << "Connecting to view associate: url=" << url;
-    associates_.emplace_back(new AssociateData(url, inspector));
-    AssociateData* data = associates_.back().get();
+  associates_.emplace_back(
+      new AssociateData(label, associate.Pass(), inspector));
+  AssociateData* data = associates_.back().get();
 
-    mojo::ConnectToService(app_impl->shell(), url, GetProxy(&data->associate));
+  if (connection_error_callback.is_null()) {
+    // Set it to use our error handler.
     data->associate.set_connection_error_handler(
-        base::Bind(connection_error_callback, url));
+        base::Bind(&ViewAssociateTable::OnAssociateConnectionError,
+                   base::Unretained(this), data));
+  }
 
-    mojo::ui::ViewInspectorPtr inspector;
-    data->inspector_binding.Bind(GetProxy(&inspector));
-    data->associate->Connect(
-        inspector.Pass(),
-        base::Bind(&ViewAssociateTable::OnConnected, base::Unretained(this),
-                   pending_connection_count_));
+  // Connect the associate to our view inspector.
+  mojo::ui::ViewInspectorPtr inspector_ptr;
+  data->inspector_binding.Bind(GetProxy(&inspector_ptr));
+  data->associate->Connect(
+      inspector_ptr.Pass(),
+      base::Bind(&ViewAssociateTable::OnConnected, base::Unretained(this),
+                 pending_connection_count_));
 
-    pending_connection_count_++;
-  }
+  // Wait for the associate to connect to our view inspector.
+  pending_connection_count_++;
+}
+
+void ViewAssociateTable::RegisterViewAssociateWatcher(
+    mojo::ui::ViewAssociateWatcherPtr view_associate_watcher) {
+  associate_watchers_.push_back(view_associate_watcher.Pass());

[jeffbrown, 2016/05/17 00:44:49]

prefer emplace_back where possible

 }
 
 void ViewAssociateTable::ConnectToViewService(
     mojo::ui::ViewTokenPtr view_token,
     const mojo::String& service_name,
     mojo::ScopedMessagePipeHandle client_handle) {
   if (pending_connection_count_) {
     deferred_work_.push_back(
         base::Bind(&ViewAssociateTable::ConnectToViewService,
                    base::Unretained(this), base::Passed(view_token.Pass()),
                    service_name, base::Passed(client_handle.Pass())));
     return;
   }
 
   for (auto& data : associates_) {
     DCHECK(data->info);
     if (Contains(data->info->view_service_names, service_name)) {
       DVLOG(2) << "Connecting to view service: view_token=" << view_token
                << ", service_name=" << service_name
-               << ", associate_url=" << data->url;
+               << ", associate_label=" << data->label;
       DCHECK(data->associate);
       data->associate->ConnectToViewService(view_token.Pass(), service_name,
                                             client_handle.Pass());
       return;
     }
   }
 
   DVLOG(2) << "Requested view service not available: view_token=" << view_token
            << ", service_name=" << service_name;
   // Allow pipe to be closed as an indication of failure.
 }
 
+void ViewAssociateTable::OnAssociateConnectionError(
+    AssociateData* associate_data) {
+  // Remove associate from our list.
+  std::string label;
+  for (auto it = associates_.begin(); it != associates_.end(); it++) {
+    AssociateData* data = it->get();
+    if (associate_data == data) {
+      DVLOG(2) << "ViewAssociate disconnected, removing from table"
+               << ", associate_label=" << data->label;
+      label = data->label;
+      associates_.erase(it);
+      break;
+    }
+  }
+  for (auto& view_associate_watcher : associate_watchers_) {
+    // TODO(mikejurka) do we even need this callback?
+    mojo::Callback<void()> null_callback;
+    view_associate_watcher->OnViewAssociateConnectionError(label,
+                                                           null_callback);
+  }
+}
+
 void ViewAssociateTable::ConnectToViewTreeService(
     mojo::ui::ViewTreeTokenPtr view_tree_token,
     const mojo::String& service_name,
     mojo::ScopedMessagePipeHandle client_handle) {
   if (pending_connection_count_) {
     deferred_work_.push_back(
         base::Bind(&ViewAssociateTable::ConnectToViewTreeService,
                    base::Unretained(this), base::Passed(view_tree_token.Pass()),
                    service_name, base::Passed(client_handle.Pass())));
     return;
   }
 
   for (auto& data : associates_) {
     DCHECK(data->info);
     if (Contains(data->info->view_tree_service_names, service_name)) {
       DVLOG(2) << "Connecting to view tree service: view_tree_token="
                << view_tree_token << ", service_name=" << service_name
-               << ", associate_url=" << data->url;
+               << ", associate_label=" << data->label;
       DCHECK(data->associate);
       data->associate->ConnectToViewTreeService(
           view_tree_token.Pass(), service_name, client_handle.Pass());
       return;
     }
   }
 
   DVLOG(2) << "Requested view tree service not available: view_tree_token="
            << view_tree_token << ", service_name=" << service_name;
   // Allow pipe to be closed as an indication of failure.
 }
 
 void ViewAssociateTable::OnConnected(uint32_t index,
                                      mojo::ui::ViewAssociateInfoPtr info) {
   DCHECK(info);
   DCHECK(pending_connection_count_);
   DCHECK(!associates_[index]->info);
 
-  DVLOG(1) << "Connected to view associate: url=" << associates_[index]->url
+  DVLOG(1) << "Connected to view associate: label=" << associates_[index]->label
            << ", info=" << info;
   associates_[index]->info = info.Pass();
 
   pending_connection_count_--;
   if (!pending_connection_count_)
     CompleteDeferredWork();
 }
 
 void ViewAssociateTable::CompleteDeferredWork() {
   DCHECK(!pending_connection_count_);
 
   for (auto& work : deferred_work_)
     work.Run();
   deferred_work_.clear();
 }
 
+size_t ViewAssociateTable::associate_count() {
+  return associates_.size();
+}
+
 ViewAssociateTable::AssociateData::AssociateData(
-    const std::string& url,
+    const std::string& label,
+    mojo::ui::ViewAssociatePtr associate,
     mojo::ui::ViewInspector* inspector)
-    : url(url), inspector_binding(inspector) {}
+    : label(label), associate(associate.Pass()), inspector_binding(inspector) {}
 
 ViewAssociateTable::AssociateData::~AssociateData() {}
 
 }  // namespace view_manager