Convert internal representation of Srtp cryptos from string to int.

talk/app/webrtc/statscollector.cc

 /*
  * libjingle
  * Copyright 2012 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 712 matching lines @@
       channel_report->AddInt(StatsReport::kStatsValueNameComponent,
                              channel_iter.component);
       if (local_cert_report_id.get()) {
         channel_report->AddId(StatsReport::kStatsValueNameLocalCertificateId,
                               local_cert_report_id);
       }
       if (remote_cert_report_id.get()) {
         channel_report->AddId(StatsReport::kStatsValueNameRemoteCertificateId,
                               remote_cert_report_id);
       }
-      const std::string& srtp_cipher = channel_iter.srtp_cipher;
-      if (!srtp_cipher.empty()) {
+      int srtp_cipher = channel_iter.srtp_cipher;

[pthatcher1, 2015/11/11 19:59:40]

Should we rename srtp_chiper to srtp_crypto_suite?  

I mean the local variable, not the member variable.

[guoweis_webrtc, 2015/11/17 01:21:15]

Done.

+      if (srtp_cipher && rtc::SrtpCryptoSuiteToName(srtp_cipher).length()) {
         channel_report->AddString(StatsReport::kStatsValueNameSrtpCipher,
-                                  srtp_cipher);
+                                  rtc::SrtpCryptoSuiteToName(srtp_cipher));
       }
       int ssl_cipher = channel_iter.ssl_cipher;

[pthatcher1, 2015/11/11 19:59:40]

And should we rename ssl_cipher to ssl_cipher_suite?

[guoweis_webrtc, 2015/11/17 01:21:15]

Done.

       if (ssl_cipher &&
-          rtc::SSLStreamAdapter::GetSslCipherSuiteName(ssl_cipher).length()) {
+          rtc::SSLStreamAdapter::SslCipherSuiteToName(ssl_cipher).length()) {
         channel_report->AddString(
             StatsReport::kStatsValueNameDtlsCipher,
-            rtc::SSLStreamAdapter::GetSslCipherSuiteName(ssl_cipher));
+            rtc::SSLStreamAdapter::SslCipherSuiteToName(ssl_cipher));
       }
 
       int connection_id = 0;
       for (const cricket::ConnectionInfo& info :
                channel_iter.connection_infos) {
         StatsReport* connection_report = AddConnectionInfoReport(
             transport_iter.first, channel_iter.component, connection_id++,
             channel_report->id(), info);
         if (info.best_connection) {
           channel_report->AddId(
@@ skipping 182 matching lines @@
     StatsReport* report = entry.second;
     report->set_timestamp(stats_gathering_started_);
   }
 }
 
 void StatsCollector::ClearUpdateStatsCacheForTest() {
   stats_gathering_started_ = 0;
 }
 
 }  // namespace webrtc