RTCCertificate::Expires() and ::HasExpired() implemented

webrtc/base/rtccertificate.h

Index: webrtc/base/rtccertificate.h
diff --git a/webrtc/base/rtccertificate.h b/webrtc/base/rtccertificate.h
index d238938ae185786b55a43c72eca05f81b627ec79..3a403ff8c5db4e15d44d821f3da1fbd6a7d45379 100644
--- a/webrtc/base/rtccertificate.h
+++ b/webrtc/base/rtccertificate.h
@@ -27,7 +27,8 @@ class RTCCertificate : public RefCountInterface {
   // Takes ownership of |identity|.
   static scoped_refptr<RTCCertificate> Create(scoped_ptr<SSLIdentity> identity);
 
-  uint64_t expires_timestamp_ns() const;
+  // Returns the expiration time in ms relative to epoch, 1970-01-01T00:00:00Z.
+  uint64_t Expires() const;
   bool HasExpired() const;

[hta-webrtc, 2015/12/03 13:49:07]

What's the argument for using CamelCase for these functions and
lower_case_dashes for all the others?

[hbos, 2015/12/03 16:03:48]

All functions should be UpperCamel except tiny and "cheap" ones like getters,
https://google.github.io/styleguide/cppguide.html#Function_Names.

Getting the Expires() time involves lower level crypto stuff and as it happens
does some parsing so I'm not making them lower_snake like identity() and
ssl_certificate().

   const SSLCertificate& ssl_certificate() const;