Clear the received slices in DownloadItemImpl when etag changed.

content/browser/download/download_item_impl.cc

 // 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.
 
 // File method ordering: Methods in this file are in the same order as
 // in download_item_impl.h, with the following exception: The public
 // interface Start is placed in chronological order with the other
 // (private) routines that together define a DownloadItem's state
 // transitions as the download progresses.  See "Download progression
 // cascade" later in this file.
@@ skipping 1010 matching lines @@
   if (*chain_iter == url_chain_.back())
     ++chain_iter;
 
   // Record some stats. If the precondition failed (the server returned
   // HTTP_PRECONDITION_FAILED), then the download will automatically retried as
   // a full request rather than a partial. Full restarts clobber validators.
   int origin_state = 0;
   if (chain_iter != new_create_info.url_chain.end())
     origin_state |= ORIGIN_STATE_ON_RESUMPTION_ADDITIONAL_REDIRECTS;
   if (etag_ != new_create_info.etag ||
-      last_modified_time_ != new_create_info.last_modified)
+      last_modified_time_ != new_create_info.last_modified) {
+    received_slices_.clear();
+    received_bytes_ = 0;

[qinmin, 2017/04/07 23:37:43]

you either need to do this after RecordOriginStateOnResumption(), or you have to
add a new boolean here, otherwise RecordOriginStateOnResumption() will record
the wrong metric.

So either:
bool is_partial = (received_bytes_ != 0);
if (etag_!=.....) {
  received_slices_.clear();
  received_bytes_ = 0;
}
RecordOriginStateOnResumption(is_partial, origin_state);


Or: 
if (etag_!=.....) {
  received_slices_.clear();
}
RecordOriginStateOnResumption(received_bytes_ != 0, origin_state);
if (etag_!=.....) {
  received_bytes_ = 0;
}

[xingliu, 2017/04/08 02:29:13]

Oh, I see, thanks for the suggestion, didn't notice it uses the received_bytes_
here.

     origin_state |= ORIGIN_STATE_ON_RESUMPTION_VALIDATORS_CHANGED;
+  }
   if (content_disposition_ != new_create_info.content_disposition)
     origin_state |= ORIGIN_STATE_ON_RESUMPTION_CONTENT_DISPOSITION_CHANGED;
   RecordOriginStateOnResumption(received_bytes_ != 0, origin_state);
 
   url_chain_.insert(
       url_chain_.end(), chain_iter, new_create_info.url_chain.end());
   etag_ = new_create_info.etag;
   last_modified_time_ = new_create_info.last_modified;
   content_disposition_ = new_create_info.content_disposition;
   // It is possible that the previous download attempt failed right before the
@@ skipping 239 matching lines @@
   job_->Start();
 }
 
 void DownloadItemImpl::StartDownload() {
   BrowserThread::PostTask(
       BrowserThread::FILE, FROM_HERE,
       base::Bind(&DownloadFile::Initialize,
                  // Safe because we control download file lifetime.
                  base::Unretained(download_file_.get()),
                  base::Bind(&DownloadItemImpl::OnDownloadFileInitialized,
-                            weak_ptr_factory_.GetWeakPtr())));
+                            weak_ptr_factory_.GetWeakPtr()),
+                 received_slices_));
 }
 
 void DownloadItemImpl::OnDownloadFileInitialized(
     DownloadInterruptReason result) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   DCHECK(state_ == TARGET_PENDING_INTERNAL ||
          state_ == INTERRUPTED_TARGET_PENDING_INTERNAL)
       << "Unexpected state: " << DebugDownloadStateString(state_);
 
   DVLOG(20) << __func__
@@ skipping 937 matching lines @@
     case RESUME_MODE_USER_CONTINUE:
       return "USER_CONTINUE";
     case RESUME_MODE_USER_RESTART:
       return "USER_RESTART";
   }
   NOTREACHED() << "Unknown resume mode " << mode;
   return "unknown";
 }
 
 }  // namespace content