Updating documentation

ppapi/c/pp_instance.h

Index: ppapi/c/pp_instance.h
===================================================================
--- ppapi/c/pp_instance.h	(revision 73518)
+++ ppapi/c/pp_instance.h	(working copy)
@@ -7,7 +7,8 @@
 
 /**
  * @file
- * Defines the API ...
+ * This file defines the PP_Instance type which uniquely identifies one module
+ * instance.
  */
 
 #include "ppapi/c/pp_macros.h"
@@ -18,12 +19,12 @@
  * @{
  */
 
-/** A PP_Instance uniquely identifies one plugin instance, which is one time
- * that a page says \<embed>. There can be multiple instances of the same plugin
- * type on a page that will all be in the same module.
+/**
+ * The PP_Instance type uniquely identifies one module instance. There will

[dmichael(do not use this one), 2011/02/02 23:36:51]

'The PP_Instance type' -> 'A PP_Instance value'

Also, I think 'module instance' is slightly confusing.  Maybe 'instance of the
module' would be better?  It's a tricky thing to convey, but the important thing
is that there are 1 or more PP_Instances for each PP_Module.  The PP_Module maps
to a particular module (a .nexe or a plugin library), and a PP_Instance maps to
an <embed> tag.  There can be multiple <embed> tags on a page that refer to the
same module, and each of those will have exactly 1 PP_Instance.  They may share
1 PP_Module and be in the same address space (but they might not).

It's a tricky concept to describe in words, I think...  so I'll leave it to you
;-)

[jond, 2011/02/03 17:28:11]

 * The PP_Instance value uniquely identifies one instance of a module
 * (.nexe/PP_Module). There will be one module instance for every
 * \<embed> tag on a page.

On 2011/02/02 23:36:51, dmichael wrote:

+ * be one module instance for every \<embed> tag on a page.
  *
- * The identifier is an opaque handle assigned by the browser to the plugin. It
- * is guaranteed never to be 0, so a plugin can initialize it to 0 to
+ * This identifier is an opaque handle assigned by the browser to the module. It
+ * is guaranteed never to be 0, so a module can initialize it to 0 to
  * indicate a "NULL handle."
  */
 typedef int32_t PP_Instance;