Update implementations of webrtc::SetSessionDescriptionObserver to us…

…e the new OnComplete signature.

Review URL: https://codereview.chromium.org/468783003

Cr-Commit-Position: refs/heads/master@{#291658}

content/renderer/media/mock_peer_connection_impl.cc

@@ -278,8 +278,8 @@ bool MockPeerConnectionImpl::GetStats(
           webrtc::StatsReport::kStatsValueNameFingerprint,
           "trackvalue"));
 
-  std::vector<webrtc::StatsReport> reports;
-  reports.push_back(report1);
+  webrtc::StatsReports reports;
+  reports.push_back(&report1);
 
   // If selector is given, we pass back one report.
   // If selector is not given, we pass back two.
@@ -291,7 +291,7 @@ bool MockPeerConnectionImpl::GetStats(
         webrtc::StatsReport::Value(
             webrtc::StatsReport::kStatsValueNameFingerprintAlgorithm,
             "somevalue"));
-    reports.push_back(report2);
+    reports.push_back(&report2);
   }
 
   // Note that the callback is synchronous, not asynchronous; it will

content/renderer/media/rtc_peer_connection_handler.cc

@@ -40,6 +40,9 @@
 #include "third_party/WebKit/public/platform/WebRTCVoidRequest.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
 
+using webrtc::StatsReport;
+using webrtc::StatsReports;
+
 namespace content {
 
 // Converter functions from libjingle types to WebKit types.
@@ -255,13 +258,12 @@ class StatsResponse : public webrtc::StatsObserver {
     TRACE_EVENT_ASYNC_BEGIN0("webrtc", "getStats_Native", this);
   }
 
-  virtual void OnComplete(
-      const std::vector<webrtc::StatsReport>& reports) OVERRIDE {
+  virtual void OnComplete(const StatsReports& reports) OVERRIDE {
     TRACE_EVENT0("webrtc", "StatsResponse::OnComplete")
-    for (std::vector<webrtc::StatsReport>::const_iterator it = reports.begin();
+    for (StatsReports::const_iterator it = reports.begin();
          it != reports.end(); ++it) {
-      if (it->values.size() > 0) {
-        AddReport(*it);
+      if ((*it)->values.size() > 0) {
+        AddReport(*(*it));
       }
     }
 
@@ -274,12 +276,11 @@ class StatsResponse : public webrtc::StatsObserver {
   }
 
  private:
-  void AddReport(const webrtc::StatsReport& report) {
+  void AddReport(const StatsReport& report) {
     int idx = response_->addReport(blink::WebString::fromUTF8(report.id),
                                    blink::WebString::fromUTF8(report.type),
                                    report.timestamp);
-    for (webrtc::StatsReport::Values::const_iterator value_it =
-         report.values.begin();
+    for (StatsReport::Values::const_iterator value_it = report.values.begin();
          value_it != report.values.end(); ++value_it) {
       AddStatistic(idx, value_it->display_name(), value_it->value);
     }
@@ -758,8 +759,7 @@ void RTCPeerConnectionHandler::getStats(LocalRTCStatsRequest* request) {
     if (!track) {
       DVLOG(1) << "GetStats: Track not found.";
       // TODO(hta): Consider how to get an error back.
-      std::vector<webrtc::StatsReport> no_reports;
-      observer->OnComplete(no_reports);
+      observer->OnComplete(StatsReports());
       return;
     }
   }
@@ -776,8 +776,7 @@ void RTCPeerConnectionHandler::GetStats(
   if (!native_peer_connection_->GetStats(observer, track, level)) {
     DVLOG(1) << "GetStats failed.";
     // TODO(hta): Consider how to get an error back.
-    std::vector<webrtc::StatsReport> no_reports;
-    observer->OnComplete(no_reports);
+    observer->OnComplete(StatsReports());
     return;
   }
 }