Metrics: cleanup and compilation fix

- Removed duplicated LogSdpAttribute entry
- Fixed compilation issues

Change-Id: I30193baf2aaeff7b6e8e2af142292fe221711478
CRs-Fixed: 2418997
diff --git a/common/metrics.cc b/common/metrics.cc
index 02a7247..cd58d0e 100644
--- a/common/metrics.cc
+++ b/common/metrics.cc
@@ -568,13 +568,6 @@
   pimpl_->wake_event_queue_->Clear();
   pimpl_->scan_event_queue_->Clear();
 }
-void LogSdpAttribute(const RawAddress& address, uint16_t protocol_uuid,
-                     uint16_t attribute_id, size_t attribute_size,
-                     const char* attribute_value) {
-  // TODO (b/124229503) This is a no-op placeholder to reduce the number of
-  // reverts. This function is properly defined in
-  // 9d9df72cf3e2215af34c8be1e1c935b097853b85
-}
 
 void LogLinkLayerConnectionEvent(const RawAddress* address,
                                  uint32_t connection_handle,
diff --git a/common/metrics.h b/common/metrics.h
index 8fe5e1c..a39fd46 100644
--- a/common/metrics.h
+++ b/common/metrics.h
@@ -279,13 +279,6 @@
 };
 
 /**
- * no-op placeholder
- */
-void LogSdpAttribute(const RawAddress& address, uint16_t protocol_uuid,
-                     uint16_t attribute_id, size_t attribute_size,
-                     const char* attribute_value);
-
-/**
  * Unknown connection handle for metrics purpose
  */
 static const uint32_t kUnknownConnectionHandle = 0xFFFF;
diff --git a/profile/sdp/Android.bp b/profile/sdp/Android.bp
index 9254337..4dca980 100644
--- a/profile/sdp/Android.bp
+++ b/profile/sdp/Android.bp
@@ -35,6 +35,7 @@
     "libgmock",
     "sdp_service",
     "lib-bt-packets",
+    "liblog",
     "libbluetooth-types",
   ],
 }