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 #include "chrome/browser/thumbnails/thumbnail_service_impl.h" | 5 #include "chrome/browser/thumbnails/thumbnail_service_impl.h" |
6 | 6 |
7 #include "base/memory/ref_counted.h" | 7 #include "base/memory/ref_counted.h" |
8 #include "chrome/browser/history/top_sites_impl.h" | 8 #include "chrome/browser/history/top_sites_impl.h" |
9 #include "chrome/browser/history/top_sites_factory.h" | |
9 #include "chrome/test/base/testing_profile.h" | 10 #include "chrome/test/base/testing_profile.h" |
10 #include "testing/gtest/include/gtest/gtest.h" | 11 #include "testing/gtest/include/gtest/gtest.h" |
11 | 12 |
12 typedef testing::Test ThumbnailServiceTest; | 13 typedef testing::Test ThumbnailServiceTest; |
13 | 14 |
15 namespace { | |
16 | |
17 scoped_refptr<RefcountedKeyedService> BuildMockTopSites( | |
18 content::BrowserContext* profile) { | |
19 return scoped_refptr<RefcountedKeyedService>( | |
20 new MockTopSites(static_cast<Profile*>(profile))); | |
21 } | |
22 | |
23 } // namespace | |
24 | |
14 // A mock version of TopSitesImpl, used for testing | 25 // A mock version of TopSitesImpl, used for testing |
15 // ShouldAcquirePageThumbnail(). | 26 // ShouldAcquirePageThumbnail(). |
16 class MockTopSites : public history::TopSitesImpl { | 27 class MockTopSites : public history::TopSitesImpl { |
17 public: | 28 public: |
18 explicit MockTopSites(Profile* profile) | 29 explicit MockTopSites(Profile* profile) |
19 : history::TopSitesImpl(profile), | 30 : history::TopSitesImpl(profile), capacity_(1) {} |
20 capacity_(1) { | |
21 } | |
22 | 31 |
23 // history::TopSitesImpl overrides. | 32 // history::TopSitesImpl overrides. |
24 bool IsNonForcedFull() override { return known_url_map_.size() >= capacity_; } | 33 bool IsNonForcedFull() override { return known_url_map_.size() >= capacity_; } |
25 bool IsForcedFull() override { return false; } | 34 bool IsForcedFull() override { return false; } |
26 bool IsKnownURL(const GURL& url) override { | 35 bool IsKnownURL(const GURL& url) override { |
27 return known_url_map_.find(url.spec()) != known_url_map_.end(); | 36 return known_url_map_.find(url.spec()) != known_url_map_.end(); |
28 } | 37 } |
29 bool GetPageThumbnailScore(const GURL& url, ThumbnailScore* score) override { | 38 bool GetPageThumbnailScore(const GURL& url, ThumbnailScore* score) override { |
30 std::map<std::string, ThumbnailScore>::const_iterator iter = | 39 std::map<std::string, ThumbnailScore>::const_iterator iter = |
31 known_url_map_.find(url.spec()); | 40 known_url_map_.find(url.spec()); |
(...skipping 15 matching lines...) Expand all Loading... | |
47 | 56 |
48 const size_t capacity_; | 57 const size_t capacity_; |
49 std::map<std::string, ThumbnailScore> known_url_map_; | 58 std::map<std::string, ThumbnailScore> known_url_map_; |
50 | 59 |
51 DISALLOW_COPY_AND_ASSIGN(MockTopSites); | 60 DISALLOW_COPY_AND_ASSIGN(MockTopSites); |
52 }; | 61 }; |
53 | 62 |
54 // A mock version of TestingProfile holds MockTopSites. | 63 // A mock version of TestingProfile holds MockTopSites. |
55 class MockProfile : public TestingProfile { | 64 class MockProfile : public TestingProfile { |
56 public: | 65 public: |
57 MockProfile() : mock_top_sites_(new MockTopSites(this)) { | 66 MockProfile() { |
67 TopSitesFactory::GetInstance()::SetTestingFactory(this, | |
Bernhard Bauer
2015/01/06 11:10:30
Looks like the indentation got messed up here?
| |
68 BuildMockTopSites); | |
58 } | 69 } |
59 | 70 |
60 history::TopSites* GetTopSites() override { return mock_top_sites_.get(); } | |
61 | |
62 void AddKnownURL(const GURL& url, const ThumbnailScore& score) { | 71 void AddKnownURL(const GURL& url, const ThumbnailScore& score) { |
63 mock_top_sites_->AddKnownURL(url, score); | 72 scoped_refptr<history::TopSites> top_sites = |
73 TopSitesFactory::GetForProfile(this); | |
74 top_sites->AddKnownURL(url, score); | |
64 } | 75 } |
65 | 76 |
66 private: | 77 private: |
67 scoped_refptr<MockTopSites> mock_top_sites_; | |
68 | 78 |
69 DISALLOW_COPY_AND_ASSIGN(MockProfile); | 79 DISALLOW_COPY_AND_ASSIGN(MockProfile); |
70 }; | 80 }; |
71 | 81 |
72 TEST_F(ThumbnailServiceTest, ShouldUpdateThumbnail) { | 82 TEST_F(ThumbnailServiceTest, ShouldUpdateThumbnail) { |
73 const GURL kGoodURL("http://www.google.com/"); | 83 const GURL kGoodURL("http://www.google.com/"); |
74 const GURL kBadURL("chrome://newtab"); | 84 const GURL kBadURL("chrome://newtab"); |
75 | 85 |
76 // Set up the mock profile along with mock top sites. | 86 // Set up the mock profile along with mock top sites. |
77 base::ScopedTempDir temp_dir; | 87 base::ScopedTempDir temp_dir; |
78 ASSERT_TRUE(temp_dir.CreateUniqueTempDir()); | 88 ASSERT_TRUE(temp_dir.CreateUniqueTempDir()); |
79 MockProfile profile; | 89 MockProfile profile; |
80 | 90 |
81 scoped_refptr<thumbnails::ThumbnailService> thumbnail_service( | 91 scoped_refptr<thumbnails::ThumbnailService> thumbnail_service( |
82 new thumbnails::ThumbnailServiceImpl(&profile)); | 92 new thumbnails::ThumbnailServiceImpl(&profile)); |
83 | 93 |
84 // Should be false because it's a bad URL. | 94 // Should be false because it's a bad URL. |
85 EXPECT_FALSE(thumbnail_service->ShouldAcquirePageThumbnail(kBadURL)); | 95 EXPECT_FALSE(thumbnail_service->ShouldAcquirePageThumbnail(kBadURL)); |
86 | 96 |
87 // Should be true, as it's a good URL. | 97 // Should be true, as it's a good URL. |
88 EXPECT_TRUE(thumbnail_service->ShouldAcquirePageThumbnail(kGoodURL)); | 98 EXPECT_TRUE(thumbnail_service->ShouldAcquirePageThumbnail(kGoodURL)); |
89 | 99 |
90 // Not checking incognito mode since the service wouldn't have been created | 100 // Not checking incognito mode since the service wouldn't have been created |
91 // in that case anyway. | 101 // in that case anyway. |
92 | 102 |
93 // Add a known URL. This makes the top sites data full. | 103 // Add a known URL. This makes the top sites data full. |
94 ThumbnailScore bad_score; | 104 ThumbnailScore bad_score; |
95 bad_score.time_at_snapshot = base::Time::UnixEpoch(); // Ancient time stamp. | 105 bad_score.time_at_snapshot = base::Time::UnixEpoch(); // Ancient time stamp. |
96 profile.AddKnownURL(kGoodURL, bad_score); | 106 profile.AddKnownURL(kGoodURL, bad_score); |
97 ASSERT_TRUE(profile.GetTopSites()->IsNonForcedFull()); | 107 scoped_refptr<history::TopSites> top_sites = |
108 TopSitesFactory::GetForProfile(profile); | |
109 ASSERT_TRUE(top_sites->IsNonForcedFull()); | |
98 | 110 |
99 // Should be false, as the top sites data is full, and the new URL is | 111 // Should be false, as the top sites data is full, and the new URL is |
100 // not known. | 112 // not known. |
101 const GURL kAnotherGoodURL("http://www.youtube.com/"); | 113 const GURL kAnotherGoodURL("http://www.youtube.com/"); |
102 EXPECT_FALSE(thumbnail_service->ShouldAcquirePageThumbnail(kAnotherGoodURL)); | 114 EXPECT_FALSE(thumbnail_service->ShouldAcquirePageThumbnail(kAnotherGoodURL)); |
103 | 115 |
104 // Should be true, as the existing thumbnail is bad (i.e. need a better one). | 116 // Should be true, as the existing thumbnail is bad (i.e. need a better one). |
105 EXPECT_TRUE(thumbnail_service->ShouldAcquirePageThumbnail(kGoodURL)); | 117 EXPECT_TRUE(thumbnail_service->ShouldAcquirePageThumbnail(kGoodURL)); |
106 | 118 |
107 // Replace the thumbnail score with a really good one. | 119 // Replace the thumbnail score with a really good one. |
108 ThumbnailScore good_score; | 120 ThumbnailScore good_score; |
109 good_score.time_at_snapshot = base::Time::Now(); // Very new. | 121 good_score.time_at_snapshot = base::Time::Now(); // Very new. |
110 good_score.at_top = true; | 122 good_score.at_top = true; |
111 good_score.good_clipping = true; | 123 good_score.good_clipping = true; |
112 good_score.boring_score = 0.0; | 124 good_score.boring_score = 0.0; |
113 good_score.load_completed = true; | 125 good_score.load_completed = true; |
114 profile.AddKnownURL(kGoodURL, good_score); | 126 profile.AddKnownURL(kGoodURL, good_score); |
115 | 127 |
116 // Should be false, as the existing thumbnail is good enough (i.e. don't | 128 // Should be false, as the existing thumbnail is good enough (i.e. don't |
117 // need to replace the existing thumbnail which is new and good). | 129 // need to replace the existing thumbnail which is new and good). |
118 EXPECT_FALSE(thumbnail_service->ShouldAcquirePageThumbnail(kGoodURL)); | 130 EXPECT_FALSE(thumbnail_service->ShouldAcquirePageThumbnail(kGoodURL)); |
119 } | 131 } |
OLD | NEW |