Chromium Code Reviews Chromium Code Reviews
Issue 815983002:
Topsites become keyedService based. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #ifndef CHROME_BROWSER_UI_TABS_TAB_STRIP_MODEL_UTILS_H_ | 5 #ifndef CHROME_BROWSER_UI_TABS_TAB_STRIP_MODEL_UTILS_H_ |
| 6 #define CHROME_BROWSER_UI_TABS_TAB_STRIP_MODEL_UTILS_H_ | 6 #define CHROME_BROWSER_UI_TABS_TAB_STRIP_MODEL_UTILS_H_ |
| 7 | 7 |
| 8 #include <set> | 8 #include <set> |
| 9 #include <string> | 9 #include <string> |
| 10 | 10 |
| 11 class TabStripModel; | 11 class TabStripModel; |
| 12 | 12 |
| 13 namespace history { | 13 namespace history { |
| 14 class TopSites; | 14 class TopSites; |
| 15 } | 15 } |
| 16 | 16 |
| 17 namespace chrome { | 17 namespace chrome { |
| 18 | 18 |
| 19 // Creates a set containing the canonical URLs of the currently open tabs. | 19 // Creates a set containing the canonical URLs of the currently open tabs. |
| 20 void GetOpenUrls(const TabStripModel& tabs, | 20 void GetOpenUrls(const TabStripModel& tabs, |
| 21 const history::TopSites& top_sites, | 21 history::TopSites& top_sites, |
|
Bernhard Bauer
2015/01/12 14:32:23
This should be a const reference. The version on t
Jitu( very slow this week)
2015/01/12 14:55:15
Done.
| |
| 22 std::set<std::string>* urls); | 22 std::set<std::string>* urls); |
| 23 | 23 |
| 24 } // namespace chrome | 24 } // namespace chrome |
| 25 | 25 |
| 26 #endif // CHROME_BROWSER_UI_TABS_TAB_STRIP_MODEL_UTILS_H_ | 26 #endif // CHROME_BROWSER_UI_TABS_TAB_STRIP_MODEL_UTILS_H_ |
| OLD | NEW |
