Updating documentation

ppapi/c/pp_point.h

 /* Copyright (c) 2010 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_PP_POINT_H_
 #define PPAPI_C_PP_POINT_H_
 
 /**
  * @file
- * Defines the API ...
+ * This file defines the APIs to create a 2 dimensional point on the screen.

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

s/APIs/API
s/create/represent
s/on the screen/in the module's portal (???)
(Maybe Dave S can help me with the right words;  it's the portion of the page
that the module can draw in, but not necessarily the screen.)

Or...  just eliminate ' on the screen'

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

Done.

+ * 0,0 is the upper-left corner of the screen.

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

s/screen/drawing window (???)

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

Done.

  */
 
 #include "ppapi/c/pp_macros.h"
 #include "ppapi/c/pp_stdint.h"
 
 /**
  *
  * @addtogroup Structs
  * @{
  */
+
+/**
+ * The PP_Point structure defines the x and y coordinates of a point on the
+ * screen.
+ */

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

Not 'on the screen'

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

Done.

 struct PP_Point {
+  /** This value represents the horizontal coordinate of a point on the
+   * screen.

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

not 'on the screen', counting from 0 on the left.

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

Done.

+   */
   int32_t x;
+
+  /* This value represents the vertical coordinate of a point on the screen. */

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

not 'on the screen', counting from 0 at the top.

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

Done.

   int32_t y;
 };
 PP_COMPILE_ASSERT_STRUCT_SIZE_IN_BYTES(PP_Point, 8);
 /**
  * @}
  */
 
 /**
  * @addtogroup Functions
  * @{
  */
+
+/**
+ * PP_MakePoint() places two 32 bit integers inside a PP_Point structure.

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

I'd rather wording like:
'creates a PP_Point given the x and y coordinates as int32_t values.'

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

Done.

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

Done.

+ * @param[in] x A 32 bit value representing a horizontal coordinate on the
+ * screen.
+ * @param[in] y A 32 bit value representing a vertical coordinate on the
+ * screen.
+ * @return A PP_Point structure.
+ */
 PP_INLINE struct PP_Point PP_MakePoint(int32_t x, int32_t y) {
   struct PP_Point ret;
   ret.x = x;
   ret.y = y;
   return ret;
 }
 /**
  * @}
  */
 
 #endif  /* PPAPI_C_PP_POINT_H_ */