Delegating the "are images allowed" decision to renderer.

chrome/renderer/content_settings_observer.cc

 // Copyright (c) 2011 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/renderer/content_settings_observer.h"
 
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "content/common/database_messages.h"
 #include "content/common/view_messages.h"
@@ skipping 35 matching lines @@
     }
   }
 
   return false;
 }
 
 }  // namespace
 
 ContentSettings ContentSettingsObserver::default_settings_;
 
-ContentSettingsObserver::ContentSettingsObserver(RenderView* render_view)
+ContentSettingsObserver::ContentSettingsObserver(
+    RenderView* render_view, ContentSettingRules* image_setting_rules)
     : RenderViewObserver(render_view),
       RenderViewObserverTracker<ContentSettingsObserver>(render_view),
+      image_setting_rules_(image_setting_rules),
       plugins_temporarily_allowed_(false) {
   ClearBlockedContentSettings();
 }
 
 ContentSettingsObserver::~ContentSettingsObserver() {
 }
 
 void ContentSettingsObserver::SetContentSettings(
     const ContentSettings& settings) {
   current_content_settings_ = settings;
@@ skipping 120 matching lines @@
 
   bool result = false;
   Send(new ChromeViewHostMsg_AllowFileSystem(
       routing_id(), GURL(frame->document().securityOrigin().toString()),
       GURL(frame->top()->document().securityOrigin().toString()), &result));
   return result;
 }
 
 bool ContentSettingsObserver::AllowImages(WebFrame* frame,
                                           bool enabled_per_settings) {
-  if (enabled_per_settings &&
-      AllowContentType(CONTENT_SETTINGS_TYPE_IMAGES)) {
-    return true;
-  }
-
   if (IsWhitelistedForContentSettings(frame))
     return true;
 
+  if (frame->document().securityOrigin().isEmpty() ||
+      frame->top()->document().securityOrigin().isEmpty()) {
+    DidBlockContentType(CONTENT_SETTINGS_TYPE_IMAGES, std::string());
+    return false;  // Uninitialized document.
+  }
+
+  if (enabled_per_settings) {
+    ContentSettingRules::const_iterator it;
+    for (it = image_setting_rules_->begin();
+         it != image_setting_rules_->end(); ++it) {
+      if (it->a.Matches(GURL(frame->document().securityOrigin().toString())) &&
+          it->b.Matches(
+              GURL(frame->top()->document().securityOrigin().toString()))) {
+        bool allow = (it->c != CONTENT_SETTING_BLOCK);

[Bernhard Bauer, 2011/09/15 12:52:19]

You already have a call to DidBlockContentType below. You could move |allow| out
of this scope, break and then return it below.

[marja, 2011/09/15 16:00:03]

Done.

+        if (!allow)
+          DidBlockContentType(CONTENT_SETTINGS_TYPE_IMAGES, std::string());
+        return allow;
+      }
+    }
+  }
+
   DidBlockContentType(CONTENT_SETTINGS_TYPE_IMAGES, std::string());
   return false;  // Other protocols fall through here.
 }
 
 bool ContentSettingsObserver::AllowIndexedDB(WebFrame* frame,
                                              const WebString& name,
                                              const WebSecurityOrigin& origin) {
   if (frame->document().securityOrigin().isEmpty() ||
       frame->top()->document().securityOrigin().isEmpty())
     return false; // Uninitialized document.
@@ skipping 69 matching lines @@
   // CONTENT_SETTING_ASK is only valid for cookies.
   return current_content_settings_.settings[settings_type] !=
       CONTENT_SETTING_BLOCK;
 }
 
 void ContentSettingsObserver::ClearBlockedContentSettings() {
   for (size_t i = 0; i < arraysize(content_blocked_); ++i)
     content_blocked_[i] = false;
   cached_storage_permissions_.clear();
 }