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

Unified Diff: webkit/glue/multipart_response_delegate.cc

Issue 295041: DevTools: report correct content length for resources (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month 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: webkit/glue/multipart_response_delegate.cc
===================================================================
--- webkit/glue/multipart_response_delegate.cc (revision 30714)
+++ webkit/glue/multipart_response_delegate.cc (working copy)
@@ -81,10 +81,6 @@
if (stop_sending_)
return;
- // TODO(tc): Figure out what to use for length_received. Maybe we can just
- // pass the value on from our caller.
- int length_received = -1;
-
data_.append(data, data_len);
if (first_received_data_) {
// Some servers don't send a boundary token before the first chunk of
@@ -132,8 +128,7 @@
// Send the last data chunk.
client_->didReceiveData(loader_,
data_.data(),
- static_cast<int>(boundary_pos),
- length_received);
+ static_cast<int>(boundary_pos));
}
size_t boundary_end_pos = boundary_pos + boundary_.length();
if (boundary_end_pos < data_.length() && '-' == data_[boundary_end_pos]) {
@@ -159,13 +154,9 @@
// If we have any pending data and we're not in a header, go ahead and send
// it to WebCore.
if (!processing_headers_ && !data_.empty()) {
- // TODO(tc): Figure out what to use for length_received. Maybe we can just
- // pass the value on from our caller.
- int length_received = -1;
client_->didReceiveData(loader_,
data_.data(),
- static_cast<int>(data_.length()),
- length_received);
+ static_cast<int>(data_.length()));
}
}
« no previous file with comments | « webkit/glue/ftp_directory_listing_response_delegate.cc ('k') | webkit/glue/multipart_response_delegate_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698