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 {
 
+// The EngineSettings includes all parameters which are configured for the
+// BlimpEngine and can be set by the client. This includes the values for
+// content::WebPreferences which override the default values.
 struct EngineSettings {
-  // -------WebPreferences-------
-  bool record_whole_document;
-
   EngineSettings();

[Wez, 2016/04/21 17:29:02]

Since this struct is pure POD I think you can declare this ctor inline and avoid
the cc file.

[Khushal, 2016/04/21 18:45:48]

Done.

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

[Wez, 2016/04/21 17:29:02]

Do you actually need an explicit copy ctor and dtor, since this is a pure POD
struct?

[Khushal, 2016/04/21 18:45:48]

Nope. Removed.

+
+  // -------WebPreferences-------
+  // These members mirror the parameters in content::WebPreferences. See
+  // content/public/common/web_preferences.h for details.
+
+  bool record_whole_document = false;
+
+  // Disable animation in images by default.
+  content::ImageAnimationPolicy animation_policy =
+      content::ImageAnimationPolicy::IMAGE_ANIMATION_POLICY_NO_ANIMATION;
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_APP_ENGINE_SETTINGS_H_