Use transposedSize in more places

Source/platform/graphics/BitmapImage.cpp

 /*
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2008 Apple 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
@@ skipping 32 matching lines @@
 
 PassRefPtr<BitmapImage> BitmapImage::createWithOrientationForTesting(const SkBitmap& bitmap, ImageOrientation orientation)
 {
     if (bitmap.isNull()) {
         return BitmapImage::create();
     }
 
     RefPtr<BitmapImage> result = adoptRef(new BitmapImage(bitmap));
     result->m_frames[0].m_orientation = orientation;
     if (orientation.usesWidthAsHeight())
-        result->m_sizeRespectingOrientation = IntSize(result->m_size.height(), result->m_size.width());
+        result->m_sizeRespectingOrientation = result->m_size.transposedSize();
     return result.release();
 }
 
 BitmapImage::BitmapImage(ImageObserver* observer)
     : Image(observer)
     , m_currentFrame(0)
     , m_repetitionCount(cAnimationNone)
     , m_repetitionCountStatus(Unknown)
     , m_repetitionsComplete(0)
     , m_desiredFrameStartTime(0)
@@ skipping 227 matching lines @@
 
         // ImageOrientation expects the origin to be at (0, 0)
         canvas->translate(adjustedDstRect.x(), adjustedDstRect.y());
         adjustedDstRect.setLocation(FloatPoint());
 
         canvas->concat(affineTransformToSkMatrix(orientation.transformFromDefault(adjustedDstRect.size())));
 
         if (orientation.usesWidthAsHeight()) {
             // The destination rect will have it's width and height already reversed for the orientation of
             // the image, as it was needed for page layout, so we need to reverse it back here.
-            adjustedDstRect = FloatRect(adjustedDstRect.x(), adjustedDstRect.y(), adjustedDstRect.height(), adjustedDstRect.width());

[leviw_travelin_and_unemployed, 2015/08/14 22:34:14]

The old code wasn't actually a transposedRect, as x and y are still in order.
It's just transposing the size.

+            adjustedDstRect = adjustedDstRect.transposedRect();
         }
     }
 
     SkRect skSrcRect = adjustedSrcRect;
     canvas->drawImageRect(image.get(), skSrcRect, adjustedDstRect, &paint,
         WebCoreClampingModeToSkiaRectConstraint(clampMode));
     canvas->restoreToCount(initialSaveCount);
 
     if (currentFrameIsLazyDecoded())
         PlatformInstrumentation::didDrawLazyPixelRef(image->uniqueID());
@@ skipping 362 matching lines @@
     destroyDecodedDataIfNecessary();
 
     // We need to draw this frame if we advanced to it while not skipping, or if
     // while trying to skip frames we hit the last frame and thus had to stop.
     if (skippingFrames != advancedAnimation)
         imageObserver()->animationAdvanced(this);
     return advancedAnimation;
 }
 
 } // namespace blink