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

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

Issue 2044613003: Adds unit tests for offline_pages::PrerenderAdapter calling PrerenderManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 "chrome/browser/prerender/prerender_manager.h" 7 #include "chrome/browser/prerender/prerender_manager.h"
8 #include "chrome/browser/prerender/prerender_manager_factory.h" 8 #include "chrome/browser/prerender/prerender_manager_factory.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "content/public/browser/browser_context.h" 10 #include "content/public/browser/browser_context.h"
11 #include "content/public/browser/web_contents.h" 11 #include "content/public/browser/web_contents.h"
12 #include "ui/gfx/geometry/size.h" 12 #include "ui/gfx/geometry/size.h"
13 13
14 namespace offline_pages { 14 namespace offline_pages {
15 15
16 PrerenderAdapter::Observer::Observer() {} 16 PrerenderAdapter::Observer::Observer() {}
17 PrerenderAdapter::Observer::~Observer() {} 17 PrerenderAdapter::Observer::~Observer() {}
18 18
19 PrerenderAdapter::PrerenderAdapter(PrerenderAdapter::Observer* observer) 19 PrerenderAdapter::PrerenderAdapter(PrerenderAdapter::Observer* observer)
20 : observer_(observer) { 20 : observer_(observer), test_prerender_manager_(nullptr) {
21 DCHECK(observer); 21 DCHECK(observer);
22 } 22 }
23 23
24 PrerenderAdapter::~PrerenderAdapter() { 24 PrerenderAdapter::~PrerenderAdapter() {
25 if (IsActive()) 25 if (IsActive())
26 DestroyActive(); 26 DestroyActive();
27 } 27 }
28 28
29 bool PrerenderAdapter::CanPrerender() const { 29 bool PrerenderAdapter::CanPrerender() const {
30 return prerender::PrerenderManager::ActuallyPrerendering(); 30 return prerender::PrerenderManager::ActuallyPrerendering();
31 } 31 }
32 32
33 bool PrerenderAdapter::StartPrerender( 33 bool PrerenderAdapter::StartPrerender(
34 content::BrowserContext* browser_context, 34 content::BrowserContext* browser_context,
35 const GURL& url, 35 const GURL& url,
36 content::SessionStorageNamespace* session_storage_namespace, 36 content::SessionStorageNamespace* session_storage_namespace,
37 const gfx::Size& size) { 37 const gfx::Size& size) {
38 DCHECK(!IsActive()); 38 DCHECK(!IsActive());
39 DCHECK(CanPrerender()); 39 DCHECK(CanPrerender());
40 40
41 Profile* profile = Profile::FromBrowserContext(browser_context); 41 Profile* profile = Profile::FromBrowserContext(browser_context);
42 prerender::PrerenderManager* manager = 42 prerender::PrerenderManager* manager =
43 prerender::PrerenderManagerFactory::GetForProfile(profile); 43 prerender::PrerenderManagerFactory::GetForProfile(profile);
44 if (test_prerender_manager_) {
45 DVLOG(1) << "Using test PrerenderManager";
46 manager = test_prerender_manager_;
47 }
44 DCHECK(manager); 48 DCHECK(manager);
45 49
46 // Start prerendering the url and capture the handle for the prerendering. 50 // Start prerendering the url and capture the handle for the prerendering.
47 active_handle_.reset( 51 active_handle_.reset(
48 manager->AddPrerenderForOffline(url, session_storage_namespace, size)); 52 manager->AddPrerenderForOffline(url, session_storage_namespace, size));
49 if (!active_handle_) 53 if (!active_handle_)
50 return false; 54 return false;
51 55
52 active_handle_->SetObserver(this); 56 active_handle_->SetObserver(this);
53 return true; 57 return true;
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 prerender::PrerenderHandle* handle) { 97 prerender::PrerenderHandle* handle) {
94 DCHECK(active_handle_.get() == handle); 98 DCHECK(active_handle_.get() == handle);
95 observer_->OnPrerenderDomContentLoaded(); 99 observer_->OnPrerenderDomContentLoaded();
96 } 100 }
97 101
98 void PrerenderAdapter::OnPrerenderStop(prerender::PrerenderHandle* handle) { 102 void PrerenderAdapter::OnPrerenderStop(prerender::PrerenderHandle* handle) {
99 DCHECK(active_handle_.get() == handle); 103 DCHECK(active_handle_.get() == handle);
100 observer_->OnPrerenderStop(); 104 observer_->OnPrerenderStop();
101 } 105 }
102 106
107 void PrerenderAdapter::SetPrerenderManagerForTest(
108 prerender::PrerenderManager* test_prerender_manager) {
109 test_prerender_manager_ = test_prerender_manager;
110 }
111
103 } // namespace offline_pages 112 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698