Added a link to the downloads tab that opens the downloads folder.

chrome/browser/ui/webui/downloads_dom_handler.cc

 // Copyright (c) 2011 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/ui/webui/downloads_dom_handler.h"
 
 #include <algorithm>
 #include <functional>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/singleton.h"
 #include "base/string_piece.h"
 #include "base/threading/thread.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_history.h"
 #include "chrome/browser/download/download_item.h"
 #include "chrome/browser/download/download_util.h"
+#include "chrome/browser/download/download_prefs.h"

[asanka, 2011/07/19 18:58:00]

download_prefs.h should go before download_util.h

+#include "chrome/browser/platform_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/chrome_url_data_manager.h"
 #include "chrome/browser/ui/webui/fileicon_source.h"
 #include "chrome/browser/ui/webui/fileicon_source_cros.h"
 #include "chrome/common/jstemplate_builder.h"
+#include "chrome/common/pref_names.h"

[asanka, 2011/07/19 18:58:00]

Do you need this anymore?

 #include "chrome/common/url_constants.h"
 #include "content/browser/browser_thread.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/browser/user_metrics.h"
 #include "grit/generated_resources.h"
 #include "ui/gfx/image/image.h"
 
 namespace {
 
 // Maximum number of downloads to show. TODO(glen): Remove this and instead
@@ skipping 54 matching lines @@
   web_ui_->RegisterMessageCallback("togglepause",
       NewCallback(this, &DownloadsDOMHandler::HandlePause));
   web_ui_->RegisterMessageCallback("resume",
       NewCallback(this, &DownloadsDOMHandler::HandlePause));
   web_ui_->RegisterMessageCallback("remove",
       NewCallback(this, &DownloadsDOMHandler::HandleRemove));
   web_ui_->RegisterMessageCallback("cancel",
       NewCallback(this, &DownloadsDOMHandler::HandleCancel));
   web_ui_->RegisterMessageCallback("clearAll",
       NewCallback(this, &DownloadsDOMHandler::HandleClearAll));
+  web_ui_->RegisterMessageCallback("openDownloadsFolder",
+      NewCallback(this, &DownloadsDOMHandler::HandleOpenDownloadsFolder));
 }
 
 void DownloadsDOMHandler::OnDownloadUpdated(DownloadItem* download) {
   // Get the id for the download. Our downloads are sorted latest to first,
   // and the id is the index into that list. We should be careful of sync
   // errors between the UI and the download_items_ list (we may wish to use
   // something other than 'id').
   OrderedDownloads::iterator it = find(download_items_.begin(),
                                        download_items_.end(),
                                        download);
@@ skipping 110 matching lines @@
 void DownloadsDOMHandler::HandleCancel(const ListValue* args) {
   DownloadItem* file = GetDownloadByValue(args);
   if (file)
     file->Cancel(true);
 }
 
 void DownloadsDOMHandler::HandleClearAll(const ListValue* args) {
   download_manager_->RemoveAllDownloads();
 }
 
+void DownloadsDOMHandler::HandleOpenDownloadsFolder(const ListValue* args) {
+  FilePath path = download_manager_->download_prefs()->download_path();
+
+#if defined(OS_MACOSX)
+  // Must be called from the UI thread on Mac.
+  platform_util::OpenItem(path);
+#else
+  BrowserThread::PostTask(
+      BrowserThread::FILE, FROM_HERE,
+      NewRunnableFunction(&platform_util::OpenItem, path));
+#endif
+}
+
 // DownloadsDOMHandler, private: ----------------------------------------------
 
 void DownloadsDOMHandler::SendCurrentDownloads() {
   ListValue results_value;
   for (OrderedDownloads::iterator it = download_items_.begin();
       it != download_items_.end(); ++it) {
     int index = static_cast<int>(it - download_items_.begin());
     if (index > kMaxDownloads)
       break;
     if (!*it)
@@ skipping 26 matching lines @@
   return NULL;
 }
 
 DownloadItem* DownloadsDOMHandler::GetDownloadByValue(const ListValue* args) {
   int id;
   if (ExtractIntegerValue(args, &id)) {
     return GetDownloadById(id);
   }
   return NULL;
 }