Stop printing LOG(ERROR) when NFC service is unknown

chromeos/dbus/nfc_property_set.h

Index: chromeos/dbus/nfc_property_set.h
diff --git a/chromeos/dbus/nfc_property_set.h b/chromeos/dbus/nfc_property_set.h
index ee18528280b8074657f7f2403b1ed4958b77104b..4b5016ad759ea65b7044aa20b7debfdd37925962 100644
--- a/chromeos/dbus/nfc_property_set.h
+++ b/chromeos/dbus/nfc_property_set.h
@@ -45,10 +45,14 @@ class NfcPropertySet : public dbus::PropertySet {
   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);
 };