Add a per-tab preference to match WebSettings::images_enabled.

Add a per-tab preference to match WebSettings::images_enabled.

kWebKitLoadsImagesAutomatically isn't enough. In plain WebKit one must also set
a flag in CachedResourceLoader. In Chrome this loader class isn't used, and
image loading control is done via ChromeRenderViewObserver.

R=bauerb@chromium.org
TBR=darin@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=118276

[Bernhard Bauer, 2012-01-18 16:32:34]

LGTM.

Does |loads_images_automatically| actually do anything? If not, I think we
should remove it to keep the API surface smaller.

[mnaganov_google.com, 2012-01-18 16:33:55]

On 2012/01/18 16:32:34, Bernhard Bauer wrote:
> LGTM.
> 
> Does |loads_images_automatically| actually do anything? If not, I think we
> should remove it to keep the API surface smaller.

Thanks!

It looks like it doesn't. I'll check if it's safe to remove it.

[mnaganov_google.com, 2012-01-18 16:40:07]

On 2012/01/18 16:33:55, mnaganov wrote:
> On 2012/01/18 16:32:34, Bernhard Bauer wrote:
> > LGTM.
> > 
> > Does |loads_images_automatically| actually do anything? If not, I think we
> > should remove it to keep the API surface smaller.
> 
> Thanks!
> 
> It looks like it doesn't. I'll check if it's safe to remove it.

Darin, I need OWNER's approval for webkit/ and content/common/.
Peter, I need an approval for chrome/browser/ui/.
Please take a look.

[Peter Kasting, 2012-01-18 17:32:19]

Rubber-stamp LGTM