More trivial cleanupi of ppapi/c/trusted headers

ppapi/c/trusted/ppp_broker.h

Index: ppapi/c/trusted/ppp_broker.h
===================================================================
--- ppapi/c/trusted/ppp_broker.h	(revision 92721)
+++ ppapi/c/trusted/ppp_broker.h	(working copy)
@@ -2,27 +2,36 @@
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
+
+/* From trusted/ppp_broker.idl modified Mon Jul 11 15:09:12 2011. */
+
 #ifndef PPAPI_C_TRUSTED_PPP_BROKER_H_
 #define PPAPI_C_TRUSTED_PPP_BROKER_H_
 
+#include "ppapi/c/pp_macros.h"
+
+/**
+ * @file
+ * This file defines functions that your module must implement to support a
+ * broker.

[dmichael (off chromium), 2011/07/17 02:00:10]

Why have you broken up the includes?

[noelallen1, 2011/07/18 17:51:56]

Because the majority of this file is direct injected code, not generated.  See
bellow.

+ */
+
+
+// {PENDING: undefine PP_EXPORT?}

[dmichael (off chromium), 2011/07/17 02:00:10]

This comment should probably be C-style

[noelallen1, 2011/07/18 17:51:56]

Currently this file is generated via a:
#inline c
#endinc
block.

This is because the generator does not emit #if __GNUC__ >= 4, etc...

Also this is a function without an interface, which is illegal in the IDL.

Our two choices in this case are to leave in an non-generated header, or to use
the #inline block to include raw source.   I figured the later was a better
path.

[dmichael (off chromium), 2011/07/18 18:00:16]

Okay, thanks for explaining. It seems we should fix the code that's being placed
inline, then. Optional for this CL.

+
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_stdint.h"
 
+
 #if __GNUC__ >= 4
+
 #define PP_EXPORT __attribute__ ((visibility("default")))
 #elif defined(_MSC_VER)
 #define PP_EXPORT __declspec(dllexport)
 #endif
 
-/**
- * @file
- * This file defines functions that your module must implement to support a
- * broker.
- */
 
-// {PENDING: undefine PP_EXPORT?}
 
-
 /* We don't want name mangling for these external functions.  We only need
  * 'extern "C"' if we're compiling with a C++ compiler.
  */
@@ -90,3 +99,4 @@
 #endif
 
 #endif  /* PPAPI_C_TRUSTED_PPP_BROKER_H_ */
+