blimp: Update animation policy for images on the blimp engine.

blimp/engine/app/settings_manager.cc

Index: blimp/engine/app/settings_manager.cc
diff --git a/blimp/engine/app/settings_manager.cc b/blimp/engine/app/settings_manager.cc
index baf0a094cadbef6970cde0f6f4305cdaafd1c358..460c754bc510b0c5f5aa75f831260df976eea39e 100644
--- a/blimp/engine/app/settings_manager.cc
+++ b/blimp/engine/app/settings_manager.cc
@@ -25,6 +25,7 @@ void SettingsManager::UpdateWebkitPreferences(content::WebPreferences* prefs) {
   DCHECK(prefs);

[Wez, 2016/04/20 23:09:06]

nit: Also not specific to this CL, but no point DCHECK()ing a pointer
immediately before you de-reference it!

[Khushal, 2016/04/21 00:21:25]

The DCHECK helps since it gives a clear stack trace in case someone passes a
nullptr rather than a segmentation fault.

[Wez, 2016/04/21 00:40:28]

The stack trace will be the same, surely? The only difference will be that the
failing pointer is a small integer, rather than actually zero.

[Khushal, 2016/04/21 01:20:17]

Woops, I didn't notice that. :P
Done.

 
   prefs->record_whole_document = settings_.record_whole_document;
+  prefs->animation_policy = settings_.animation_policy;
 }
 
 const EngineSettings& SettingsManager::GetEngineSettings() const {