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

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

Issue 2321083002: [NoStatePrefetch] Add performance histograms. (Closed)
Patch Set: Review comments Created 4 years, 3 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"
10 #include "content/public/browser/browser_context.h" 9 #include "content/public/browser/browser_context.h"
11 #include "content/public/browser/web_contents.h" 10 #include "content/public/browser/web_contents.h"
12 #include "ui/gfx/geometry/size.h" 11 #include "ui/gfx/geometry/size.h"
13 12
14 namespace offline_pages { 13 namespace offline_pages {
15 14
16 PrerenderAdapter::Observer::Observer() {} 15 PrerenderAdapter::Observer::Observer() {}
17 PrerenderAdapter::Observer::~Observer() {} 16 PrerenderAdapter::Observer::~Observer() {}
18 17
19 PrerenderAdapter::PrerenderAdapter(PrerenderAdapter::Observer* observer) 18 PrerenderAdapter::PrerenderAdapter(PrerenderAdapter::Observer* observer)
(...skipping 11 matching lines...) Expand all
31 } 30 }
32 31
33 bool PrerenderAdapter::StartPrerender( 32 bool PrerenderAdapter::StartPrerender(
34 content::BrowserContext* browser_context, 33 content::BrowserContext* browser_context,
35 const GURL& url, 34 const GURL& url,
36 content::SessionStorageNamespace* session_storage_namespace, 35 content::SessionStorageNamespace* session_storage_namespace,
37 const gfx::Size& size) { 36 const gfx::Size& size) {
38 DCHECK(!IsActive()); 37 DCHECK(!IsActive());
39 DCHECK(CanPrerender()); 38 DCHECK(CanPrerender());
40 39
41 Profile* profile = Profile::FromBrowserContext(browser_context);
42 prerender::PrerenderManager* manager = 40 prerender::PrerenderManager* manager =
43 prerender::PrerenderManagerFactory::GetForProfile(profile); 41 prerender::PrerenderManagerFactory::GetForBrowserContext(browser_context);
44 DCHECK(manager); 42 DCHECK(manager);
45 43
46 // Start prerendering the url and capture the handle for the prerendering. 44 // Start prerendering the url and capture the handle for the prerendering.
47 active_handle_ = 45 active_handle_ =
48 manager->AddPrerenderForOffline(url, session_storage_namespace, size); 46 manager->AddPrerenderForOffline(url, session_storage_namespace, size);
49 if (!active_handle_) 47 if (!active_handle_)
50 return false; 48 return false;
51 49
52 active_handle_->SetObserver(this); 50 active_handle_->SetObserver(this);
53 return true; 51 return true;
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 } 97 }
100 98
101 void PrerenderAdapter::OnPrerenderStop(prerender::PrerenderHandle* handle) { 99 void PrerenderAdapter::OnPrerenderStop(prerender::PrerenderHandle* handle) {
102 if (IsActive()) { 100 if (IsActive()) {
103 DCHECK_EQ(active_handle_.get(), handle); 101 DCHECK_EQ(active_handle_.get(), handle);
104 observer_->OnPrerenderStop(); 102 observer_->OnPrerenderStop();
105 } 103 }
106 } 104 }
107 105
108 } // namespace offline_pages 106 } // namespace offline_pages
OLDNEW
« no previous file with comments | « chrome/browser/BUILD.gn ('k') | chrome/browser/android/offline_pages/prerender_adapter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698