New issue
Advanced search Search tips

Issue 629436 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 627816



Sign in to add a comment

RTCPeerConnection.getStats: RTCCertificateStats is missing

Project Member Reported by hbos@chromium.org, Jul 19 2016

Issue description

RTCCertificateStats (http://rawgit.com/w3c/webrtc-stats/master/webrtc-stats.html#certificatestats-dict*) is missing.

As part of updating getStats ( bug 627816 ) these stats needs to be made available in WebRTC and surfaced to Blink.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/external/webrtc.git/+/e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4

commit e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4
Author: hbos <hbos@webrtc.org>
Date: Thu Aug 25 10:52:38 2016

Refactor certificate stats collection, added SSLCertificateStats.

The code that extracts certificate stats from an SSLCertificate and its
certificate chain is moved into SSLCertificate::GetStats. The stats
collector code loops through the resulting SSLCertificateStats and
creates the StatsReports for those stats.

This will allow the new stats collector to reuse GetStats in a future
CL.

BUG= chromium:627816 ,  chromium:629436 

Review-Url: https://codereview.webrtc.org/2259283002
Cr-Commit-Position: refs/heads/master@{#13917}

[modify] https://crrev.com/e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4/webrtc/api/statscollector.cc
[modify] https://crrev.com/e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4/webrtc/api/statscollector.h
[modify] https://crrev.com/e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4/webrtc/base/sslidentity.cc
[modify] https://crrev.com/e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4/webrtc/base/sslidentity.h
[modify] https://crrev.com/e29352bb34de60bd0a56d4ce46c2ce35ac2b27b4/webrtc/base/sslidentity_unittest.cc

Comment 3 by hbos@chromium.org, Oct 3 2016

Status: Verified (was: Started)

Sign in to add a comment