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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ppapi/tests/test_url_loader.cc

Issue 4182010: Use PASS() everywhere in ppapi/tests. (Closed) Base URL: https://ppapi.googlecode.com/svn/trunk/tests
Patch Set: First. Created 9 years, 11 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
« no previous file with comments | « ppapi/tests/test_transport.cc ('k') | ppapi/tests/test_url_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/tests/test_url_loader.cc
diff --git a/ppapi/tests/test_url_loader.cc b/ppapi/tests/test_url_loader.cc
index 7d4b3b2535cd9d5894e9e14855e38a3d7e18c77a..8b0dfc62f5b88ae26c18213e2e790513bf0d223f 100644
--- a/ppapi/tests/test_url_loader.cc
+++ b/ppapi/tests/test_url_loader.cc
@@ -242,7 +242,7 @@ std::string TestURLLoader::TestStreamToFile() {
if (file_descriptor < 0)
return "FileIO::GetOSFileDescriptor() returned a bad file descriptor.";
- return "";
+ PASS();
}
std::string TestURLLoader::TestSameOriginRestriction() {
@@ -265,7 +265,7 @@ std::string TestURLLoader::TestSameOriginRestriction() {
}
}
- return "";
+ PASS();
}
// This test should cause a redirect and ensure that the loader runs
« no previous file with comments | « ppapi/tests/test_transport.cc ('k') | ppapi/tests/test_url_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698