[Android WebView] Fix LoadsImagesAutomatically tests after crbug.com/224317

android_webview/renderer/aw_render_view_ext.cc

Index: android_webview/renderer/aw_render_view_ext.cc
diff --git a/android_webview/renderer/aw_render_view_ext.cc b/android_webview/renderer/aw_render_view_ext.cc
index 8452b458f05621bfd606c004db669942927a94ac..08e1395b22fff06eb005dd6b39cef58f128437e3 100644
--- a/android_webview/renderer/aw_render_view_ext.cc
+++ b/android_webview/renderer/aw_render_view_ext.cc
@@ -167,21 +167,6 @@ void AwRenderViewExt::OnDocumentHasImagesRequest(int id) {
                                                    hasImages));
 }
 
-bool AwRenderViewExt::allowImage(WebKit::WebFrame* frame,
-                                 bool enabled_per_settings,
-                                 const WebKit::WebURL& image_url) {

[abarth-chromium, 2013/06/13 18:20:10]

Oh, it's great that you don't need this override anymore.

[joth, 2013/06/14 00:25:25]

Hmmm I looked again, and I'm not sure we don't?
This functionality is used by setBlockNetworkImage which was not the subject of
this CL, is still supported (AFAIK) and is covered by the still existing (but
alas, disaled)  
testBlockNetworkImagesBlocksNetworkImageAndReloadInPlace test.

mnaganov - can you confirm this is not needed for AwSettings.setImagesEnabled()
?

-  // Implementing setBlockNetworkImages, so allow local scheme images to be
-  // loaded.
-  if (enabled_per_settings)
-    return true;
-
-  // For compatibility, only blacklist network schemes instead of whitelisting.
-  const GURL url(image_url);
-  return !(url.SchemeIs(chrome::kHttpScheme) ||
-           url.SchemeIs(chrome::kHttpsScheme) ||
-           url.SchemeIs(chrome::kFtpScheme));
-}
-
 void AwRenderViewExt::DidCommitProvisionalLoad(WebKit::WebFrame* frame,
                                                bool is_new_navigation) {
   content::DocumentState* document_state =