dbus: don't fail when reconnecting object signals

dbus/object_proxy.cc

 // Copyright (c) 2012 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 "dbus/bus.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
@@ skipping 263 matching lines @@
   delete data;
 }
 
 void ObjectProxy::ConnectToSignalInternal(
     const std::string& interface_name,
     const std::string& signal_name,
     SignalCallback signal_callback,
     OnConnectedCallback on_connected_callback) {
   bus_->AssertOnDBusThread();
 
-  // Check if the object is already connected to the signal.
   const std::string absolute_signal_name =
       GetAbsoluteSignalName(interface_name, signal_name);
-  if (method_table_.find(absolute_signal_name) != method_table_.end()) {
-    LOG(ERROR) << "The object proxy is already connected to "
-               << absolute_signal_name;
-    return;
-  }
 
   // Will become true, if everything is successful.
   bool success = false;
 
   if (bus_->Connect() && bus_->SetUpAsyncOperations()) {
     // We should add the filter only once. Otherwise, HandleMessage() will
     // be called more than once.
     if (!filter_added_) {
       if (bus_->AddFilterFunction(&ObjectProxy::HandleMessageThunk, this)) {
         filter_added_ = true;
@@ skipping 127 matching lines @@
 void ObjectProxy::LogMethodCallFailure(
     const base::StringPiece& error_name,
     const base::StringPiece& error_message) const {
   if (ignore_service_unknown_errors_ && error_name == kErrorServiceUnknown)
     return;
   LOG(ERROR) << "Failed to call method: " << error_name
              << ": " << error_message;
 }
 
 }  // namespace dbus