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

blimp/engine/app/engine_settings.h

Index: blimp/engine/app/engine_settings.h
diff --git a/blimp/engine/app/engine_settings.h b/blimp/engine/app/engine_settings.h
index 99d5663a4a2c7a3d741c07de849a48290d8267d2..54b9ff410a998ef38cbf8ca0014b9972475ac0c9 100644
--- a/blimp/engine/app/engine_settings.h
+++ b/blimp/engine/app/engine_settings.h
@@ -5,12 +5,15 @@
 #ifndef BLIMP_ENGINE_APP_ENGINE_SETTINGS_H_
 #define BLIMP_ENGINE_APP_ENGINE_SETTINGS_H_
 
+#include "content/public/common/web_preferences.h"
+
 namespace blimp {
 namespace engine {
 
 struct EngineSettings {
   // -------WebPreferences-------
   bool record_whole_document;
+  content::ImageAnimationPolicy animation_policy;

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

Not specific to this CL, but...

1. This structure needs a comment to explain what "settings" it carries.
2. The members need comments to explain what they actually affect (though "These
parameters mirror those in blink::WebPreferences" or something would be fine.)
3. Order of definitions is odd; ctor/dtor etc normally precede members.
4. Not clear what the default values are?

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

Done.
Should the default values be marked in the header? I was being consistent with
the pattern in content::WebPreferences where the default values are just
specified in the implementation.

 
   EngineSettings();
   EngineSettings(const EngineSettings& other);