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

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

Issue 2608553002: [OfflinePages] Improve visiblity/handling of "Loading not started" case (Closed)
Patch Set: Addes ! low-end device default to test Setup Created 3 years, 11 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 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
178 prerender_manager_->SetPrerenderContentsFactoryForTest( 178 prerender_manager_->SetPrerenderContentsFactoryForTest(
179 prerender_contents_factory_); 179 prerender_contents_factory_);
180 prerender_manager_->SetMode(PrerenderManager::PRERENDER_MODE_ENABLED); 180 prerender_manager_->SetMode(PrerenderManager::PRERENDER_MODE_ENABLED);
181 } 181 }
182 observer_stop_loading_called_ = false; 182 observer_stop_loading_called_ = false;
183 observer_dom_content_loaded_called_ = false; 183 observer_dom_content_loaded_called_ = false;
184 observer_stop_called_ = false; 184 observer_stop_called_ = false;
185 ASSERT_TRUE(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); 185 ASSERT_TRUE(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
186 } 186 }
187 187
188 TEST_F(PrerenderAdapterTest, CanPrerender) {
189 // Skip test on low end device until supported.
190 if (base::SysInfo::IsLowEndDevice())
191 return;
192
193 EXPECT_TRUE(adapter()->CanPrerender());
194
195 prerender_manager()->SetMode(PrerenderManager::PRERENDER_MODE_DISABLED);
196 EXPECT_FALSE(adapter()->CanPrerender());
197 }
198
199 TEST_F(PrerenderAdapterTest, StartPrerenderFailsForUnsupportedScheme) { 188 TEST_F(PrerenderAdapterTest, StartPrerenderFailsForUnsupportedScheme) {
200 // Skip test on low end device until supported. 189 // Skip test on low end device until supported.
201 if (base::SysInfo::IsLowEndDevice()) 190 if (base::SysInfo::IsLowEndDevice())
202 return; 191 return;
203 192
204 std::unique_ptr<content::WebContents> session_contents( 193 std::unique_ptr<content::WebContents> session_contents(
205 content::WebContents::Create( 194 content::WebContents::Create(
206 content::WebContents::CreateParams(profile()))); 195 content::WebContents::CreateParams(profile())));
207 content::SessionStorageNamespace* sessionStorageNamespace = 196 content::SessionStorageNamespace* sessionStorageNamespace =
208 session_contents->GetController().GetDefaultSessionStorageNamespace(); 197 session_contents->GetController().GetDefaultSessionStorageNamespace();
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 EXPECT_EQ(FinalStatus::FINAL_STATUS_CANCELLED, adapter()->GetFinalStatus()); 237 EXPECT_EQ(FinalStatus::FINAL_STATUS_CANCELLED, adapter()->GetFinalStatus());
249 238
250 // Exercise access methods even though no interesting values set beneath. 239 // Exercise access methods even though no interesting values set beneath.
251 EXPECT_EQ(nullptr, adapter()->GetWebContents()); 240 EXPECT_EQ(nullptr, adapter()->GetWebContents());
252 241
253 adapter()->DestroyActive(); 242 adapter()->DestroyActive();
254 EXPECT_FALSE(adapter()->IsActive()); 243 EXPECT_FALSE(adapter()->IsActive());
255 } 244 }
256 245
257 } // namespace offline_pages 246 } // namespace offline_pages
OLDNEW
« no previous file with comments | « chrome/browser/android/offline_pages/prerender_adapter.cc ('k') | chrome/browser/android/offline_pages/prerendering_loader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698