Move Windows Jump List boilerplate out of jumplist_win for reuse

chrome/browser/jumplist_win.h

 // 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.
 
 #ifndef CHROME_BROWSER_JUMPLIST_WIN_H_
 #define CHROME_BROWSER_JUMPLIST_WIN_H_
 
 #include <list>
 #include <string>
 #include <utility>
 #include <vector>
 
-#include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/synchronization/lock.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "chrome/browser/history/history_service.h"
 #include "chrome/browser/history/history_types.h"
+#include "chrome/browser/jumplist_updater_win.h"
 #include "chrome/browser/sessions/tab_restore_service.h"
 #include "chrome/browser/sessions/tab_restore_service_observer.h"
 #include "content/public/browser/browser_thread.h"
-#include "third_party/skia/include/core/SkBitmap.h"
 
 namespace base {
 class FilePath;

[tapted, 2014/02/18 06:12:43]

nit: this should be an include, rather than forward-dec since JumpList has a
(non-pointer) FilePath member.

[tmdiep, 2014/02/18 06:58:34]

Done.

 }
 
 namespace chrome {
 struct FaviconImageResult;
 }
 
 namespace content {
 class NotificationRegistrar;
 }
 
 class Profile;
 class PageUsageData;
 
-// Represents a class used for creating an IShellLink object by the utility
-// functions in this file.
-// This class consists of three strings and a integer.
-// * arguments (std::wstring)
-//   The arguments for the application.
-// * title (std::wstring)
-//   The string to be displayed in a JumpList.
-// * icon (std::wstring)
-//   The absolute path to an icon to be displayed in a JumpList.
-// * index (int)
-//   The icon index in the icon file. If an icon file consists of two or more
-//   icons, set this value to identify the icon. If an icon file consists of
-// one icon, this value is 0.
-// Even though an IShellLink also needs the absolute path to an application to
-// be executed, this class does not have any variables for it because our
-// utility functions always use "chrome.exe" as the application and we don't
-// need it.
-class ShellLinkItem : public base::RefCountedThreadSafe<ShellLinkItem> {
- public:
-  ShellLinkItem() : index_(0), favicon_(false) {
-  }
-
-  const std::wstring& arguments() const { return arguments_; }
-  const std::wstring& title() const { return title_; }
-  const std::wstring& icon() const { return icon_; }
-  int index() const { return index_; }
-  const SkBitmap& data() const { return data_; }
-
-  void SetArguments(const std::wstring& arguments) {
-    arguments_ = arguments;
-  }
-
-  void SetTitle(const std::wstring& title) {
-    title_ = title;
-  }
-
-  void SetIcon(const std::wstring& icon, int index, bool favicon) {
-    icon_ = icon;
-    index_ = index;
-    favicon_ = favicon;
-  }
-
-  void SetIconData(const SkBitmap& data) {
-    data_ = data;
-  }
-
- private:
-  friend class base::RefCountedThreadSafe<ShellLinkItem>;
-
-  ~ShellLinkItem() {}
-
-  std::wstring arguments_;
-  std::wstring title_;
-  std::wstring icon_;
-  SkBitmap data_;
-  int index_;
-  bool favicon_;
-
-  DISALLOW_COPY_AND_ASSIGN(ShellLinkItem);
-};
-
-typedef std::vector<scoped_refptr<ShellLinkItem> > ShellLinkItemList;
-
 // A class which implements an application JumpList.
 // This class encapsulates operations required for updating an application
 // JumpList:
 // * Retrieving "Most Visited" pages from HistoryService;
 // * Retrieving strings from the application resource;
 // * Creatng COM objects used by JumpList from PageUsageData objects;
 // * Adding COM objects to JumpList, etc.
 //
 // This class also implements TabRestoreServiceObserver. So, once we call
 // AddObserver() and register this class as an observer, it automatically
@@ skipping 138 matching lines @@
   base::CancelableTaskTracker::TaskId task_id_;
 
   // Lock for most_visited_pages_, recently_closed_pages_, icon_urls_
   // as they may be used by up to 3 threads.
   base::Lock list_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(JumpList);
 };
 
 #endif  // CHROME_BROWSER_JUMPLIST_WIN_H_