Stop printing LOG(ERROR) when NFC service is unknown

chromeos/dbus/nfc_property_set.h

 // Copyright 2013 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.
 
 #ifndef CHROMEOS_DBUS_NFC_PROPERTY_SET_H_
 #define CHROMEOS_DBUS_NFC_PROPERTY_SET_H_
 
 #include <string>
 
 #include "base/callback.h"
@@ skipping 27 matching lines @@
   void Get(dbus::PropertyBase* property, GetCallback callback) override;
   void GetAll() override;
   void OnGetAll(dbus::Response* response) override;
   void Set(dbus::PropertyBase* property, SetCallback callback) override;
   void ChangedReceived(dbus::Signal* signal) override;
 
  protected:
   const base::Closure& on_get_all_callback() { return on_get_all_callback_; }
 
  private:
+  void OnGetAllError(dbus::ErrorResponse* response);
+
   // Optional callback used to notify clients when all properties were received
   // after a call to GetAll.
   base::Closure on_get_all_callback_;
 
+  base::WeakPtrFactory<NfcPropertySet> weak_ptr_factory_;

[stevenjb, 2016/03/22 16:17:51]

Instead of adding a second WeakPtrFactory to protect the callback, call 
weak_ptr_factory_.InvalidateWeakPtrs() in the NfcPropertySet destructor. (You'll
need to expose that in dbus::PropertySet).

[hashimoto, 2016/03/23 03:27:03]

This WeakPtrFactory is needed to make a callback of OnGetAllError which requires
WeakPtr<NfcPropertySet>, instead of WeakPtr<dbus::PropertySet>.

[stevenjb, 2016/03/23 04:03:33]

Ugh. I see. Fair enough.

+
   DISALLOW_COPY_AND_ASSIGN(NfcPropertySet);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_NFC_PROPERTY_SET_H_