Change implementation of resize ImageBitmap to meet specs

third_party/WebKit/Source/core/frame/ImageBitmap.cpp

Index: third_party/WebKit/Source/core/frame/ImageBitmap.cpp
diff --git a/third_party/WebKit/Source/core/frame/ImageBitmap.cpp b/third_party/WebKit/Source/core/frame/ImageBitmap.cpp
index d614506a5ab4cdfd4d50a589b35002bd0e239507..80c1587b8081bed0db5e7441b94ec0537bb16b6d 100644
--- a/third_party/WebKit/Source/core/frame/ImageBitmap.cpp
+++ b/third_party/WebKit/Source/core/frame/ImageBitmap.cpp
@@ -65,25 +65,25 @@ ParsedOptions parseOptions(const ImageBitmapOptions& options, Optional<IntRect>
 
     int sourceWidth = sourceSize.width();
     int sourceHeight = sourceSize.height();
+    if (!cropRect) {
+        parsedOptions.cropRect = IntRect(0, 0, sourceWidth, sourceHeight);
+    } else  {
+        parsedOptions.cropRect = normalizeRect(*cropRect);
+    }
     if (!options.hasResizeWidth() && !options.hasResizeHeight()) {
-        parsedOptions.resizeWidth = sourceWidth;
-        parsedOptions.resizeHeight = sourceHeight;
+        parsedOptions.resizeWidth = parsedOptions.cropRect.width();
+        parsedOptions.resizeHeight = parsedOptions.cropRect.height();
     } else if (options.hasResizeWidth() && options.hasResizeHeight()) {
         parsedOptions.resizeWidth = options.resizeWidth();
         parsedOptions.resizeHeight = options.resizeHeight();
     } else if (options.hasResizeWidth() && !options.hasResizeHeight()) {
         parsedOptions.resizeWidth = options.resizeWidth();
-        parsedOptions.resizeHeight = ceil(static_cast<float>(options.resizeWidth()) / sourceWidth * sourceHeight);
+        parsedOptions.resizeHeight = ceil(static_cast<float>(options.resizeWidth()) / parsedOptions.cropRect.width() * parsedOptions.cropRect.height());
     } else {
         parsedOptions.resizeHeight = options.resizeHeight();
-        parsedOptions.resizeWidth = ceil(static_cast<float>(options.resizeHeight()) / sourceHeight * sourceWidth);
-    }
-    if (!cropRect) {
-        parsedOptions.cropRect = IntRect(0, 0, parsedOptions.resizeWidth, parsedOptions.resizeHeight);
-    } else  {
-        parsedOptions.cropRect = normalizeRect(*cropRect);
+        parsedOptions.resizeWidth = ceil(static_cast<float>(options.resizeHeight()) / parsedOptions.cropRect.height() * parsedOptions.cropRect.width());
     }
-    if (static_cast<int>(parsedOptions.resizeWidth) == sourceWidth && static_cast<int>(parsedOptions.resizeHeight) == sourceHeight) {
+    if (static_cast<int>(parsedOptions.resizeWidth) == parsedOptions.cropRect.width() && static_cast<int>(parsedOptions.resizeHeight) == parsedOptions.cropRect.height()) {
         parsedOptions.shouldScaleInput = false;
         return parsedOptions;
     }
@@ -212,9 +212,10 @@ static PassRefPtr<StaticBitmapImage> cropImage(Image* image, const ParsedOptions
     // In the case when cropRect doesn't intersect the source image and it requires a umpremul image
     // We immediately return a transparent black image with cropRect.size()
     if (srcRect.isEmpty() && !parsedOptions.premultiplyAlpha) {
-        SkImageInfo info = SkImageInfo::Make(parsedOptions.cropRect.width(), parsedOptions.cropRect.height(), kN32_SkColorType, kUnpremul_SkAlphaType);
-        std::unique_ptr<uint8_t[]> dstPixels = wrapArrayUnique(new uint8_t[parsedOptions.cropRect.width() * parsedOptions.cropRect.height() * info.bytesPerPixel()]());
-        return StaticBitmapImage::create(newSkImageFromRaster(info, std::move(dstPixels), parsedOptions.cropRect.width() * info.bytesPerPixel()));
+        SkImageInfo info = SkImageInfo::Make(parsedOptions.shouldScaleInput ? parsedOptions.resizeWidth : parsedOptions.cropRect.width(),

[Justin Novosad, 2016/07/25 18:03:10]

This ternary operation is not necessary. 
Reason: If shouldScaleInput is false then we know that cropRect.width() ==
resizeWidth, so you can just use resizeWidth in either case,

[xidachen, 2016/07/25 19:02:01]

Good catch, changes are made. Thanks.

+            parsedOptions.shouldScaleInput ? parsedOptions.resizeHeight : parsedOptions.cropRect.height(), kN32_SkColorType, kUnpremul_SkAlphaType);

[Justin Novosad, 2016/07/25 18:03:10]

same here.

[xidachen, 2016/07/25 19:02:01]

Done.

+        std::unique_ptr<uint8_t[]> dstPixels = wrapArrayUnique(new uint8_t[info.width() * info.height() * info.bytesPerPixel()]());
+        return StaticBitmapImage::create(newSkImageFromRaster(info, std::move(dstPixels), info.width() * info.bytesPerPixel()));
     }
 
     RefPtr<SkImage> skiaImage = image->imageForCurrentFrame();
@@ -231,7 +232,7 @@ static PassRefPtr<StaticBitmapImage> cropImage(Image* image, const ParsedOptions
             return nullptr;
     }
 
-    if (parsedOptions.cropRect == srcRect) {
+    if (parsedOptions.cropRect == srcRect && !parsedOptions.shouldScaleInput) {
         RefPtr<SkImage> croppedSkImage = fromSkSp(skiaImage->makeSubset(srcRect));
         if (parsedOptions.flipY)
             return StaticBitmapImage::create(flipSkImageVertically(croppedSkImage.get(), parsedOptions.premultiplyAlpha ? PremultiplyAlpha : DontPremultiplyAlpha));
@@ -243,7 +244,8 @@ static PassRefPtr<StaticBitmapImage> cropImage(Image* image, const ParsedOptions
         return StaticBitmapImage::create(croppedSkImage.release());
     }
 
-    sk_sp<SkSurface> surface = SkSurface::MakeRasterN32Premul(parsedOptions.cropRect.width(), parsedOptions.cropRect.height());
+    sk_sp<SkSurface> surface = SkSurface::MakeRasterN32Premul(parsedOptions.shouldScaleInput ? parsedOptions.resizeWidth : parsedOptions.cropRect.width(),

[Justin Novosad, 2016/07/25 18:03:10]

same here.

[xidachen, 2016/07/25 19:02:01]

Done.

+        parsedOptions.shouldScaleInput ? parsedOptions.resizeHeight : parsedOptions.cropRect.height());

[Justin Novosad, 2016/07/25 18:03:10]

same here.

[xidachen, 2016/07/25 19:02:02]

Done.

     if (!surface)
         return nullptr;
     if (srcRect.isEmpty())
@@ -260,10 +262,11 @@ static PassRefPtr<StaticBitmapImage> cropImage(Image* image, const ParsedOptions
         surface->getCanvas()->scale(1, -1);
     }
     if (parsedOptions.shouldScaleInput) {
-        SkRect drawDstRect = SkRect::MakeXYWH(dstLeft, dstTop, parsedOptions.resizeWidth, parsedOptions.resizeHeight);
+        SkRect drawSrcRect = SkRect::MakeXYWH(parsedOptions.cropRect.x(), parsedOptions.cropRect.y(), parsedOptions.cropRect.width(), parsedOptions.cropRect.height());
+        SkRect drawDstRect = SkRect::MakeXYWH(0, 0, parsedOptions.resizeWidth, parsedOptions.resizeHeight);
         SkPaint paint;
         paint.setFilterQuality(parsedOptions.resizeQuality);
-        surface->getCanvas()->drawImageRect(skiaImage.get(), drawDstRect, &paint);
+        surface->getCanvas()->drawImageRect(skiaImage.get(), drawSrcRect, drawDstRect, &paint);
     } else {
         surface->getCanvas()->drawImage(skiaImage.get(), dstLeft, dstTop);
     }