Download shelf autohides on showing in shell, just same as regular open

chrome/browser/download/download_item_model.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.
 
 #include "chrome/browser/download/download_item_model.h"
 
 #include "base/i18n/number_formatting.h"
 #include "base/i18n/rtl.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/string16.h"
@@ skipping 52 matching lines @@
   // for the file type.
   bool should_prefer_opening_in_browser_;
 
   // Danger level of the file determined based on the file type and whether
   // there was a user action associated with the download.
   DownloadFileType::DangerLevel danger_level_;
 
   // Whether the download is currently being revived.
   bool is_being_revived_;
 
+  // Whether download was opened or shown in the shell.
+  bool opened_or_shown_;
+
  private:
   DownloadItemModelData();
   ~DownloadItemModelData() override {}
 
   static const char kKey[];
 };
 
 // static
 const char DownloadItemModelData::kKey[] = "DownloadItemModelData key";
 
@@ skipping 13 matching lines @@
     download->SetUserData(kKey, data);
   }
   return data;
 }
 
 DownloadItemModelData::DownloadItemModelData()
     : should_show_in_shelf_(true),
       was_ui_notified_(false),
       should_prefer_opening_in_browser_(false),
       danger_level_(DownloadFileType::NOT_DANGEROUS),
-      is_being_revived_(false) {}
+      is_being_revived_(false),
+      opened_or_shown_(false) {}
 
 base::string16 InterruptReasonStatusMessage(
     content::DownloadInterruptReason reason) {
   int string_id = IDS_DOWNLOAD_INTERRUPTED_STATUS;
 
   switch (reason) {
     case content::DOWNLOAD_INTERRUPT_REASON_FILE_ACCESS_DENIED:
       string_id = IDS_DOWNLOAD_INTERRUPTED_STATUS_ACCESS_DENIED;
       break;
     case content::DOWNLOAD_INTERRUPT_REASON_FILE_NO_SPACE:
@@ skipping 535 matching lines @@
 bool DownloadItemModel::IsBeingRevived() const {
   const DownloadItemModelData* data = DownloadItemModelData::Get(download_);
   return data && data->is_being_revived_;
 }
 
 void DownloadItemModel::SetIsBeingRevived(bool is_being_revived) {
   DownloadItemModelData* data = DownloadItemModelData::GetOrCreate(download_);
   data->is_being_revived_ = is_being_revived;
 }
 
+bool DownloadItemModel::GetOpenedOrShown() const {
+  const DownloadItemModelData* data = DownloadItemModelData::Get(download_);
+  return download_->GetOpened() || (data && data->opened_or_shown_);
+}
+
+void DownloadItemModel::SetOpenedOrShown(bool opened_or_shown) {
+  DownloadItemModelData* data = DownloadItemModelData::GetOrCreate(download_);
+  data->opened_or_shown_ = opened_or_shown;
+}
+
 base::string16 DownloadItemModel::GetProgressSizesString() const {
   base::string16 size_ratio;
   int64_t size = GetCompletedBytes();
   int64_t total = GetTotalBytes();
   if (total > 0) {
     ui::DataUnits amount_units = ui::GetByteDisplayUnits(total);
     base::string16 simple_size = ui::FormatBytesWithUnits(size, amount_units, false);
 
     // In RTL locales, we render the text "size/total" in an RTL context. This
     // is problematic since a string such as "123/456 MB" is displayed
@@ skipping 70 matching lines @@
   if (!download_service)
     return;
 
   ChromeDownloadManagerDelegate* delegate =
       download_service->GetDownloadManagerDelegate();
   if (!delegate)
     return;
   delegate->OpenDownloadUsingPlatformHandler(download_);
   RecordDownloadOpenMethod(DOWNLOAD_OPEN_METHOD_USER_PLATFORM);
 }