Reset is_completed_ before attempting a new crx component download.

When downloading a url, the state of the BackgroundDownloader must be
reset.

A better solution later on would be implementing a BitsFetcher that can
only do one download, similar to UrlFetcher. This avoids the situation
when dirty state post-download is carried over to a subsequent
download.

BUG=304354

Review URL: https://codereview.chromium.org/102993014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242108 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/component_updater/background_downloader_win.cc

@@ -389,6 +389,10 @@ void BackgroundDownloader::BeginDownload(const GURL& url) {
 
   DCHECK(!timer_);
 
+  is_completed_ = false;
+  download_start_time_ = base::Time::Now();
+  job_stuck_begin_time_ = download_start_time_;
+
   HRESULT hr = QueueBitsJob(url);
   if (FAILED(hr)) {
     EndDownload(hr);
@@ -402,9 +406,6 @@ void BackgroundDownloader::BeginDownload(const GURL& url) {
                 base::TimeDelta::FromSeconds(kJobPollingIntervalSec),
                 this,
                 &BackgroundDownloader::OnDownloading);
-
-  download_start_time_ = base::Time::Now();
-  job_stuck_begin_time_ = download_start_time_;
 }
 
 // Called any time the timer fires.