RTCCertificate and RTCPeerConnection.generateCertificate added to JavaScript

public/platform/WebRTCCertificate.h

 /*
  * Copyright (C) 2015 Google Inc. All rights reserved.
- * Copyright (C) 2015 Intel 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
  * copyright notice, this list of conditions and the following disclaimer
  * in the documentation and/or other materials provided with the
  * distribution.
  *     * Neither the name of Google Inc. nor the names of its
  * contributors may be used to endorse or promote products derived from
  * this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
  * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * 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.
  */
 
-#ifndef PerformanceCompositeTiming_h
-#define PerformanceCompositeTiming_h
+#ifndef WebRTCCertificate_h
+#define WebRTCCertificate_h
 
-#include "core/timing/PerformanceEntry.h"
-#include "platform/heap/Handle.h"
-#include "wtf/Forward.h"
-#include "wtf/text/WTFString.h"
+#include "public/platform/WebCallbacks.h"
+#include "public/platform/WebURL.h"
+#include "wtf/Noncopyable.h"
 
 namespace blink {
 
-class Document;
+// Interface defining what blink needs to know about certificates (abstracting away
+// Chrome and WebRTC layer implementation details). You can create shallow copies of the
+// WebRTCCertificate. When all copies are destroyed, the implementation-specific data must
+// be freed.

[jochen (gone - plz use gerrit), 2015/09/15 08:04:19]

why is this restriction required?

[hbos_chromium, 2015/09/21 16:12:22]

See previous message about why the shallow copying. Either WebRTCCertificate
itself should be reference counted or its internal data should (hence shallow
copying).
Updated comment.

+class WebRTCCertificate {

[jochen (gone - plz use gerrit), 2015/09/15 08:04:19]

should there be a method for getting the keyType?

[hbos_chromium, 2015/09/21 16:12:22]

Done.

+    WTF_MAKE_NONCOPYABLE(WebRTCCertificate);
+public:
+    WebRTCCertificate() = default;
+    virtual ~WebRTCCertificate() = default;
 
-class PerformanceCompositeTiming final : public PerformanceEntry {
-    DEFINE_WRAPPERTYPEINFO();
+    // Copies the WebRTCCertificate object without copying the underlying
+    // implementation-specific (WebRTC layer) certificate. When all copies are destroyed
+    // the underlying data is freed.
+    virtual WebRTCCertificate* shallowCopy() const = 0;

[jochen (gone - plz use gerrit), 2015/09/15 08:04:19]

does it matter that it's a shallow copy?

[hbos_chromium, 2015/09/21 16:12:22]

(Originals and shallow copies are "equals". Besides being an abstraction layer
they're basically just a way to have reference counting to the internal data.
See previous message addressing why the shallow copying.)

+
+    virtual double expires() const = 0;

[jochen (gone - plz use gerrit), 2015/09/15 08:04:19]

it's unclear what this should return. please add a comment, and rename the
method to something like

notValidAfterTimestampInMS()

or whatever this means.

shouldn't this also have a notValidBeforeTimestampInMS()?

[hbos_chromium, 2015/09/21 16:12:22]

Keeping the name due to the specs so that everything calls it the same thing,
not just RTCCertificate. (Is that OK?)
Added a comment that should clear up any confusion.

+};
+
+// Interface defining a class that can generate WebRTCCertificates asynchronously.
+class WebRTCCertificateGenerator {

[jochen (gone - plz use gerrit), 2015/09/15 08:04:19]

please move this class to a separate header

[hbos_chromium, 2015/09/21 16:12:22]

Done.

 public:
-    static PerformanceCompositeTiming* create(Document* requestingDocument, unsigned sourceFrame, double startTime)
-    {
-        return new PerformanceCompositeTiming(requestingDocument, sourceFrame, startTime);
-    }
+    virtual ~WebRTCCertificateGenerator() { }
 
-    unsigned sourceFrame() const;
-
-    bool isComposite() override { return true; }
-
-    DECLARE_VIRTUAL_TRACE();
-
-private:
-    PerformanceCompositeTiming(Document* requestingDocument, unsigned sourceFrame, double startTime);
-    ~PerformanceCompositeTiming() override;
-
-    unsigned m_sourceFrame;
-    RefPtrWillBeMember<Document> m_requestingDocument;
+    // Start generating a certificate. The observer's onGenerateComplete is called when
+    // the asynchronous operation finished (whether or not successful).
+    virtual void generateCertificate(
+        int keyType,

[jochen (gone - plz use gerrit), 2015/09/15 08:04:19]

should be an enum? (which in turn should live in its own header)

[hbos_chromium, 2015/09/21 16:12:22]

Moved to new file and changed to parameterizable class (WebRTCKeyType) to
reflect how the KeyType will look in webrtc in the future.

+        const WebURL& /*url*/,
+        const WebURL& /*firstPartyForCookies*/,
+        WebCallbacks<WebRTCCertificate*, void>* observer) = 0;
 };
 
 } // namespace blink
 
-#endif // PerformanceCompositeTiming_h
+#endif // WebRTCCertificate_h