Make NetworkList and NetworkMonitor APIs public

ppapi/api/ppb_network_monitor.idl

Index: ppapi/api/ppb_network_monitor.idl
diff --git a/ppapi/api/private/ppb_network_monitor_private.idl b/ppapi/api/ppb_network_monitor.idl
similarity index 83%
rename from ppapi/api/private/ppb_network_monitor_private.idl
rename to ppapi/api/ppb_network_monitor.idl
index fe264a616bc16326f969518501f358709646b2d1..abe1e64ce8ab09b0829a0ba8d3d878c2118e0ba6 100644
--- a/ppapi/api/private/ppb_network_monitor_private.idl
+++ b/ppapi/api/ppb_network_monitor.idl
@@ -4,29 +4,29 @@
  */
 
 /**
- * This file defines the <code>PPB_NetworkMonitor_Private</code> interface.
+ * This file defines the <code>PPB_NetworkMonitor</code> interface.
  */
 
 label Chrome {
-  M19 = 0.2
+  M31 = 1.0
 };
 
 /**
  * <code>PPB_NetworkMonitor_Callback</code> is a callback function
  * type that is used to receive notifications about network
  * configuration changes. The <code>network_list</code> passed to this
- * callback is a <code>PPB_NetworkList_Private</code> resource that
- * contains current configuration of network interfaces.
+ * callback is a <code>PPB_NetworkList</code> resource that contains
+ * current configuration of network interfaces.
  */
 typedef void PPB_NetworkMonitor_Callback([inout] mem_t user_data,
                                          [in] PP_Resource network_list);
 
 
 /**
- * The <code>PPB_NetworkMonitor_Private</code> provides access to
+ * The <code>PPB_NetworkMonitor</code> provides access to
  * notifications of network configuration changes.
  */
-interface PPB_NetworkMonitor_Private {
+interface PPB_NetworkMonitor {
   /**
    * Starts network change monitoring. The specified

[yzshen1, 2013/08/30 17:31:46]

There were discussions about how event notifications should be done. And people
think that the preferred way is to do something like this:

// Create() doesn't take a callback.
PP_Resource Create(PP_Instance);

// Monitor() returns with the next change event. If you would like to 
// continue monitoring, call Monitor() again when |callback| is run.
int32_t MonitorNetworkListChange([in] PP_Resource network_monitor,
                                 [out] PP_Resource* network_list,
                                 [in] PP_CompletionCallback callback);

Besides, in order to get the initial network list, we can either define that the
first call to MonitorNetworkListChange() after the resource is created returns
the current network list. Or we have to provide another method:

int32_t GetNetworkList([in] PP_Resource network_monitor,
                       [out] PP_Resource* network_list,
                       [in] PP_CompletionCallback callback);

What do you think?

    * <code>callback</code> will be called on the main thread once

[yzshen1, 2013/08/30 17:31:46]

We should issue the callback on the thread where Create() is called. (If we
decide not to change this interface according to my previous comment.)

@@ -57,7 +57,7 @@ interface PPB_NetworkMonitor_Private {
    * @param[in] resource A <code>PP_Resource</code> resource.
    *
    * @return Returns <code>PP_TRUE</code> if <code>resource</code> is
-   * a <code>PPB_NetworkMonitor_Private</code>, <code>PP_FALSE</code>
+   * a <code>PPB_NetworkMonitor</code>, <code>PP_FALSE</code>
    * otherwise.
    */
   PP_Bool IsNetworkMonitor([in] PP_Resource resource);