Speculative fix for Mac quarantine crash

chrome/browser/download/download_file.cc

 // Copyright (c) 2006-2008 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_file.h"
 
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/stl_util-inl.h"
 #include "base/string_util.h"
@@ skipping 119 matching lines @@
 }
 
 bool DownloadFile::Open(const char* open_mode) {
   DCHECK(!full_path_.empty());
   file_ = file_util::OpenFile(full_path_, open_mode);
   if (!file_) {
     return false;
   }
 
 #if defined(OS_WIN)
+  AnnotateWithSourceInformation();
+#endif
+  return true;
+}
+
+void DownloadFile::AnnotateWithSourceInformation() {
+#if defined(OS_WIN)
   // Sets the Zone to tell Windows that this file comes from the internet.
   // We ignore the return value because a failure is not fatal.
   win_util::SetInternetZoneIdentifier(full_path_);
 #elif defined(OS_MACOSX)
   quarantine_mac::AddQuarantineMetadataToFile(full_path_, source_url_,
                                               referrer_url_);
 #endif
-  return true;
 }
 
 // DownloadFileManager implementation ------------------------------------------
 
 DownloadFileManager::DownloadFileManager(MessageLoop* ui_loop,
                                          ResourceDispatcherHost* rdh)
     : next_id_(0),
       ui_loop_(ui_loop),
       resource_dispatcher_host_(rdh) {
 }
@@ skipping 400 matching lines @@
                                               DownloadManager* manager) {
   DCHECK(MessageLoop::current() == file_loop_);
   DownloadFileMap::iterator it = downloads_.find(id);
   if (it == downloads_.end())
     return;
 
   file_util::CreateDirectory(full_path.DirName());
 
   DownloadFile* download = it->second;
   if (download->Rename(full_path)) {
+#if defined(OS_MACOSX)
+    // Done here because we only want to do this once; see
+    // http://crbug.com/13120 for details.
+    download->AnnotateWithSourceInformation();
+#endif
     ui_loop_->PostTask(FROM_HERE,
         NewRunnableMethod(manager,
                           &DownloadManager::DownloadRenamedToFinalName,
                           id,
                           full_path));
   } else {
     // Error. Between the time the UI thread generated 'full_path' to the time
     // this code runs, something happened that prevents us from renaming.
     DownloadManagerMap::iterator dmit = managers_.find(download->id());
     if (dmit != managers_.end()) {
@@ skipping 21 matching lines @@
     ui_loop_->PostTask(FROM_HERE, NewRunnableMethod(
         this, &DownloadFileManager::StopUpdateTimer));
 }
 
 // static
 void DownloadFileManager::DeleteFile(const FilePath& path) {
   // Make sure we only delete files.
   if (!file_util::DirectoryExists(path))
     file_util::Delete(path, false);
 }