Bug 845125. Fix impromper use of CheckedInt.
authorJeff Muizelaar <jmuizelaar@mozilla.com>
Tue, 30 Apr 2013 15:46:22 -0400
changeset 130382 fa4d8053055dcd623f54f81a3ec8a78ebe1ed03e
parent 130381 fb8ae1b71f3f957873f5f15109adf716a04658e8
child 130383 533d3fb8a7e98cb9dcf9ddba9aa1d03be11db692
push id27335
push userjmuizelaar@mozilla.com
push dateTue, 30 Apr 2013 19:46:45 +0000
treeherdermozilla-inbound@fa4d8053055d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs845125
milestone23.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 845125. Fix impromper use of CheckedInt. Previously we're missing the multiplication.
gfx/2d/Blur.cpp
--- a/gfx/2d/Blur.cpp
+++ b/gfx/2d/Blur.cpp
@@ -396,17 +396,17 @@ AlphaBoxBlur::AlphaBoxBlur(const Rect& a
                            float aSigma)
   : mRect(int32_t(aRect.x), int32_t(aRect.y),
           int32_t(aRect.width), int32_t(aRect.height)),
     mSpreadRadius(),
     mBlurRadius(CalculateBlurRadius(Point(aSigma, aSigma))),
     mStride(aStride),
     mSurfaceAllocationSize(-1)
 {
-  CheckedInt<int32_t> minDataSize = CheckedInt<int32_t>(aRect.width*aRect.height);
+  CheckedInt<int32_t> minDataSize = CheckedInt<int32_t>(aRect.width)*aRect.height;
   if (minDataSize.isValid()) {
     mSurfaceAllocationSize = minDataSize.value();
   }
 }
 
 
 AlphaBoxBlur::~AlphaBoxBlur()
 {