Make StunMessage::attrs_ a vector of unique_ptrs instead of a pointer to a vector of pointers.

webrtc/p2p/base/stun.h

Index: webrtc/p2p/base/stun.h
diff --git a/webrtc/p2p/base/stun.h b/webrtc/p2p/base/stun.h
index 028787c72aa9fd80c985bf156182803b221edcce..532fe5bec9e947f127972bc2f4f27c0b4ca3563d 100644
--- a/webrtc/p2p/base/stun.h
+++ b/webrtc/p2p/base/stun.h
@@ -132,7 +132,7 @@ class StunUInt16ListAttribute;
 class StunMessage {
  public:
   StunMessage();
-  virtual ~StunMessage();
+  virtual ~StunMessage() = default;
 
   int type() const { return type_; }
   size_t length() const { return length_; }
@@ -159,6 +159,7 @@ class StunMessage {
   const StunUInt16ListAttribute* GetUnknownAttributes() const;
 
   // Takes ownership of the specified attribute and adds it to the message.
+  // TODO(zstein): Take a unique_ptr instead of a raw pointer.
   void AddAttribute(StunAttribute* attr);
 
   // Validates that a raw STUN message has a correct MESSAGE-INTEGRITY value.
@@ -199,7 +200,7 @@ class StunMessage {
   uint16_t type_;
   uint16_t length_;
   std::string transaction_id_;
-  std::vector<StunAttribute*>* attrs_;
+  std::vector<std::unique_ptr<StunAttribute>> attrs_;
 };
 
 // Base class for all STUN/TURN attributes.