Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1076)

Side by Side Diff: chrome/browser/android/offline_pages/prerender_adapter_unittest.cc

Issue 2711693002: [Offline Pages] Make prerenderer notify us of network progress. (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
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 "chrome/browser/android/offline_pages/prerender_adapter.h" 5 #include "chrome/browser/android/offline_pages/prerender_adapter.h"
6 6
7 #include "base/sys_info.h" 7 #include "base/sys_info.h"
8 #include "chrome/browser/prerender/prerender_manager_factory.h" 8 #include "chrome/browser/prerender/prerender_manager_factory.h"
9 #include "chrome/test/base/testing_profile.h" 9 #include "chrome/test/base/testing_profile.h"
10 #include "content/public/browser/browser_thread.h" 10 #include "content/public/browser/browser_thread.h"
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 class PrerenderAdapterTest : public testing::Test, 106 class PrerenderAdapterTest : public testing::Test,
107 public PrerenderAdapter::Observer { 107 public PrerenderAdapter::Observer {
108 public: 108 public:
109 PrerenderAdapterTest(); 109 PrerenderAdapterTest();
110 ~PrerenderAdapterTest() override; 110 ~PrerenderAdapterTest() override;
111 111
112 // PrerenderAdapter::Observer implementation: 112 // PrerenderAdapter::Observer implementation:
113 void OnPrerenderStopLoading() override; 113 void OnPrerenderStopLoading() override;
114 void OnPrerenderDomContentLoaded() override; 114 void OnPrerenderDomContentLoaded() override;
115 void OnPrerenderStop() override; 115 void OnPrerenderStop() override;
116 void OnPrerenderNetworkBytesChanged(int64_t bytes) override;
116 117
117 void SetUp() override; 118 void SetUp() override;
118 119
119 // Returns the PrerenderLoader to test. 120 // Returns the PrerenderLoader to test.
120 PrerenderAdapter* adapter() const { return adapter_.get(); } 121 PrerenderAdapter* adapter() const { return adapter_.get(); }
121 StubPrerenderContentsFactory* prerender_contents_factory() { 122 StubPrerenderContentsFactory* prerender_contents_factory() {
122 return prerender_contents_factory_; 123 return prerender_contents_factory_;
123 } 124 }
124 Profile* profile() { return &profile_; } 125 Profile* profile() { return &profile_; }
125 PrerenderManager* prerender_manager() { return prerender_manager_; } 126 PrerenderManager* prerender_manager() { return prerender_manager_; }
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 } 162 }
162 163
163 void PrerenderAdapterTest::OnPrerenderDomContentLoaded() { 164 void PrerenderAdapterTest::OnPrerenderDomContentLoaded() {
164 observer_dom_content_loaded_called_ = true; 165 observer_dom_content_loaded_called_ = true;
165 } 166 }
166 167
167 void PrerenderAdapterTest::OnPrerenderStop() { 168 void PrerenderAdapterTest::OnPrerenderStop() {
168 observer_stop_called_ = true; 169 observer_stop_called_ = true;
169 } 170 }
170 171
172 void PrerenderAdapterTest::OnPrerenderNetworkBytesChanged(int64_t bytes) {}
Pete Williamson 2017/02/22 18:46:30 Might be good to add an actual test here.
Dmitry Titov 2017/02/23 04:27:54 Done.
173
171 void PrerenderAdapterTest::SetUp() { 174 void PrerenderAdapterTest::SetUp() {
172 if (base::SysInfo::IsLowEndDevice()) 175 if (base::SysInfo::IsLowEndDevice())
173 return; 176 return;
174 adapter_.reset(new PrerenderAdapter(this)); 177 adapter_.reset(new PrerenderAdapter(this));
175 prerender_contents_factory_ = new StubPrerenderContentsFactory(); 178 prerender_contents_factory_ = new StubPrerenderContentsFactory();
176 prerender_manager_ = PrerenderManagerFactory::GetForBrowserContext(profile()); 179 prerender_manager_ = PrerenderManagerFactory::GetForBrowserContext(profile());
177 if (prerender_manager_) { 180 if (prerender_manager_) {
178 prerender_manager_->SetPrerenderContentsFactoryForTest( 181 prerender_manager_->SetPrerenderContentsFactoryForTest(
179 prerender_contents_factory_); 182 prerender_contents_factory_);
180 prerender_manager_->SetMode(PrerenderManager::PRERENDER_MODE_ENABLED); 183 prerender_manager_->SetMode(PrerenderManager::PRERENDER_MODE_ENABLED);
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
237 EXPECT_EQ(FinalStatus::FINAL_STATUS_CANCELLED, adapter()->GetFinalStatus()); 240 EXPECT_EQ(FinalStatus::FINAL_STATUS_CANCELLED, adapter()->GetFinalStatus());
238 241
239 // Exercise access methods even though no interesting values set beneath. 242 // Exercise access methods even though no interesting values set beneath.
240 EXPECT_EQ(nullptr, adapter()->GetWebContents()); 243 EXPECT_EQ(nullptr, adapter()->GetWebContents());
241 244
242 adapter()->DestroyActive(); 245 adapter()->DestroyActive();
243 EXPECT_FALSE(adapter()->IsActive()); 246 EXPECT_FALSE(adapter()->IsActive());
244 } 247 }
245 248
246 } // namespace offline_pages 249 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698