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

Side by Side Diff: chrome/browser/prerender/external_prerender_handler_android.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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/prerender/external_prerender_handler_android.h" 5 #include "chrome/browser/prerender/external_prerender_handler_android.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/android/jni_string.h" 8 #include "base/android/jni_string.h"
9 #include "base/strings/string16.h" 9 #include "base/strings/string16.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 22 matching lines...) Expand all
33 PrerenderManager** prerender_manager, 33 PrerenderManager** prerender_manager,
34 content::WebContents** web_contents) { 34 content::WebContents** web_contents) {
35 if (!jurl) 35 if (!jurl)
36 return false; 36 return false;
37 37
38 *url = GURL(ConvertJavaStringToUTF16(env, jurl)); 38 *url = GURL(ConvertJavaStringToUTF16(env, jurl));
39 if (!url->is_valid()) 39 if (!url->is_valid())
40 return false; 40 return false;
41 41
42 Profile* profile = ProfileAndroid::FromProfileAndroid(jprofile); 42 Profile* profile = ProfileAndroid::FromProfileAndroid(jprofile);
43 *prerender_manager = PrerenderManagerFactory::GetForProfile(profile); 43 *prerender_manager = PrerenderManagerFactory::GetForBrowserContext(profile);
44 if (!*prerender_manager) 44 if (!*prerender_manager)
45 return false; 45 return false;
46 46
47 *web_contents = content::WebContents::FromJavaWebContents(jweb_contents); 47 *web_contents = content::WebContents::FromJavaWebContents(jweb_contents);
48 return true; 48 return true;
49 } 49 }
50 50
51 } // namespace 51 } // namespace
52 52
53 bool ExternalPrerenderHandlerAndroid::AddPrerender( 53 bool ExternalPrerenderHandlerAndroid::AddPrerender(
(...skipping 17 matching lines...) Expand all
71 content::Referrer referrer; 71 content::Referrer referrer;
72 if (!jreferrer.is_null()) { 72 if (!jreferrer.is_null()) {
73 GURL referrer_url(ConvertJavaStringToUTF16(env, jreferrer)); 73 GURL referrer_url(ConvertJavaStringToUTF16(env, jreferrer));
74 if (referrer_url.is_valid()) { 74 if (referrer_url.is_valid()) {
75 referrer = 75 referrer =
76 content::Referrer(referrer_url, blink::WebReferrerPolicyDefault); 76 content::Referrer(referrer_url, blink::WebReferrerPolicyDefault);
77 } 77 }
78 } 78 }
79 79
80 PrerenderManager* prerender_manager = 80 PrerenderManager* prerender_manager =
81 PrerenderManagerFactory::GetForProfile(profile); 81 PrerenderManagerFactory::GetForBrowserContext(profile);
82 if (!prerender_manager) 82 if (!prerender_manager)
83 return false; 83 return false;
84 84
85 content::WebContents* web_contents = 85 content::WebContents* web_contents =
86 content::WebContents::FromJavaWebContents(jweb_contents); 86 content::WebContents::FromJavaWebContents(jweb_contents);
87 if (prerender_handle_) 87 if (prerender_handle_)
88 prerender_handle_->OnNavigateAway(); 88 prerender_handle_->OnNavigateAway();
89 89
90 if (prerender_on_cellular && net::NetworkChangeNotifier::IsConnectionCellular( 90 if (prerender_on_cellular && net::NetworkChangeNotifier::IsConnectionCellular(
91 net::NetworkChangeNotifier::GetConnectionType())) { 91 net::NetworkChangeNotifier::GetConnectionType())) {
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 new ExternalPrerenderHandlerAndroid(); 155 new ExternalPrerenderHandlerAndroid();
156 return reinterpret_cast<intptr_t>(external_handler); 156 return reinterpret_cast<intptr_t>(external_handler);
157 } 157 }
158 158
159 bool ExternalPrerenderHandlerAndroid::RegisterExternalPrerenderHandlerAndroid( 159 bool ExternalPrerenderHandlerAndroid::RegisterExternalPrerenderHandlerAndroid(
160 JNIEnv* env) { 160 JNIEnv* env) {
161 return RegisterNativesImpl(env); 161 return RegisterNativesImpl(env);
162 } 162 }
163 163
164 } // namespace prerender 164 } // namespace prerender
OLDNEW
« no previous file with comments | « chrome/browser/predictors/autocomplete_action_predictor.cc ('k') | chrome/browser/prerender/prerender_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698