Load a placeholder when a url to an image is broken.

I decided to use the old Netscape broken image link icon (later we may
replace the image asset for something more trendier). The ref test will
expect that a failed load should display the rippy image.

ImageCacheTask users can define if a placeholder image should be loaded
at start up or not. This enables both the new behavior (e.g. always
return an image even for broken urls) as also the previous one.

1 files changed, 3 insertions, 3 deletions

diff --git a/components/servo/lib.rs b/components/servo/lib.rs
index 1afdbf8dca5..b197ad30e6f 100644
--- a/components/servo/lib.rs
+++ b/components/servo/lib.rs
@@ -33,7 +33,7 @@ use msg::constellation_msg::ConstellationChan;
 use script::dom::bindings::codegen::RegisterBindings;
 
 #[cfg(not(test))]
-use net::image_cache_task::ImageCacheTask;
+use net::image_cache_task::{ImageCacheTask, LoadPlaceholder};
 #[cfg(not(test))]
 use net::resource_task::new_resource_task;
 #[cfg(not(test))]
@@ -84,10 +84,10 @@ impl Browser  {
         // image.
         let image_cache_task = if opts.output_file.is_some() {
             ImageCacheTask::new_sync(resource_task.clone(), shared_task_pool,
-                                     time_profiler_chan.clone())
+                                     time_profiler_chan.clone(), LoadPlaceholder::Preload)
         } else {
             ImageCacheTask::new(resource_task.clone(), shared_task_pool,
-                                time_profiler_chan.clone())
+                                time_profiler_chan.clone(), LoadPlaceholder::Preload)
         };
 
         let font_cache_task = FontCacheTask::new(resource_task.clone());