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

Unified Diff: webkit/blob/blob_url_request_job.cc

Issue 10828252: Support FileSystem URL in File object (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comment fix Created 8 years, 4 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 | « webkit/blob/blob_storage_controller.cc ('k') | webkit/blob/view_blob_internals_job.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/blob/blob_url_request_job.cc
diff --git a/webkit/blob/blob_url_request_job.cc b/webkit/blob/blob_url_request_job.cc
index 642f9554244a139c41decb6913cf41cd885679c2..1ef9545a77663f7368750b97ddddfa6010c60d0f 100644
--- a/webkit/blob/blob_url_request_job.cc
+++ b/webkit/blob/blob_url_request_job.cc
@@ -308,6 +308,9 @@ bool BlobURLRequestJob::ReadItem() {
case BlobData::Item::TYPE_FILE:
return ReadFileItem(GetFileStreamReader(current_item_index_),
bytes_to_read);
+ case BlobData::Item::TYPE_FILE_FILESYSTEM:
+ // TODO(kinuko): Support TYPE_FILE_FILESYSTEM case.
+ // http://crbug.com/141835
default:
DCHECK(false);
return false;
@@ -530,6 +533,8 @@ LocalFileStreamReader* BlobURLRequestJob::GetFileStreamReader(size_t index) {
const BlobData::Item& item = blob_data_->items().at(index);
if (item.type() != BlobData::Item::TYPE_FILE)
return NULL;
+ // TODO(kinuko): Create appropriate FileStreamReader for TYPE_FILE_FILESYSTEM.
+ // http://crbug.com/141835
if (index_to_reader_.find(index) == index_to_reader_.end()) {
index_to_reader_[index] = new LocalFileStreamReader(
file_thread_proxy_,
« no previous file with comments | « webkit/blob/blob_storage_controller.cc ('k') | webkit/blob/view_blob_internals_job.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698