Rename the download to its final name only after the download is finished

chrome/browser/download/download_item.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/browser/download/download_item.h"
 
 #include "base/logging.h"
 #include "base/timer.h"
 #include "chrome/browser/download/download_manager.h"
 #include "chrome/browser/download/download_util.h"
@@ skipping 25 matching lines @@
       open_when_complete_(false),
       safety_state_(SAFE),
       auto_opened_(false),
       original_name_(info.original_name),
       render_process_id_(-1),
       request_id_(-1),
       save_as_(false),
       is_otr_(false),
       is_extension_install_(info.is_extension_install),
       name_finalized_(false),
-      is_temporary_(false) {
+      is_temporary_(false),
+      need_final_rename_(false) {
   if (state_ == IN_PROGRESS)
     state_ = CANCELLED;
   Init(false /* don't start progress timer */);
 }
 
 // Constructor for DownloadItem created via user action in the main thread.
 DownloadItem::DownloadItem(int32 download_id,
                            const FilePath& path,
                            int path_uniquifier,
                            const GURL& url,
@@ skipping 28 matching lines @@
       open_when_complete_(false),
       safety_state_(is_dangerous ? DANGEROUS : SAFE),
       auto_opened_(false),
       original_name_(original_name),
       render_process_id_(render_process_id),
       request_id_(request_id),
       save_as_(save_as),
       is_otr_(is_otr),
       is_extension_install_(is_extension_install),
       name_finalized_(false),
-      is_temporary_(is_temporary) {
+      is_temporary_(is_temporary),
+      need_final_rename_(false) {
   Init(true /* start progress timer */);
 }
 
 void DownloadItem::Init(bool start_timer) {
   file_name_ = full_path_.BaseName();
   if (start_timer)
     StartProgressTimer();
 }
 
 DownloadItem::~DownloadItem() {
@@ skipping 120 matching lines @@
 FilePath DownloadItem::GetFileName() const {
   if (safety_state_ == DownloadItem::SAFE)
     return file_name_;
   if (path_uniquifier_ > 0) {
     FilePath name(original_name_);
     download_util::AppendNumberToPath(&name, path_uniquifier_);
     return name;
   }
   return original_name_;
 }