Made BookmarkService::GetBookmarks return both urls and title.

chrome/browser/bookmarks/bookmark_service.h

 // Copyright (c) 2010 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_BOOKMARKS_BOOKMARK_SERVICE_H_
 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_SERVICE_H_
 
 #include <vector>
 
-class GURL;
+#include "base/string16.h"
+#include "googleurl/src/gurl.h"
 
 // BookmarkService provides a thread safe view of bookmarks. It is used by
 // HistoryBackend when it needs to determine the set of bookmarked URLs
 // or if a URL is bookmarked.
 //
 // BookmarkService is owned by Profile and deleted when the Profile is deleted.
 class BookmarkService {
  public:
+  struct Bookmark {

[sky, 2012/08/03 16:04:39]

This name is too generic, how about URLAndTitle.

[michaelbai, 2012/08/03 17:57:49]

I was thinking it might not so generic as it is defined in BookmarkService. 

Changed it to URLAndTitle

On 2012/08/03 16:04:39, sky wrote:

+    string16 title;
+    GURL url;
+  };
+
   // Returns true if the specified URL is bookmarked.
   //
   // If not on the main thread you *must* invoke BlockTillLoaded first.
   virtual bool IsBookmarked(const GURL& url) = 0;
 
-  // Returns, by reference in urls, the set of bookmarked urls. This returns
-  // the unique set of URLs. For example, if two bookmarks reference the same
-  // URL only one entry is added.
+  // Returns, by reference in |bookmarks|, the set of bookmarked urls and their
+  // titles. This returns the unique set of URLs. For example, if two bookmarks
+  // reference the same URL only one entry is added not matter the titles are
+  // same or not.
   //
   // If not on the main thread you *must* invoke BlockTillLoaded first.
-  virtual void GetBookmarks(std::vector<GURL>* urls) = 0;
+  virtual void GetBookmarks(std::vector<Bookmark>* bookmarks) = 0;
 
   // Blocks until loaded. This is intended for usage on a thread other than
   // the main thread.
   virtual void BlockTillLoaded() = 0;
 
  protected:
   virtual ~BookmarkService() {}
 };
 
 #endif  // CHROME_BROWSER_BOOKMARKS_BOOKMARK_SERVICE_H_