Remove AppBannerManager code to skip secure context checks for tests.

chrome/browser/banners/app_banner_manager.cc

Index: chrome/browser/banners/app_banner_manager.cc
diff --git a/chrome/browser/banners/app_banner_manager.cc b/chrome/browser/banners/app_banner_manager.cc
index a46c9dd012bfe2d0e632f0e29f8051c6811b7250..f866b23960b0e2edacd720d1ebeb9ce00217c418 100644
--- a/chrome/browser/banners/app_banner_manager.cc
+++ b/chrome/browser/banners/app_banner_manager.cc
@@ -63,11 +63,6 @@ InstallableParams ParamsToPerformInstallableCheck(int ideal_icon_size_in_dp,
 namespace banners {
 
 // static
-void AppBannerManager::DisableSecureSchemeCheckForTesting() {
-  gDisableSecureCheckForTesting = true;
-}
-
-// static
 base::Time AppBannerManager::GetCurrentTime() {
   return base::Time::Now() + gTimeDeltaForTesting.Get();
 }
@@ -119,6 +114,7 @@ void AppBannerManager::RequestAppBanner(const GURL& validated_url,
 
   // A secure origin is required to show banners, so exit early if we see the
   // URL is invalid.
+  LOG(ERROR) << validated_url;

[dominickn, 2016/10/25 03:13:39]

Remove this log spam.

   if (!content::IsOriginSecure(validated_url) &&
       !gDisableSecureCheckForTesting) {

[dominickn, 2016/10/25 03:13:39]

Remove the check for the global here. It's useless if you remove the static
setter.

     ReportStatus(contents, NOT_FROM_SECURE_ORIGIN);