Remove loop unrolling code in UpdateYUVComponentSizes.

third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 83 matching lines @@
   void* pixels_;
   size_t row_bytes_;
 };
 
 static bool UpdateYUVComponentSizes(ImageDecoder* decoder,
                                     SkISize component_sizes[3],
                                     size_t component_width_bytes[3]) {
   if (!decoder->CanDecodeToYUV())
     return false;
 
-  IntSize size = decoder->DecodedYUVSize(0);
-  component_sizes[0].set(size.Width(), size.Height());
-  component_width_bytes[0] = decoder->DecodedYUVWidthBytes(0);
-  size = decoder->DecodedYUVSize(1);
-  component_sizes[1].set(size.Width(), size.Height());
-  component_width_bytes[1] = decoder->DecodedYUVWidthBytes(1);
-  size = decoder->DecodedYUVSize(2);
-  component_sizes[2].set(size.Width(), size.Height());
-  component_width_bytes[2] = decoder->DecodedYUVWidthBytes(2);
+  for (int i = 0; i < 3; ++i) {

[Srirama, 2017/05/22 13:29:30]

Suggestion: may be use componentIndex or some other better name instead of 'i'.

+    IntSize size = decoder->DecodedYUVSize(i);
+    component_sizes[i].set(size.Width(), size.Height());
+    component_width_bytes[i] = decoder->DecodedYUVWidthBytes(i);
+  }
+
   return true;
 }
 
 ImageFrameGenerator::ImageFrameGenerator(const SkISize& full_size,
                                          bool is_multi_frame,
                                          const ColorBehavior& color_behavior)
     : full_size_(full_size),
       decoder_color_behavior_(color_behavior),
       is_multi_frame_(is_multi_frame),
       decode_failed_(false),
@@ skipping 274 matching lines @@
   // do YUV decoding.
   std::unique_ptr<ImagePlanes> dummy_image_planes =
       WTF::WrapUnique(new ImagePlanes);
   decoder->SetImagePlanes(std::move(dummy_image_planes));
 
   return UpdateYUVComponentSizes(decoder.get(), size_info->fSizes,
                                  size_info->fWidthBytes);
 }
 
 }  // namespace blink