Add functionality to the Windows 8 notification display functionality to invoke a caller specified …

chrome/browser/download/download_status_updater_win.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_status_updater.h"
 
 #include <string>
 #include <shobjidl.h>
 
+#include "base/file_path.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/string_number_conversions.h"
 #include "base/win/metro.h"
 #include "base/win/scoped_comptr.h"
 #include "base/win/windows_version.h"
+#include "chrome/browser/platform_util.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
+#include "content/public/browser/browser_thread.h"
 #include "googleurl/src/gurl.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 // This code doesn't compile with Aura on. TODO(avi): hook it up so that
 // win_aura can do platform integration.
 #if !defined(USE_AURA)
 
 namespace {
 
@@ skipping 31 matching lines @@
     HWND frame = window->GetNativeWindow();
     if (download_count == 0 || progress == 1.0f)
       taskbar->SetProgressState(frame, TBPF_NOPROGRESS);
     else if (!progress_known)
       taskbar->SetProgressState(frame, TBPF_INDETERMINATE);
     else
       taskbar->SetProgressValue(frame, static_cast<int>(progress * 100), 100);
   }
 }
 
+void MetroDownloadNotificationClickedHandler(const wchar_t* download_path) {
+  // Metro chrome will invoke these handlers on the metro thread.
+  DCHECK(!content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+
+  // Ensure that we invoke the function to display the downloaded item on the
+  // UI thread.
+  content::BrowserThread::PostTask(
+      content::BrowserThread::UI, FROM_HERE,
+      base::Bind(platform_util::ShowItemInFolder, FilePath(download_path)));
+}
+
 }  // namespace
 
 void DownloadStatusUpdater::UpdateAppIconDownloadProgress(
     content::DownloadItem* download) {
 
   // Always update overall progress.
-
   float progress = 0;
   int download_count = 0;
   bool progress_known = GetProgress(&progress, &download_count);
   UpdateTaskbarProgressBar(download_count, progress_known, progress);
 
   // Fire notifications when downloads complete.
-
   if (!base::win::IsMetroProcess())
     return;
 
   if (download->GetState() != content::DownloadItem::COMPLETE)
     return;
 
   if (download->GetOpenWhenComplete() ||
       download->ShouldOpenFileBasedOnExtension() ||
       download->IsTemporary() ||
       download->GetAutoOpened())
@@ skipping 15 matching lines @@
     // Dummy notification id. Every metro style notification needs a
     // unique notification id.
     std::string notification_id = kDownloadNotificationPrefix;
     notification_id += base::IntToString(g_next_notification_id++);
 
     display_notification(download->GetURL().spec().c_str(),
                          "",
                          title.c_str(),
                          body.c_str(),
                          L"",
-                         notification_id.c_str());
+                         notification_id.c_str(),
+                         MetroDownloadNotificationClickedHandler,
+                         download->GetFullPath().value().c_str());

[asanka, 2012/10/09 21:34:15]

Nit: download->GetTargetFilePath() is preferred.

For a completed download, GetFullPath() and GetTargetFilePath() is the same, but
the canonical method is GetTargetFilePath().

[ananta, 2012/10/09 21:58:34]

Done.

   }
 }
 
 #endif  // !USE_AURA