Fix a bug where immediate failure in PPB_URLLoader_Impl::Open doesn't report an error, breaking NaCl integration tests. Before the recent changes to this class, violations of same-origin restrictions returned an error immediately. Now, the loader calls the didFail callback, before the callback is set, so there is no notification of an error.
Fix a bug where immediate failure in PPB_URLLoader_Impl::Open doesn't report an error, breaking NaCl integration tests. Before the recent changes to this class, violations of same-origin restrictions returned an error immediately. Now, the loader calls the didFail callback, before the callback is set, so there is no notification of an error.

This patch checks for an immediate failure, and returns PP_ERROR_NOACCESS to keep existing cross-origin tests green. We should go ahead with some WebKit refactoring to return proper error codes, as per Darin's TODO.
Review URL: http://codereview.chromium.org/6864021

git-svn-id: http://src.chromium.org/svn/trunk/src@81838 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Former-commit-id: 8bdbbef9cc0f957a9578a36195204d37daaf65bd
meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser