Make image-rendering:pixelated use nearest neighbor when downscaling.

Make image-rendering:pixelated use nearest neighbor when downscaling.

The spec has changed and no longer specifies that "pixelated" should be
the same as "auto" when downscaling.

BUG=317991
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=186928

[jackhou1, 2014-12-11 00:18:52]

jackhou@chromium.org changed reviewers:
+ pdr@chromium.org

[jackhou1, 2014-12-11 00:18:53]

pdr, could you take a look?

[pdr., 2014-12-11 05:19:57]

The CQ bit was checked by pdr@chromium.org

[pdr., 2014-12-11 05:19:58]

Nice! LGTM

[commit-bot: I haz the power, 2014-12-11 05:20:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/754083003/20001

[commit-bot: I haz the power, 2014-12-11 05:21:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-11 05:21:03]

Failed to apply patch for Source/core/paint/HTMLCanvasPainter.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/paint/HTMLCanvasPainter.cpp
  Hunk #1 FAILED at 32.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/paint/HTMLCanvasPainter.cpp.rej

Patch:       Source/core/paint/HTMLCanvasPainter.cpp
Index: Source/core/paint/HTMLCanvasPainter.cpp
diff --git a/Source/core/paint/HTMLCanvasPainter.cpp
b/Source/core/paint/HTMLCanvasPainter.cpp
index
e4ac72522478d69b04bf9aceaa3ef14dac5c5c5d..0847ecc8cc5ca2b28bb1169936502e676e71a2b4
100644
--- a/Source/core/paint/HTMLCanvasPainter.cpp
+++ b/Source/core/paint/HTMLCanvasPainter.cpp
@@ -32,18 +32,13 @@ void HTMLCanvasPainter::paintReplaced(const PaintInfo&
paintInfo, const LayoutPo
     // FIXME: InterpolationNone should be used if
ImageRenderingOptimizeContrast is set.
     // See bug for more details: crbug.com/353716.
     InterpolationQuality interpolationQuality =
m_renderHTMLCanvas.style()->imageRendering() == ImageRenderingOptimizeContrast ?
InterpolationLow : CanvasDefaultInterpolationQuality;
-
-    HTMLCanvasElement* canvas = toHTMLCanvasElement(m_renderHTMLCanvas.node());
-    LayoutSize layoutSize = contentRect.size();
-    if (m_renderHTMLCanvas.style()->imageRendering() == ImageRenderingPixelated
-        && (layoutSize.width() > canvas->width() || layoutSize.height() >
canvas->height() || layoutSize == canvas->size())) {
+    if (m_renderHTMLCanvas.style()->imageRendering() ==
ImageRenderingPixelated)
         interpolationQuality = InterpolationNone;
-    }
 
     DrawingRecorder recorder(context, &m_renderHTMLCanvas,
localPaintInfo.phase, pixelSnappedIntRect(paintRect));
     InterpolationQuality previousInterpolationQuality =
context->imageInterpolationQuality();
     context->setImageInterpolationQuality(interpolationQuality);
-    canvas->paint(context, paintRect);
+    toHTMLCanvasElement(m_renderHTMLCanvas.node())->paint(context, paintRect);
     context->setImageInterpolationQuality(previousInterpolationQuality);
 }

[jackhou1, 2014-12-11 07:18:38]

The CQ bit was checked by jackhou@chromium.org

[commit-bot: I haz the power, 2014-12-11 07:18:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/754083003/40001

[commit-bot: I haz the power, 2014-12-11 09:43:06]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=186928