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

Unified Diff: ppapi/tests/test_url_loader.cc

Issue 7046091: Fix problems with PPB_URLLoader_Impl and PPAPITests.URLLoader. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 6 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: ppapi/tests/test_url_loader.cc
===================================================================
--- ppapi/tests/test_url_loader.cc (revision 88604)
+++ ppapi/tests/test_url_loader.cc (working copy)
@@ -49,7 +49,9 @@
RUN_TEST(CustomRequestHeader);
RUN_TEST(IgnoresBogusContentLength);
RUN_TEST(SameOriginRestriction);
- RUN_TEST(CrossOriginRequest);
+ // TODO(bbudge) Find out how to create a request that URL loaders would
+ // consider cross-origin, but that can be served by test server.
+ //RUN_TEST(CrossOriginRequest);
RUN_TEST(StreamToFile);
RUN_TEST(AuditURLRedirect);
RUN_TEST(AbortCalls);
@@ -284,7 +286,7 @@
rv = callback.WaitForResult();
// We expect success since we allowed a cross-origin request.
- if (rv == PP_ERROR_NOACCESS)
+ if (rv != PP_OK)
return ReportError("URLLoader::Open()", rv);
PASS();
« no previous file with comments | « no previous file | webkit/plugins/ppapi/ppb_url_loader_impl.cc » ('j') | webkit/plugins/ppapi/ppb_url_loader_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698