Move the PP_ALLOW_THIS_IN_INITIALIZER_LIST macro from test_utils.h to

pp_macros.idl.

Now, PP_ALLOW_THIS_IN_INITIALIZER_LIST macro is provided for not only tests
but also C++ api implementation in NaCl.

BUG=87310
TEST=ui_tests


Review URL: http://codereview.chromium.org/8885012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113615 0039d316-1c4b-4281-b951-d872f2087c98

ppapi/api/pp_macros.idl

@@ -86,6 +86,22 @@ PP_COMPILE_ASSERT_SIZE_IN_BYTES_IMPL(NAME, struct NAME, SIZE)
 #define PP_COMPILE_ASSERT_ENUM_SIZE_IN_BYTES(NAME, SIZE) \
 PP_COMPILE_ASSERT_SIZE_IN_BYTES_IMPL(NAME, enum NAME, SIZE)
 
+/* This is roughly copied from base/compiler_specific.h, and makes it possible
+   to pass 'this' in a constructor initializer list, when you really mean it.
+   E.g.:
+   Foo::Foo(MyInstance* instance)
+       : PP_ALLOW_THIS_IN_INITIALIZER_LIST(callback_factory_(this)) {}
+ */
+#if defined(COMPILER_MSVC)
+# define PP_ALLOW_THIS_IN_INITIALIZER_LIST(code) \
+    __pragma(warning(push)) \
+    __pragma(warning(disable:4355)) \
+    code \
+    __pragma(warning(pop))
+#else
+# define PP_ALLOW_THIS_IN_INITIALIZER_LIST(code) code
+#endif
+
 /**
  * @}
  * End of addtogroup PP

ppapi/c/pp_macros.h

@@ -3,7 +3,7 @@
  * found in the LICENSE file.
  */
 
-/* From pp_macros.idl modified Sat Jul 16 16:50:26 2011. */
+/* From pp_macros.idl modified Thu Dec  8 23:25:05 2011. */
 
 #ifndef PPAPI_C_PP_MACROS_H_
 #define PPAPI_C_PP_MACROS_H_
@@ -93,6 +93,22 @@ PP_COMPILE_ASSERT_SIZE_IN_BYTES_IMPL(NAME, struct NAME, SIZE)
 #define PP_COMPILE_ASSERT_ENUM_SIZE_IN_BYTES(NAME, SIZE) \
 PP_COMPILE_ASSERT_SIZE_IN_BYTES_IMPL(NAME, enum NAME, SIZE)
 
+/* This is roughly copied from base/compiler_specific.h, and makes it possible
+   to pass 'this' in a constructor initializer list, when you really mean it.
+   E.g.:
+   Foo::Foo(MyInstance* instance)
+       : PP_ALLOW_THIS_IN_INITIALIZER_LIST(callback_factory_(this)) {}
+ */
+#if defined(COMPILER_MSVC)
+# define PP_ALLOW_THIS_IN_INITIALIZER_LIST(code) \
+    __pragma(warning(push)) \
+    __pragma(warning(disable:4355)) \
+    code \
+    __pragma(warning(pop))
+#else
+# define PP_ALLOW_THIS_IN_INITIALIZER_LIST(code) code
+#endif
+
 /**
  * @}
  * End of addtogroup PP

ppapi/tests/test_flash.cc

@@ -4,11 +4,11 @@
 
 #include "ppapi/tests/test_flash.h"
 
+#include "ppapi/c/pp_macros.h"
 #include "ppapi/c/private/ppb_flash.h"
 #include "ppapi/cpp/instance.h"
 #include "ppapi/cpp/module.h"
 #include "ppapi/cpp/var.h"
-#include "ppapi/tests/test_utils.h"
 #include "ppapi/tests/testing_instance.h"
 
 REGISTER_TEST_CASE(Flash);

ppapi/tests/test_transport.cc

@@ -12,6 +12,7 @@
 
 #include "ppapi/c/dev/ppb_testing_dev.h"
 #include "ppapi/c/pp_errors.h"
+#include "ppapi/c/pp_macros.h"
 #include "ppapi/cpp/completion_callback.h"
 #include "ppapi/cpp/dev/transport_dev.h"
 #include "ppapi/cpp/instance.h"

ppapi/tests/test_utils.h

@@ -79,20 +79,4 @@ class TestCompletionCallback {
 #define PPAPI_POSIX 1
 #endif
 
-// This is roughly copied from base/compiler_specific.h, and makes it possible
-// to pass 'this' in a constructor initializer list, when you really mean it.
-//
-// Example usage:
-// Foo::Foo(MyInstance* instance)
-//     : ALLOW_THIS_IN_INITIALIZER_LIST(callback_factory_(this)) {}
-#if defined(COMPILER_MSVC)
-#define PP_ALLOW_THIS_IN_INITIALIZER_LIST(code) \
-    __pragma(warning(push)) \
-    __pragma(warning(disable:4355)) \
-    code \
-    __pragma(warning(pop))
-#else
-#define PP_ALLOW_THIS_IN_INITIALIZER_LIST(code) code
-#endif
-
 #endif  // PPAPI_TESTS_TEST_UTILS_H_