Delegating the "are images allowed" decision to renderer.

chrome/renderer/chrome_content_renderer_client.h

 // 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.
 
 #ifndef CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
 #define CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
 #pragma once
 
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string16.h"
+#include "chrome/common/content_settings.h"
 #include "content/public/renderer/content_renderer_client.h"
 
 class ChromeRenderProcessObserver;
 class ExtensionDispatcher;
 class RendererHistogramSnapshots;
 class RendererNetPredictor;
 class SpellCheck;
 class SpellCheckProvider;
 class VisitedLinkSlave;
 
@@ skipping 66 matching lines @@
                                       const std::string& value) OVERRIDE;
   virtual bool IsProtocolSupportedForMedia(const GURL& url) OVERRIDE;
 
   // TODO(mpcomplete): remove after we collect histogram data.
   // http://crbug.com/100411
   bool IsAdblockInstalled();
   bool IsAdblockPlusInstalled();
   bool IsAdblockWithWebRequestInstalled();
   bool IsAdblockPlusWithWebRequestInstalled();
 
+  void SetImageSettingRules(
+      const ContentSettingsForOneType& image_setting_rules);
+
   // For testing.
   void SetExtensionDispatcher(ExtensionDispatcher* extension_dispatcher);
 
   // Called in low-memory conditions to dump the memory used by the spellchecker
   // and start over.
   void OnPurgeMemory();
 
  private:
   WebKit::WebPlugin* CreatePlugin(
       content::RenderView* render_view,
@@ skipping 20 matching lines @@
   scoped_ptr<ChromeRenderProcessObserver> chrome_observer_;
   scoped_ptr<ExtensionDispatcher> extension_dispatcher_;
   scoped_ptr<RendererHistogramSnapshots> histogram_snapshots_;
   scoped_ptr<RendererNetPredictor> net_predictor_;
   scoped_ptr<SpellCheck> spellcheck_;
   // The SpellCheckProvider is a RenderViewObserver, and handles its own
   // destruction.
   SpellCheckProvider* spellcheck_provider_;
   scoped_ptr<VisitedLinkSlave> visited_link_slave_;
   scoped_ptr<safe_browsing::PhishingClassifierFilter> phishing_classifier_;
+  ContentSettingsForOneType image_setting_rules_;

[Bernhard Bauer, 2011/10/17 13:13:19]

The content client should be the glue between the content layer and the chrome
embedder, and carry no state itself. If you want to store one copy of the image
setting rules, you can store it them in the ChromeRenderProcessObserver.

[marja, 2011/10/18 12:23:02]

Done. But, the ChromeRenderProcessObserver is not created in the tests, so I
added a ha.. err, indirection here: ChromeContentRendererClient holds a pointer
to the image setting rules. Normally ChromeRenderProcessObserver sets it to
point to its member, but also the tests can set it.

 };
 
 }  // namespace chrome
 
 #endif  // CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_