Add instrumentation to track the IceCandidatePairType and IceConnectionState

content/renderer/media/rtc_peer_connection_handler.h

 // 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.
 
 #ifndef CONTENT_RENDERER_MEDIA_RTC_PEER_CONNECTION_HANDLER_H_
 #define CONTENT_RENDERER_MEDIA_RTC_PEER_CONNECTION_HANDLER_H_
 
 #include <map>
 #include <string>
 
@@ skipping 186 matching lines @@
       const scoped_refptr<webrtc::MediaStreamInterface>& stream);
   void OnDataChannel(scoped_ptr<RtcDataChannelHandler> handler);
   void OnIceCandidate(const std::string& sdp, const std::string& sdp_mid,
       int sdp_mline_index, int component, int address_family);
 
  private:
   webrtc::SessionDescriptionInterface* CreateNativeSessionDescription(
       const std::string& sdp, const std::string& type,
       webrtc::SdpParseError* error);
 
+  // Report to UMA whether an IceConnectionState has occurred. It only records
+  // the first occurrence of a given state.
+  void ReportICEState(
+      webrtc::PeerConnectionInterface::IceConnectionState new_state);
+
+  // Reset UMA related members to the initial state. This is invoked at the
+  // constructor as well as after Ice Restart.
+  void ResetUMAStats();
+
   // Virtual to allow mocks to override.
   virtual scoped_refptr<base::SingleThreadTaskRunner> signaling_thread() const;
 
   void RunSynchronousClosureOnSignalingThread(const base::Closure& closure,
                                               const char* trace_event_name);
 
   base::ThreadChecker thread_checker_;
 
   // |client_| is a weak pointer, and is valid until stop() has returned.
   blink::WebRTCPeerConnectionHandlerClient* client_;
 
   // |dependency_factory_| is a raw pointer, and is valid for the lifetime of
   // RenderThreadImpl.
   PeerConnectionDependencyFactory* const dependency_factory_;
 
-  blink::WebFrame* frame_;
+  blink::WebFrame* frame_ = nullptr;
 
   ScopedVector<WebRtcMediaStreamAdapter> local_streams_;
 
   base::WeakPtr<PeerConnectionTracker> peer_connection_tracker_;
 
   MediaStreamTrackMetrics track_metrics_;
 
   // Counter for a UMA stat reported at destruction time.
-  int num_data_channels_created_;
+  int num_data_channels_created_ = 0;
 
   // Counter for number of IPv4 and IPv6 local candidates.
-  int num_local_candidates_ipv4_;
-  int num_local_candidates_ipv6_;
+  int num_local_candidates_ipv4_ = 0;
+  int num_local_candidates_ipv6_ = 0;
 
   // To make sure the observer is released after the native_peer_connection_,
   // it has to come first.
   scoped_refptr<Observer> peer_connection_observer_;
 
   // |native_peer_connection_| is the libjingle native PeerConnection object.
   scoped_refptr<webrtc::PeerConnectionInterface> native_peer_connection_;
 
   typedef std::map<webrtc::MediaStreamInterface*,
       content::RemoteMediaStreamImpl*> RemoteStreamMap;
   RemoteStreamMap remote_streams_;
   scoped_refptr<webrtc::UMAObserver> uma_observer_;
   base::TimeTicks ice_connection_checking_start_;
+
+  // Track which ICE Connection state that this PeerConnection has gone through.
+  bool ice_state_seen_[webrtc::PeerConnectionInterface::kIceConnectionMax] = {};
+
   base::WeakPtrFactory<RTCPeerConnectionHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RTCPeerConnectionHandler);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_RTC_PEER_CONNECTION_HANDLER_H_