| Index: chrome/test/nacl/pnacl_header_test.cc
|
| diff --git a/chrome/test/nacl/pnacl_header_test.cc b/chrome/test/nacl/pnacl_header_test.cc
|
| index 8ba8dc8271e275c2b1f58e4f6a7f978e05245e8c..e4922aa5867f96e9a8a3a069aee37c589b11546e 100644
|
| --- a/chrome/test/nacl/pnacl_header_test.cc
|
| +++ b/chrome/test/nacl/pnacl_header_test.cc
|
| @@ -12,35 +12,15 @@
|
| #include "chrome/common/chrome_paths.h"
|
| #include "chrome/test/base/ui_test_utils.h"
|
| #include "chrome/test/nacl/nacl_browsertest_util.h"
|
| -#include "content/public/browser/resource_dispatcher_host.h"
|
| #include "content/public/browser/web_contents.h"
|
| #include "net/test/embedded_test_server/embedded_test_server.h"
|
| #include "net/test/embedded_test_server/http_request.h"
|
| #include "net/test/embedded_test_server/http_response.h"
|
| -#include "net/url_request/url_request.h"
|
|
|
| using net::test_server::BasicHttpResponse;
|
| using net::test_server::EmbeddedTestServer;
|
| using net::test_server::HttpRequest;
|
| using net::test_server::HttpResponse;
|
| -
|
| -void TestDispatcherHostDelegate::RequestBeginning(
|
| - net::URLRequest* request,
|
| - content::ResourceContext* resource_context,
|
| - content::AppCacheService* appcache_service,
|
| - content::ResourceType resource_type,
|
| - ScopedVector<content::ResourceThrottle>* throttles) {
|
| - // This checks the same condition as the one for PNaCl in
|
| - // AppendComponentUpdaterThrottles.
|
| - if (resource_type == content::RESOURCE_TYPE_OBJECT) {
|
| - const net::HttpRequestHeaders& headers = request->extra_request_headers();
|
| - std::string accept_headers;
|
| - if (headers.GetHeader("Accept", &accept_headers)) {
|
| - if (accept_headers.find("application/x-pnacl") != std::string::npos)
|
| - found_pnacl_header_ = true;
|
| - }
|
| - }
|
| -}
|
|
|
| PnaclHeaderTest::PnaclHeaderTest() : noncors_loads_(0), cors_loads_(0) {}
|
|
|
| @@ -61,7 +41,6 @@
|
| void PnaclHeaderTest::RunLoadTest(const std::string& url,
|
| int expected_noncors,
|
| int expected_cors) {
|
| - content::ResourceDispatcherHost::Get()->SetDelegate(&test_delegate_);
|
| StartServer();
|
| LoadTestMessageHandler handler;
|
| content::JavascriptTestObserver observer(
|
| @@ -78,9 +57,6 @@
|
| base::ScopedPathOverride component_dir(chrome::DIR_PNACL_COMPONENT);
|
|
|
| ui_test_utils::NavigateToURL(browser(), embedded_test_server()->GetURL(url));
|
| -
|
| - content::ResourceDispatcherHost::Get()->SetDelegate(NULL);
|
| -
|
| // Wait until the NMF and pexe are also loaded, not just the HTML.
|
| // Do this by waiting till the LoadTestMessageHandler responds.
|
| EXPECT_TRUE(observer.Run()) << handler.error_message();
|
| @@ -105,14 +81,14 @@
|
| if (absolute_url.path().find(".pexe") == std::string::npos)
|
| return scoped_ptr<HttpResponse>();
|
|
|
| - // For pexe files, check for the special Accept header,
|
| - // along with the expected ResourceType of the URL request.
|
| + // For pexe files, check for the special Accept header.
|
| + // This must match whatever is in:
|
| + // ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc
|
| EXPECT_NE(0U, request.headers.count("Accept"));
|
| std::map<std::string, std::string>::const_iterator it =
|
| request.headers.find("Accept");
|
| EXPECT_NE(std::string::npos, it->second.find("application/x-pnacl"));
|
| EXPECT_NE(std::string::npos, it->second.find("*/*"));
|
| - EXPECT_TRUE(test_delegate_.found_pnacl_header());
|
|
|
| // Also make sure that other headers like CORS-related headers
|
| // are preserved when injecting the special Accept header.
|
|
|