Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(618)

Unified Diff: chrome/test/nacl/nacl_browsertest.cc

Issue 10918152: Port nacl_integration exit_status test to browser_tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/nacl/nacl_browsertest.cc
diff --git a/chrome/test/nacl/nacl_browsertest.cc b/chrome/test/nacl/nacl_browsertest.cc
index bd0f2cfcd928596b3f98bd72b8438e5b39a00646..9fec2e01a1409fbbf3117a1dec56428cef6ea514 100644
--- a/chrome/test/nacl/nacl_browsertest.cc
+++ b/chrome/test/nacl/nacl_browsertest.cc
@@ -10,4 +10,19 @@ NACL_BROWSER_TEST_F(NaClBrowserTest, SimpleLoad, {
RunLoadTest(FILE_PATH_LITERAL("nacl_load_test.html"));
})
+NACL_BROWSER_TEST_F(NaClBrowserTest, ExitStatus, {
+ RunNaClIntegrationTest(FILE_PATH_LITERAL(
Mark Seaborn 2012/09/11 22:10:06 Indenting by 4 isn't consistent with the block abo
Nick Bray (chromium) 2012/09/12 01:19:13 Done.
+ "pm_exit_status_test.html?trigger=exit0&expected_exit=0"
+ ));
Mark Seaborn 2012/09/11 22:10:06 Closing ) on a new line isn't the usual C++ style?
Nick Bray (chromium) 2012/09/12 01:19:13 Done.
+ RunNaClIntegrationTest(FILE_PATH_LITERAL(
+ "pm_exit_status_test.html?trigger=exit7&expected_exit=7"
+ ));
+ RunNaClIntegrationTest(FILE_PATH_LITERAL(
+ "pm_exit_status_test.html?trigger=exit254&expected_exit=254"
+ ));
+ RunNaClIntegrationTest(FILE_PATH_LITERAL(
+ "pm_exit_status_test.html?trigger=exitneg2&expected_exit=254"
+ ));
+})
+
} // namespace anonymous

Powered by Google App Engine
This is Rietveld 408576698