Returning IntSize to store calculated kernel size values

Source/platform/graphics/filters/FEDropShadow.cpp

 /*
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 44 matching lines @@
     Filter* filter = this->filter();
     ASSERT(filter);
 
     FloatRect offsetRect = rect;
     if (forward)
         offsetRect.move(filter->applyHorizontalScale(m_dx), filter->applyVerticalScale(m_dy));
     else
         offsetRect.move(-filter->applyHorizontalScale(m_dx), -filter->applyVerticalScale(m_dy));
     result.unite(offsetRect);
 
-    unsigned kernelSizeX = 0;
-    unsigned kernelSizeY = 0;
-    FEGaussianBlur::calculateKernelSize(filter, kernelSizeX, kernelSizeY, m_stdX, m_stdY);
+    IntSize kernelSize(0, 0);

[pdr., 2014/03/26 00:00:13]

Is this needed?

[Savago, 2014/03/26 00:12:24]

Just checked IntSize() and it already initializes its members to 0. Indeed, I
can remove it.

+    kernelSize = FEGaussianBlur::calculateKernelSize(filter, m_stdX, m_stdY);
 
     // We take the half kernel size and multiply it with three, because we run box blur three times.
-    result.inflateX(3 * kernelSizeX * 0.5f);
-    result.inflateY(3 * kernelSizeY * 0.5f);
+    result.inflateX(3 * kernelSize.width() * 0.5f);
+    result.inflateY(3 * kernelSize.height() * 0.5f);
     return result;
 }
 
 void FEDropShadow::applySoftware()
 {
     FilterEffect* in = inputEffect(0);
 
     ImageBuffer* resultImage = createImageBufferResult();
     if (!resultImage)
         return;
@@ skipping 41 matching lines @@
 {
     writeIndent(ts, indent);
     ts << "[feDropShadow";
     FilterEffect::externalRepresentation(ts);
     ts << " stdDeviation=\"" << m_stdX << ", " << m_stdY << "\" dx=\"" << m_dx << "\" dy=\"" << m_dy << "\" flood-color=\"" << m_shadowColor.nameForRenderTreeAsText() <<"\" flood-opacity=\"" << m_shadowOpacity << "]\n";
     inputEffect(0)->externalRepresentation(ts, indent + 1);
     return ts;
 }
 
 } // namespace WebCore