Reduce number of unnamed-type-template-args violations.

ppapi/c/pp_errors.h

Index: ppapi/c/pp_errors.h
===================================================================
--- ppapi/c/pp_errors.h	(revision 97489)
+++ ppapi/c/pp_errors.h	(working copy)
@@ -24,7 +24,7 @@
  * This enumeration contains enumerators of all PPAPI error codes.
  * Errors are negative valued.
  */
-enum {
+enum ErrorCodes {

[brettw, 2011/08/21 05:37:03]

This isn't in any namespace so I don't think we want to use this generic name
here!

We've previously discussed having a named enum type here but have decided
against it so far. PPAPI error codes represent either a value from here, or a
positive value representing success (usually number of bytes read or something).
We've been worried that naming this type would encourage people to actually use
it, which would be confusing in some cases.

[Peter Kasting, 2011/08/22 03:56:41]

Oof.  Honestly that seems bad.  Can't you namespace this?  I feel uncomfortable
adding this sort of thing to the global namespace.
That's also true of various other enums I name in this patch, though, such as
the content-and-chrome notification types (where using either type separately is
bad because any function which works with them needs to handle both).  The
downside of not naming the enums is that it is impossible to refer to the type
by name even when you really want to (for example, for a function which has no
return value other than one of these codes) and (until C++11) impossible to pass
these values to templates.

[brettw, 2011/08/22 04:11:08]

This is the public Pepper API which must be compilable in C mode, so no
namespaces. We normally prefix things with PP_, so if we were to name this it
would be PP_Error or something.

   /**
    * This value is returned by a function on successful synchronous completion
    * or is passed as a result to a PP_CompletionCallback_Func on successful