diff options
author | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-14 11:23:23 +0000 |
---|---|---|
committer | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-14 11:23:23 +0000 |
commit | 192bd691ed6eedb3e08b5b619019cb7cb4300b84 (patch) | |
tree | a9164dc70a18a2e008d28103d582a83841e7bc8a /webkit/blob/blob_url_request_job_unittest.cc | |
parent | a91ef5e4ab532cc708ad90a1ddb890bb1c839300 (diff) | |
download | chromium_src-192bd691ed6eedb3e08b5b619019cb7cb4300b84.zip chromium_src-192bd691ed6eedb3e08b5b619019cb7cb4300b84.tar.gz chromium_src-192bd691ed6eedb3e08b5b619019cb7cb4300b84.tar.bz2 |
Separate FileStream related code out of BlobURLRequestJob
This patch separates FileStream related code out of BlobURLRequestJob to:
- make BlobURLRequesetJob's responsibility simpler and clearer by moving file-related details out of the class
- remove blocking file operation on IO thread
Long-term goal includes using the same reader class/interface to FileSystemURLRequestJob and other fileapi streaming related code.
(Simplified the patch from the previous version: was: "Add FileStreamInterface for fileapi/blob stream related operations")
BUG=114999,75548
TEST=blob tests
Review URL: https://chromiumcodereview.appspot.com/9651032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126623 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/blob/blob_url_request_job_unittest.cc')
-rw-r--r-- | webkit/blob/blob_url_request_job_unittest.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/webkit/blob/blob_url_request_job_unittest.cc b/webkit/blob/blob_url_request_job_unittest.cc index 12a0cb7..515574a 100644 --- a/webkit/blob/blob_url_request_job_unittest.cc +++ b/webkit/blob/blob_url_request_job_unittest.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2012 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -259,10 +259,10 @@ class BlobURLRequestJobTest : public testing::Test { void VerifyResponse() { EXPECT_TRUE(request_->status().is_success()); - EXPECT_EQ(request_->response_headers()->response_code(), - expected_status_code_); - EXPECT_STREQ(url_request_delegate_->response_data().c_str(), - expected_response_.c_str()); + EXPECT_EQ(expected_status_code_, + request_->response_headers()->response_code()); + EXPECT_STREQ(expected_response_.c_str(), + url_request_delegate_->response_data().c_str()); TestFinished(); } |