RTCCertificate.getFingerprints added (exposed to the web)

third_party/WebKit/Source/modules/peerconnection/RTCCertificate.cpp

 /*
  * Copyright (C) 2015 Google Inc. All rights reserved.
  *
  * 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
@@ skipping 12 matching lines @@
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "modules/peerconnection/RTCCertificate.h"
 
+#include "bindings/core/v8/ScriptState.h"
+#include "bindings/core/v8/ToV8.h"
 #include "platform/wtf/PtrUtil.h"
 
 namespace blink {
 
 RTCCertificate::RTCCertificate(std::unique_ptr<WebRTCCertificate> certificate)
     : certificate_(WTF::WrapUnique(certificate.release())) {}
 
 std::unique_ptr<WebRTCCertificate> RTCCertificate::CertificateShallowCopy()
     const {
   return certificate_->ShallowCopy();
 }
 
 DOMTimeStamp RTCCertificate::expires() const {
   return static_cast<DOMTimeStamp>(certificate_->Expires());
 }
 
+Vector<v8::Local<v8::Value>> RTCCertificate::fingerprints(
+    ScriptState* script_state) {
+  v8::Isolate* isolate = script_state->GetIsolate();
+  if (!fingerprints_.size()) {

[Guido Urdaneta, 2017/04/19 12:20:42]

use IsEmpty() instead of !size

[hbos_chromium, 2017/04/20 09:07:09]

Done.

+    v8::Local<v8::Object> creation_context =
+        script_state->GetContext()->Global();
+    for (const WebRTCDtlsFingerprint& web_fingerprint :
+         certificate_->Fingerprints()) {
+      RTCDtlsFingerprint fingerprint;
+      fingerprint.setAlgorithm(web_fingerprint.Algorithm());
+      fingerprint.setValue(web_fingerprint.Value());
+      fingerprints_.push_back(v8::Global<v8::Value>(

[Guido Urdaneta, 2017/04/19 12:20:42]

I've seen that other cases that return a FrozenArray of a dictionary type use a
FreezeV8Object function. Should you use it too?

[hbos_chromium, 2017/04/20 09:07:09]

Yes, good catch. Without it the array is frozen but not the objects inside it.

+          isolate, ToV8(fingerprint, creation_context, isolate)));
+    }
+  }
+  Vector<v8::Local<v8::Value>> fingerprints(fingerprints_.size());
+  for (size_t i = 0; i < fingerprints_.size(); ++i) {
+    fingerprints[i] = v8::Local<v8::Value>::New(isolate, fingerprints_[i]);
+  }
+  return fingerprints;
+}
+
 }  // namespace blink