Define Stable API for WebRTC/Quartc

net/quic/quartc/quartc_factory_interface.h

+// Copyright (c) 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NET_QUIC_QUARTC_QUARTC_FACTORY_INTERFACE_H_
+#define NET_QUIC_QUARTC_QUARTC_FACTORY_INTERFACE_H_
+
+#include "net/base/net_export.h"

[zhihuang1, 2016/10/13 06:22:41]

Same as before. This is a stand-alone header with compiling related macro. It
will not introduce other indirect dependency

+#include "net/quic/quartc/quartc_session_interface.h"
+#include "net/quic/quartc/quartc_task_runner_interface.h"
+
+namespace net {
+
+// Used to create instances for Quartc objects such as QuartcSession.
+class QuartcFactoryInterface {
+ public:
+  virtual ~QuartcFactoryInterface() {}
+
+  struct QuartcSessionConfig {
+    // When using Quartc, there are two endpoints. The QuartcSession on one
+    // endpoint must act as a server and the one on the other side must act as a
+    // client.
+    bool is_server = false;
+    // This is only needed when is_server = false.  It must be unique
+    // for each endpoint the local endpoint may communicate with. For example,
+    // a WebRTC client could use the remote endpoint's crypto fingerprint
+    std::string unique_remote_server_id;
+    // The way the QuicConnection will send and receive packets, like a virtual
+    // UDP socket. For WebRTC, this will typically be an IceTransport.
+    QuartcSessionInterface::PacketTransport* packet_transport = nullptr;
+  };
+
+  virtual std::unique_ptr<QuartcSessionInterface> CreateQuartcSession(
+      const QuartcSessionConfig& quartc_config) = 0;
+};
+
+// The configuration for creating a QuartcFactory.
+struct QuartcFactoryConfig {
+  // The task runner used by the QuartcAlarm. Implemented by the Quartc user
+  // with different mechanism. For example in WebRTC, it is implemented with
+  // rtc::Thread.
+  QuartcTaskRunnerInterface* quartc_task_runner;

[pthatcher1, 2016/10/14 18:59:00]

"task_runner" is probably sufficient.

And should it have "= nullptr" like the others?

[zhihuang1, 2016/10/16 00:45:28]

I removed the "=nullptr" from other classes because it will be set in the
constructor. But I think it make sense to have "=nullptr" here because it won't
be set in the constructor of the Config.

+};
+
+// Creates a new instance of QuartcFactoryInterface.
+NET_EXPORT_PRIVATE std::unique_ptr<QuartcFactoryInterface> CreateQuartcFactory(
+    const QuartcFactoryConfig& factory_config);
+
+}  // namepace net
+
+#endif  // NET_QUIC_QUARTC_QUARTC_FACTORY_INTERFACE_H_