PPAPI URLRequestInfo test: port NaCl version to ppapi_tests. Fix Var and Resource leaks.

chrome/test/ui/ppapi_uitest.cc

Index: chrome/test/ui/ppapi_uitest.cc
===================================================================
--- chrome/test/ui/ppapi_uitest.cc	(revision 117755)
+++ chrome/test/ui/ppapi_uitest.cc	(working copy)
@@ -465,6 +465,23 @@
 TEST_PPAPI_NACL_VIA_HTTP(URLLoader_AbortCalls)
 TEST_PPAPI_NACL_VIA_HTTP(URLLoader_UntendedLoad)
 
+// URLRequestInfo tests.
+TEST_PPAPI_IN_PROCESS_VIA_HTTP(URLRequest_Create)
+TEST_PPAPI_OUT_OF_PROCESS_VIA_HTTP(URLRequest_Create)
+TEST_PPAPI_NACL_VIA_HTTP(URLRequest_Create)
+TEST_PPAPI_IN_PROCESS_VIA_HTTP(URLRequest_IsURLRequestInfo)
+TEST_PPAPI_OUT_OF_PROCESS_VIA_HTTP(URLRequest_IsURLRequestInfo)
+TEST_PPAPI_NACL_VIA_HTTP(URLRequest_IsURLRequestInfo)
+TEST_PPAPI_IN_PROCESS_VIA_HTTP(URLRequest_SetProperty)
+TEST_PPAPI_OUT_OF_PROCESS_VIA_HTTP(URLRequest_SetProperty)
+TEST_PPAPI_NACL_VIA_HTTP(URLRequest_SetProperty)
+TEST_PPAPI_IN_PROCESS_VIA_HTTP(URLRequest_AppendDataToBody)
+TEST_PPAPI_OUT_OF_PROCESS_VIA_HTTP(URLRequest_AppendDataToBody)
+TEST_PPAPI_NACL_VIA_HTTP(URLRequest_AppendDataToBody)
+TEST_PPAPI_IN_PROCESS_VIA_HTTP(URLRequest_Stress)
+TEST_PPAPI_OUT_OF_PROCESS_VIA_HTTP(URLRequest_Stress)
+TEST_PPAPI_NACL_VIA_HTTP(URLRequest_Stress)

[bbudge, 2012/01/16 16:24:29]

Each PPAPITest takes 10-20 seconds to run. Could we collapse the really basic
ones like Create, IsURLRequestInfo, and SetProperty into a single test, ala
ResourceArray_Basics, below?

[polina1, 2012/01/26 02:56:26]

I think it is a very good idea to combine the first two (and I did), but not
convinced that SetProperty should be merged as well. It pretty long and
detailed. And I think the whole point of separate test cases was to be able to
narrow things down and disable/enable them independently of the rest.

+
 TEST_PPAPI_IN_PROCESS(PaintAggregator)
 TEST_PPAPI_OUT_OF_PROCESS(PaintAggregator)
 TEST_PPAPI_NACL_VIA_HTTP(PaintAggregator)