Remove ui/gfx/screen_type_delegate.h

chrome/browser/banners/app_banner_data_fetcher.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/banners/app_banner_data_fetcher.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 324 matching lines @@
   OnHasServiceWorker(web_contents);
 }
 
 void AppBannerDataFetcher::OnHasServiceWorker(
     content::WebContents* web_contents) {
   GURL icon_url =
     ManifestIconSelector::FindBestMatchingIcon(
         web_app_data_.icons,
         ideal_icon_size_in_dp_,
         minimum_icon_size_in_dp_,
-        gfx::Screen::GetScreenFor(web_contents->GetNativeView()));
+        gfx::Screen::GetScreen());

[sky, 2016/01/19 22:47:54]

Can the Screen argument be removed from FindBestMatchingIcon?

[scottmg, 2016/01/20 00:52:57]

Done.

 
   if (!FetchAppIcon(web_contents, icon_url)) {
     OutputDeveloperNotShownMessage(web_contents, kCannotDetermineBestIcon);
     Cancel();
   }
 }
 
 bool AppBannerDataFetcher::FetchAppIcon(content::WebContents* web_contents,
                                         const GURL& icon_url) {
   return ManifestIconDownloader::Download(
@@ skipping 91 matching lines @@
     return false;
   }
   if (!DoesManifestContainRequiredIcon(manifest)) {
     OutputDeveloperNotShownMessage(web_contents, kManifestMissingSuitableIcon);
     return false;
   }
   return true;
 }
 
 }  // namespace banners