diff options
| author | palmer <palmer@chromium.org> | 2015-06-04 20:10:40 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-06-05 03:11:29 +0000 |
| commit | ee0e5afcdb605346ac619b0eced7ab4db10abf18 (patch) | |
| tree | 73e536593685cd9b31441dee94caf6107c3868f6 /chrome/browser/notifications/platform_notification_service_impl.cc | |
| parent | 0ef24d4197503e228606a4523022d9e9f918202f (diff) | |
| download | chromium_src-ee0e5afcdb605346ac619b0eced7ab4db10abf18.zip chromium_src-ee0e5afcdb605346ac619b0eced7ab4db10abf18.tar.gz chromium_src-ee0e5afcdb605346ac619b0eced7ab4db10abf18.tar.bz2 | |
Audit calls to GURL::SchemeIs(url::kHttp*Scheme).
Check for correctness, safety, and generality. Now that we have
GURL::SchemeIsCryptographic, it may be a better callee in certain call sites.
BUG=486087
Review URL: https://codereview.chromium.org/1133593005
Cr-Commit-Position: refs/heads/master@{#333001}
Diffstat (limited to 'chrome/browser/notifications/platform_notification_service_impl.cc')
| -rw-r--r-- | chrome/browser/notifications/platform_notification_service_impl.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/chrome/browser/notifications/platform_notification_service_impl.cc b/chrome/browser/notifications/platform_notification_service_impl.cc index cafd270..d1fb2b2 100644 --- a/chrome/browser/notifications/platform_notification_service_impl.cc +++ b/chrome/browser/notifications/platform_notification_service_impl.cc @@ -392,6 +392,8 @@ base::string16 PlatformNotificationServiceImpl::DisplayNameForOrigin( } // static +// TODO(palmer): It might be good to replace this with a call to +// |FormatUrlForSecurityDisplay|. crbug.com/496965 base::string16 PlatformNotificationServiceImpl::WebOriginDisplayName( const GURL& origin, const std::string& languages) { |