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

blimp/engine/app/engine_settings.h

 // Copyright 2016 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 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);
   ~EngineSettings();
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_APP_ENGINE_SETTINGS_H_