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

Side by Side Diff: chrome/browser/signin/signin_header_helper.cc

Issue 1131493004: Switch //chrome functions to use SchemeIsCryptographic() instead of SchemeIsSecure(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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/signin/signin_header_helper.h" 5 #include "chrome/browser/signin/signin_header_helper.h"
6 6
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "base/strings/stringprintf.h" 9 #include "base/strings/stringprintf.h"
10 #include "chrome/browser/prefs/incognito_mode_prefs.h" 10 #include "chrome/browser/prefs/incognito_mode_prefs.h"
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 } else { 166 } else {
167 AccountManagementScreenHelper::OpenAccountManagementScreen( 167 AccountManagementScreenHelper::OpenAccountManagementScreen(
168 Profile::FromBrowserContext(web_contents->GetBrowserContext()), 168 Profile::FromBrowserContext(web_contents->GetBrowserContext()),
169 service_type); 169 service_type);
170 } 170 }
171 #endif // OS_ANDROID 171 #endif // OS_ANDROID
172 } 172 }
173 #endif // !defined(OS_IOS) 173 #endif // !defined(OS_IOS)
174 174
175 bool IsDriveOrigin(const GURL& url) { 175 bool IsDriveOrigin(const GURL& url) {
176 if (!url.SchemeIsSecure()) 176 if (!url.SchemeIsCryptographic())
lgarron 2015/05/07 22:53:45 This seems a bit redundant, considering we're dire
177 return false; 177 return false;
178 178
179 const GURL kGoogleDriveURL("https://drive.google.com"); 179 const GURL kGoogleDriveURL("https://drive.google.com");
180 const GURL kGoogleDocsURL("https://docs.google.com"); 180 const GURL kGoogleDocsURL("https://docs.google.com");
181 return url == kGoogleDriveURL || url == kGoogleDocsURL; 181 return url == kGoogleDriveURL || url == kGoogleDocsURL;
182 } 182 }
183 183
184 } // empty namespace 184 } // empty namespace
185 185
186 namespace signin { 186 namespace signin {
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 301
302 params.child_id = child_id; 302 params.child_id = child_id;
303 params.route_id = route_id; 303 params.route_id = route_id;
304 content::BrowserThread::PostTask( 304 content::BrowserThread::PostTask(
305 content::BrowserThread::UI, FROM_HERE, 305 content::BrowserThread::UI, FROM_HERE,
306 base::Bind(ProcessMirrorHeaderUIThread, child_id, route_id, params)); 306 base::Bind(ProcessMirrorHeaderUIThread, child_id, route_id, params));
307 #endif // defined(OS_IOS) 307 #endif // defined(OS_IOS)
308 } 308 }
309 309
310 } // namespace signin 310 } // namespace signin
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698