Refine PNaCl pexe fetch UMA errors to know if the user aborted, etc.

ppapi/native_client/tests/ppapi_browser/bad/ppapi_bad.html

Index: ppapi/native_client/tests/ppapi_browser/bad/ppapi_bad.html
diff --git a/ppapi/native_client/tests/ppapi_browser/bad/ppapi_bad.html b/ppapi/native_client/tests/ppapi_browser/bad/ppapi_bad.html
index a76d4e234996813c4d7e5af15aa5678a2557a72e..539517cb36d0a8a06ac4b20b89c3824910060edd 100644
--- a/ppapi/native_client/tests/ppapi_browser/bad/ppapi_bad.html
+++ b/ppapi/native_client/tests/ppapi_browser/bad/ppapi_bad.html
@@ -21,12 +21,12 @@
 //<![CDATA[
 
 // PNaCl may have slightly different error messages (pexe instead of nexe).
-function couldNotLoadNexe(is_pnacl, expected_pp_error) {
+function couldNotLoadNexe(is_pnacl, pnacl_refinement) {
   if (!is_pnacl) {
     return 'NaCl module load failed: could not load nexe url.';
   } else {
-    return 'NaCl module load failed: PnaclCoordinator: pexe load failed ' +
-      '(pp_error=' + expected_pp_error + ').';
+    return 'NaCl module load failed: PnaclCoordinator: pexe load failed' +
+      pnacl_refinement;

[Derek Schuff, 2013/01/16 16:50:51]

might make more sense to put the parens and the period here instead of in the
input string

[jvoung (off chromium), 2013/01/16 17:30:30]

Done.

   }
 }
 
@@ -46,7 +46,7 @@ function declareTests(tester) {
       tester,
       'cross_origin',
       'ppapi_bad_crossorigin.nmf',
-      couldNotLoadNexe(is_pnacl, -7 /* PP_ERROR_NOACCESS */));
+      couldNotLoadNexe(is_pnacl, ' (no access).'));
 
   // 'cross_manifest' tries to load a cross-origin manifest.
   badLoadTest(
@@ -61,7 +61,7 @@ function declareTests(tester) {
       tester,
       'nonexistent_nexe',
       'ppapi_bad_doesnotexist.nmf',
-      couldNotLoadNexe(is_pnacl, -2 /* PP_ERROR_FAILED */));
+      couldNotLoadNexe(is_pnacl, ' (pp_error=-2).' /* PP_ERROR_FAILED */));
 
   // 'nonexistent_nexe_only' loads a manifest, then tries to load a nonexistent
   // nexe, given only a nexe to choose from.