| OLD | NEW |
|---|
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 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 #include "components/ntp_tiles/popular_sites.h" | 5 #include "components/ntp_tiles/popular_sites_impl.h" |
| 6 | 6 |
| 7 #include <string> | 7 #include <string> |
| 8 #include <utility> | 8 #include <utility> |
| 9 #include <vector> | 9 #include <vector> |
| 10 | 10 |
| 11 #include "base/bind.h" | 11 #include "base/bind.h" |
| 12 #include "base/files/scoped_temp_dir.h" | 12 #include "base/files/scoped_temp_dir.h" |
| 13 #include "base/json/json_writer.h" | 13 #include "base/json/json_writer.h" |
| 14 #include "base/run_loop.h" | 14 #include "base/run_loop.h" |
| 15 #include "base/test/sequenced_worker_pool_owner.h" | 15 #include "base/test/sequenced_worker_pool_owner.h" |
| (...skipping 304 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 320 RespondWithJSON( | 320 RespondWithJSON( |
| 321 "https://www.gstatic.com/chrome/ntp/suggested_sites_ZZ_9.json", | 321 "https://www.gstatic.com/chrome/ntp/suggested_sites_ZZ_9.json", |
| 322 {kChromium}); | 322 {kChromium}); |
| 323 EXPECT_TRUE(FetchPopularSites(/*force_download=*/false, &sites)); | 323 EXPECT_TRUE(FetchPopularSites(/*force_download=*/false, &sites)); |
| 324 ASSERT_THAT(sites.size(), Eq(1u)); | 324 ASSERT_THAT(sites.size(), Eq(1u)); |
| 325 EXPECT_THAT(sites[0].url, URLEq("https://www.chromium.org/")); | 325 EXPECT_THAT(sites[0].url, URLEq("https://www.chromium.org/")); |
| 326 } | 326 } |
| 327 | 327 |
| 328 } // namespace | 328 } // namespace |
| 329 } // namespace ntp_tiles | 329 } // namespace ntp_tiles |
| OLD | NEW |
|---|
Chromium Code Reviews
Issue 2575823002:
[Popular Sites] Move PopularSitesImpl to dedicated file (Closed)
