Add SetProperty() in the PPB_Transport_Dev interface.

ppapi/c/dev/ppb_transport_dev.h

 /* Copyright (c) 2011 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 PPAPI_C_PPB_TRANSPORT_DEV_H_
 #define PPAPI_C_PPB_TRANSPORT_DEV_H_
 
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_module.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_resource.h"
 #include "ppapi/c/pp_stdint.h"
 #include "ppapi/c/pp_var.h"
 
 #define PPB_TRANSPORT_DEV_INTERFACE_0_5 "PPB_Transport;0.5"

[brettw, 2011/08/25 16:48:43]

Since we're not keeping backwards compat, you can remove this old line.

[Sergey Ulanov, 2011/08/25 17:36:53]

Done.

-#define PPB_TRANSPORT_DEV_INTERFACE PPB_TRANSPORT_DEV_INTERFACE_0_5
+#define PPB_TRANSPORT_DEV_INTERFACE_0_6 "PPB_Transport;0.6"
+#define PPB_TRANSPORT_DEV_INTERFACE PPB_TRANSPORT_DEV_INTERFACE_0_6
+
+typedef enum {
+  // STUN server address and port, e.g  "stun.l.google.com:19302".

[brettw, 2011/08/25 16:48:43]

Why don't you use "...example.com" here and below for the examples.

[Sergey Ulanov, 2011/08/25 17:36:53]

Done.

+  PP_TRANSPORTPROPERTY_STUN_SERVER,

[brettw, 2011/08/25 16:48:43]

These should have explicit values, so can you say "= 0", etc for them?

Wasn't at least one of these a list of servers? It seems like the comments
should say the format for that one.

[Sergey Ulanov, 2011/08/25 17:36:53]

Done.

+
+  // Relay server name, e.g. "relay.google.com".
+  PP_TRANSPORTPROPERTY_RELAY_SERVER,
+
+  // Single string that specifies token for use with relay server.
+  PP_TRANSPORTPROPERTY_RELAY_TOKEN,
+
+  // TCP receive window in bytes. Takes effect only for PseudoTCP
+  // connections.
+  PP_TRANSPORTPROPERTY_TCP_RECEIVE_WINDOW,
+
+  // TCP send window in bytes. Takes effect only for PseudoTCP
+  // connections.
+  PP_TRANSPORTPROPERTY_TCP_SEND_WINDOW
+} PP_TransportProperty;
+PP_COMPILE_ASSERT_SIZE_IN_BYTES(PP_TransportProperty, 4);
 
 struct PPB_Transport_Dev {
   // Creates a new transport object with the specified name using the
   // specified protocol.
   PP_Resource (*CreateTransport)(PP_Instance instance,
                                  const char* name,
                                  const char* proto);
 
   // Returns PP_TRUE if resource is a Transport, PP_FALSE otherwise.
   PP_Bool (*IsTransport)(PP_Resource resource);
 
   // Returns PP_TRUE if the transport is currently writable (i.e. can
   // send data to the remote peer), PP_FALSE otherwise.
   PP_Bool (*IsWritable)(PP_Resource transport);
   // TODO(juberti): other getters/setters
   // connect state
   // connect type, protocol
   // RTT
 
+  // Sets various configuration properties of the transport.
+  int32_t (*SetProperty)(PP_Resource transport,
+                         PP_TransportProperty property,
+                         struct PP_Var value);
+
   // Establishes a connection to the remote peer.  Returns
   // PP_OK_COMPLETIONPENDING and notifies on |cb| when connectivity is
   // established (or timeout occurs).
   int32_t (*Connect)(PP_Resource transport,
                      struct PP_CompletionCallback cb);
 
   // Obtains another ICE candidate address to be provided to the
   // remote peer. Returns PP_OK_COMPLETIONPENDING if there are no more
   // addresses to be sent. After the callback is called
   // GetNextAddress() must be called again to get the address.
@@ skipping 18 matching lines @@
   int32_t (*Send)(PP_Resource transport,
                   const void* data,
                   uint32_t len,
                   struct PP_CompletionCallback cb);
 
   // Disconnects from the remote peer.
   int32_t (*Close)(PP_Resource transport);
 };
 
 #endif  /* PPAPI_C_PPB_TRANSPORT_DEV_H_ */