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

blimp/engine/app/engine_settings.cc

 // 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.
 
 #include "blimp/engine/app/engine_settings.h"
 
 namespace blimp {
 namespace engine {
 
-EngineSettings::EngineSettings() : record_whole_document(false) {}
+EngineSettings::EngineSettings()
+    : record_whole_document(false),
+      // The default animation policy for images is no animation. This means
+      // that
+      // only the first frame will be shown.

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

Weird line-wrap?

This comment seems unnecessarily verbose; I'd suggest something like "Disable
animation in images (e.g. GIFs) by default.". Reader should be able to find the
enum declaration and read the comments on its precise meaning themselves.

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

Done.

+      // See third_party/WebKit/Source/platform/graphics/ImageAnimationPolicy.h
+      // for details.
+      animation_policy(
+          content::ImageAnimationPolicy::IMAGE_ANIMATION_POLICY_NO_ANIMATION) {}
 
 EngineSettings::~EngineSettings() {}
 
 EngineSettings::EngineSettings(const EngineSettings& other) = default;
 
 }  // namespace engine
 }  // namespace blimp