[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.20-204-g221d8e8

eric at webkit.org eric at webkit.org
Wed Feb 10 22:16:03 UTC 2010


The following commit has been merged in the webkit-1.1 branch:
commit 670d68d8a74efe5fbc4332bcf38df51b0369c7ac
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Feb 5 07:18:10 2010 +0000

    2010-02-04  Yaar Schnitman  <yaar at chromium.org>
    
            Reviewed by Darin Fisher.
    
            Upstreaming gtests from chromium: UniscribeHelper and TransparencyWin
            https://bugs.webkit.org/show_bug.cgi?id=34509
    
            Resubmit: The previous commit (r54333) was rolled back.
    
            * WebKit.gyp:
            * tests/TransparencyWinTest.cpp: Added.
            (WebCore::RECTToFloatRect):
            (WebCore::drawNativeRect):
            (WebCore::getPixelAt):
            (WebCore::clearTopLayerAlphaChannel):
            (WebCore::clearTopLayerAlphaPixel):
            (WebCore::operator<<):
            (WebCore::TEST):
            * tests/UniscribeHelperTest.cpp: Added.
            (WebCore::UniscribeTest::UniscribeTest):
            (WebCore::UniscribeTest::MakeFont):
            (WebCore::UniscribeTest::SetUp):
            (WebCore::UniscribeTest::TearDown):
            (TEST_F):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@54410 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKit/chromium/ChangeLog b/WebKit/chromium/ChangeLog
index 0d65d80..7396b4a 100644
--- a/WebKit/chromium/ChangeLog
+++ b/WebKit/chromium/ChangeLog
@@ -1,3 +1,28 @@
+2010-02-04  Yaar Schnitman  <yaar at chromium.org>
+
+        Reviewed by Darin Fisher.
+
+        Upstreaming gtests from chromium: UniscribeHelper and TransparencyWin
+        https://bugs.webkit.org/show_bug.cgi?id=34509
+
+        Resubmit: The previous commit (r54333) was rolled back.
+
+        * WebKit.gyp:
+        * tests/TransparencyWinTest.cpp: Added.
+        (WebCore::RECTToFloatRect):
+        (WebCore::drawNativeRect):
+        (WebCore::getPixelAt):
+        (WebCore::clearTopLayerAlphaChannel):
+        (WebCore::clearTopLayerAlphaPixel):
+        (WebCore::operator<<):
+        (WebCore::TEST):
+        * tests/UniscribeHelperTest.cpp: Added.
+        (WebCore::UniscribeTest::UniscribeTest):
+        (WebCore::UniscribeTest::MakeFont):
+        (WebCore::UniscribeTest::SetUp):
+        (WebCore::UniscribeTest::TearDown):
+        (TEST_F):
+
 2010-02-04  Drew Wilson  <atwilson at chromium.org>
 
         Reviewed by David Levin.
diff --git a/WebKit/chromium/WebKit.gyp b/WebKit/chromium/WebKit.gyp
index 4f6d3f6..51693ba 100644
--- a/WebKit/chromium/WebKit.gyp
+++ b/WebKit/chromium/WebKit.gyp
@@ -453,6 +453,14 @@
                 'tests/KURLTest.cpp',
                 'tests/RunAllTests.cpp',
             ],
+            'conditions': [
+                ['OS=="win"', {
+                    'sources': [
+                        'tests/TransparencyWinTest.cpp',
+                        'tests/UniscribeHelperTest.cpp',
+                    ],
+                }],
+            ],
         },
     ], # targets
 }
diff --git a/WebKit/chromium/tests/TransparencyWinTest.cpp b/WebKit/chromium/tests/TransparencyWinTest.cpp
new file mode 100644
index 0000000..5fdff11
--- /dev/null
+++ b/WebKit/chromium/tests/TransparencyWinTest.cpp
@@ -0,0 +1,696 @@
+/*
+ * Copyright (C) 2010 Google 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:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+
+#include <gtest/gtest.h>
+#include <windows.h>
+
+#include "ImageBuffer.h"
+#include "TransformationMatrix.h"
+#include "TransparencyWin.h"
+
+namespace WebCore {
+
+static FloatRect RECTToFloatRect(const RECT* rect)
+{
+    return FloatRect(static_cast<float>(rect->left),
+                     static_cast<float>(rect->top),
+                     static_cast<float>(rect->right - rect->left),
+                     static_cast<float>(rect->bottom - rect->top));
+}
+
+static void drawNativeRect(GraphicsContext* context,
+                           int x, int y, int w, int h)
+{
+    skia::PlatformCanvas* canvas = context->platformContext()->canvas();
+    HDC dc = canvas->beginPlatformPaint();
+
+    RECT innerRc;
+    innerRc.left = x;
+    innerRc.top = y;
+    innerRc.right = x + w;
+    innerRc.bottom = y + h;
+    FillRect(dc, &innerRc,
+             reinterpret_cast<HBRUSH>(GetStockObject(BLACK_BRUSH)));
+
+    canvas->endPlatformPaint();
+}
+
+static Color getPixelAt(GraphicsContext* context, int x, int y)
+{
+    const SkBitmap& bitmap = context->platformContext()->canvas()->
+        getTopPlatformDevice().accessBitmap(false);
+    return Color(*reinterpret_cast<const RGBA32*>(bitmap.getAddr32(x, y)));
+}
+
+// Resets the top layer's alpha channel to 0 for each pixel. This simulates
+// Windows messing it up.
+static void clearTopLayerAlphaChannel(GraphicsContext* context)
+{
+    SkBitmap& bitmap = const_cast<SkBitmap&>(context->platformContext()->
+        canvas()->getTopPlatformDevice().accessBitmap(false));
+    for (int y = 0; y < bitmap.height(); y++) {
+        uint32_t* row = bitmap.getAddr32(0, y);
+        for (int x = 0; x < bitmap.width(); x++)
+            row[x] &= 0x00FFFFFF;
+    }
+}
+
+// Clears the alpha channel on the specified pixel.
+static void clearTopLayerAlphaPixel(GraphicsContext* context, int x, int y)
+{
+    SkBitmap& bitmap = const_cast<SkBitmap&>(context->platformContext()->
+        canvas()->getTopPlatformDevice().accessBitmap(false));
+    *bitmap.getAddr32(x, y) &= 0x00FFFFFF;
+}
+
+static std::ostream& operator<<(std::ostream& out, const Color& c)
+{
+    std::ios_base::fmtflags oldFlags = out.flags(std::ios_base::hex |
+                                                 std::ios_base::showbase);
+    out << c.rgb();
+    out.flags(oldFlags);
+    return out;
+}
+
+TEST(TransparencyWin, NoLayer)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(17, 16), DeviceRGB));
+
+    // KeepTransform
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::NoLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 12));
+
+        EXPECT_TRUE(src->context() == helper.context());
+        EXPECT_TRUE(IntSize(14, 12) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(1, 1, 14, 12) == helper.drawRect());
+    }
+
+    // Untransform is not allowed for NoLayer.
+
+    // ScaleTransform
+    src->context()->save();
+    src->context()->scale(FloatSize(2.0, 0.5));
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::NoLayer,
+                    TransparencyWin::ScaleTransform,
+                    IntRect(2, 2, 6, 6));
+        helper.composite();
+
+        // The coordinate system should be based in the upper left of our box.
+        // It should be post-transformed.
+        EXPECT_TRUE(src->context() == helper.context());
+        EXPECT_TRUE(IntSize(12, 3) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(4, 1, 12, 3) == helper.drawRect());
+    }
+    src->context()->restore();
+}
+
+TEST(TransparencyWin, WhiteLayer)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    // KeepTransform
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::WhiteLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 12));
+        helper.composite();
+
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(14, 12) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(1, 1, 14, 12) == helper.drawRect());
+    }
+
+    // Untransform
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::WhiteLayer,
+                    TransparencyWin::Untransform,
+                    IntRect(1, 1, 14, 12));
+        helper.composite();
+
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(14, 12) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(0, 0, 14, 12) == helper.drawRect());
+    }
+
+    // ScaleTransform
+    src->context()->save();
+    src->context()->scale(FloatSize(2.0, 0.5));
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::WhiteLayer,
+                    TransparencyWin::ScaleTransform,
+                    IntRect(2, 2, 6, 6));
+        helper.composite();
+
+        // The coordinate system should be based in the upper left of our box.
+        // It should be post-transformed.
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(12, 3) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(0, 0, 12, 3) == helper.drawRect());
+    }
+    src->context()->restore();
+}
+
+TEST(TransparencyWin, TextComposite)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    // KeepTransform is the only valid transform mode for TextComposite.
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::TextComposite,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 12));
+        helper.composite();
+
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(14, 12) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(1, 1, 14, 12) == helper.drawRect());
+    }
+}
+
+TEST(TransparencyWin, OpaqueCompositeLayer)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    // KeepTransform
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 12));
+        helper.composite();
+
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(14, 12) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(1, 1, 14, 12) == helper.drawRect());
+    }
+
+    // KeepTransform with scroll applied.
+    src->context()->save();
+    src->context()->translate(0, -1);
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 14));
+        helper.composite();
+
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(14, 14) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(1, 1, 14, 14) == helper.drawRect());
+    }
+    src->context()->restore();
+
+    // Untransform
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::Untransform,
+                    IntRect(1, 1, 14, 12));
+        helper.composite();
+
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(14, 12) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(0, 0, 14, 12) == helper.drawRect());
+    }
+
+    // ScaleTransform
+    src->context()->save();
+    src->context()->scale(FloatSize(2.0, 0.5));
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::ScaleTransform,
+                    IntRect(2, 2, 6, 6));
+        helper.composite();
+
+        // The coordinate system should be based in the upper left of our box.
+        // It should be post-transformed.
+        EXPECT_TRUE(src->context() != helper.context());
+        EXPECT_TRUE(IntSize(12, 3) == helper.m_layerSize);
+        EXPECT_TRUE(IntRect(0, 0, 12, 3) == helper.drawRect());
+    }
+    src->context()->restore();
+}
+
+TEST(TransparencyWin, WhiteLayerPixelTest)
+{
+    // Make a total transparent buffer, and draw the white layer inset by 1 px.
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::WhiteLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 14));
+
+        // Coordinates should be in the original space, not the layer.
+        drawNativeRect(helper.context(), 3, 3, 1, 1);
+        clearTopLayerAlphaChannel(helper.context());
+        helper.composite();
+    }
+
+    // The final image should be transparent around the edges for 1 px, white
+    // in the middle, with (3,3) (what we drew above) being opaque black.
+    EXPECT_EQ(Color(Color::transparent), getPixelAt(src->context(), 0, 0));
+    EXPECT_EQ(Color(Color::white), getPixelAt(src->context(), 2, 2));
+    EXPECT_EQ(Color(Color::black), getPixelAt(src->context(), 3, 3));
+    EXPECT_EQ(Color(Color::white), getPixelAt(src->context(), 4, 4));
+}
+
+TEST(TransparencyWin, OpaqueCompositeLayerPixel)
+{
+    Color red(0xFFFF0000), darkRed(0xFFBF0000);
+    Color green(0xFF00FF00);
+
+    // Make a red bottom layer, followed by a half green next layer @ 50%.
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    FloatRect fullRect(0, 0, 16, 16);
+    src->context()->fillRect(fullRect, red, DeviceColorSpace);
+    src->context()->beginTransparencyLayer(0.5);
+    FloatRect rightHalf(8, 0, 8, 16);
+    src->context()->fillRect(rightHalf, green, DeviceColorSpace);
+
+    // Make a transparency layer inset by one pixel, and fill it inset by
+    // another pixel with 50% black.
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, 1, 14, 14));
+
+        FloatRect inner(2, 2, 12, 12);
+        helper.context()->fillRect(inner, Color(0x7f000000), DeviceColorSpace);
+        // These coordinates are relative to the layer, whish is inset by 1x1
+        // pixels from the top left. So we're actually clearing (2, 2) and
+        // (13,13), which are the extreme corners of the black area (and which
+        // we check below).
+        clearTopLayerAlphaPixel(helper.context(), 1, 1);
+        clearTopLayerAlphaPixel(helper.context(), 12, 12);
+        helper.composite();
+    }
+
+    // Finish the compositing.
+    src->context()->endTransparencyLayer();
+
+    // Check that we got the right values, it should be like the rectangle was
+    // drawn with half opacity even though the alpha channel got messed up.
+    EXPECT_EQ(red, getPixelAt(src->context(), 0, 0));
+    EXPECT_EQ(red, getPixelAt(src->context(), 1, 1));
+    EXPECT_EQ(darkRed, getPixelAt(src->context(), 2, 2));
+
+    // The dark result is:
+    //   (black @ 50% atop green) @ 50% atop red = 0xFF804000
+    // which is 0xFFA02000 (Skia computes 0xFFA11F00 due to rounding).
+    Color darkGreenRed(0xFF803f00);
+    EXPECT_EQ(darkGreenRed, getPixelAt(src->context(), 13, 13));
+
+    // 50% green on top of red = FF808000 (rounded to what Skia will produce).
+    Color greenRed(0xFF807F00);
+    EXPECT_EQ(greenRed, getPixelAt(src->context(), 14, 14));
+    EXPECT_EQ(greenRed, getPixelAt(src->context(), 15, 15));
+}
+
+// Tests that translations are properly handled when using KeepTransform.
+TEST(TransparencyWin, TranslateOpaqueCompositeLayer)
+{
+    // Fill with white.
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+    Color white(0xFFFFFFFF);
+    FloatRect fullRect(0, 0, 16, 16);
+    src->context()->fillRect(fullRect, white, DeviceColorSpace);
+
+    // Scroll down by 8 (coordinate system goes up).
+    src->context()->save();
+    src->context()->translate(0, -8);
+
+    Color red(0xFFFF0000);
+    Color green(0xFF00FF00);
+    {
+        // Make the transparency layer after translation will be @ (0, -8) with
+        // size 16x16.
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(0, 0, 16, 16));
+
+        // Draw a red pixel at (15, 15). This should be the at (15, 7) after
+        // the transform.
+        FloatRect bottomRight(15, 15, 1, 1);
+        helper.context()->fillRect(bottomRight, green, DeviceColorSpace);
+        helper.composite();
+    }
+
+    src->context()->restore();
+
+    // Check the pixel we wrote.
+    EXPECT_EQ(green, getPixelAt(src->context(), 15, 7));
+}
+
+// Same as OpaqueCompositeLayer, but the canvas has a rotation applied. This
+// tests that the propert transform is applied to the copied layer.
+TEST(TransparencyWin, RotateOpaqueCompositeLayer)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    // The background is white.
+    Color white(0xFFFFFFFF);
+    FloatRect fullRect(0, 0, 16, 16);
+    src->context()->fillRect(fullRect, white, DeviceColorSpace);
+
+    // Rotate the image by 90 degrees. This matrix is the same as
+    // cw90.rotate(90); but avoids rounding errors. Rounding errors can cause
+    // Skia to think that !rectStaysRect() and it will fall through to path
+    // drawing mode, which in turn gives us antialiasing. We want no
+    // antialiasing or other rounding problems since we're testing exact pixel
+    // values.
+    src->context()->save();
+    TransformationMatrix cw90(0, 1, 0, 0,
+                              -1, 0, 0, 0,
+                              0, 0, 1, 0,
+                              0, 0, 0, 1);
+    src->context()->concatCTM(cw90);
+
+    // Make a transparency layer consisting of a horizontal line of 50% black.
+    // Since the rotation is applied, this will actually be a vertical line
+    // down the middle of the image.
+    src->context()->beginTransparencyLayer(0.5);
+    FloatRect blackRect(0, -9, 16, 2);
+    Color black(0xFF000000);
+    src->context()->fillRect(blackRect, black, DeviceColorSpace);
+
+    // Now draw 50% red square.
+    {
+        // Create a transparency helper inset one pixel in the buffer. The
+        // coordinates are before transforming into this space, and maps to
+        // IntRect(1, 1, 14, 14).
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::Untransform,
+                    IntRect(1, -15, 14, 14));
+
+        // Fill with red.
+        helper.context()->fillRect(helper.drawRect(), Color(0x7f7f0000), DeviceColorSpace);
+        clearTopLayerAlphaChannel(helper.context());
+        helper.composite();
+    }
+
+    // Finish the compositing.
+    src->context()->endTransparencyLayer();
+
+    // Top corner should be the original background.
+    EXPECT_EQ(white, getPixelAt(src->context(), 0, 0));
+
+    // Check the stripe down the middle, first at the top...
+    Color gray(0xFF808080);
+    EXPECT_EQ(white, getPixelAt(src->context(), 6, 0));
+    EXPECT_EQ(gray, getPixelAt(src->context(), 7, 0));
+    EXPECT_EQ(gray, getPixelAt(src->context(), 8, 0));
+    EXPECT_EQ(white, getPixelAt(src->context(), 9, 0));
+
+    // ...now at the bottom.
+    EXPECT_EQ(white, getPixelAt(src->context(), 6, 15));
+    EXPECT_EQ(gray, getPixelAt(src->context(), 7, 15));
+    EXPECT_EQ(gray, getPixelAt(src->context(), 8, 15));
+    EXPECT_EQ(white, getPixelAt(src->context(), 9, 15));
+
+    // Our red square should be 25% red over the top of those two.
+    Color redwhite(0xFFdfbfbf);
+    Color redgray(0xFF9f8080);
+    EXPECT_EQ(white, getPixelAt(src->context(), 0, 1));
+    EXPECT_EQ(redwhite, getPixelAt(src->context(), 1, 1));
+    EXPECT_EQ(redwhite, getPixelAt(src->context(), 6, 1));
+    EXPECT_EQ(redgray, getPixelAt(src->context(), 7, 1));
+    EXPECT_EQ(redgray, getPixelAt(src->context(), 8, 1));
+    EXPECT_EQ(redwhite, getPixelAt(src->context(), 9, 1));
+    EXPECT_EQ(redwhite, getPixelAt(src->context(), 14, 1));
+    EXPECT_EQ(white, getPixelAt(src->context(), 15, 1));
+
+    // Complete the 50% transparent layer.
+    src->context()->restore();
+}
+
+TEST(TransparencyWin, TranslateScaleOpaqueCompositeLayer)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    // The background is white on top with red on bottom.
+    Color white(0xFFFFFFFF);
+    FloatRect topRect(0, 0, 16, 8);
+    src->context()->fillRect(topRect, white, DeviceColorSpace);
+    Color red(0xFFFF0000);
+    FloatRect bottomRect(0, 8, 16, 8);
+    src->context()->fillRect(bottomRect, red, DeviceColorSpace);
+
+    src->context()->save();
+
+    // Translate left by one pixel.
+    TransformationMatrix left;
+    left.translate(-1, 0);
+
+    // Scale by 2x.
+    TransformationMatrix scale;
+    scale.scale(2.0);
+    src->context()->concatCTM(scale);
+
+    // Then translate up by one pixel (which will actually be 2 due to scaling).
+    TransformationMatrix up;
+    up.translate(0, -1);
+    src->context()->concatCTM(up);
+
+    // Now draw 50% red square.
+    {
+        // Create a transparency helper inset one pixel in the buffer. The
+        // coordinates are before transforming into this space, and maps to
+        // IntRect(1, 1, 14, 14).
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::KeepTransform,
+                    IntRect(1, -15, 14, 14));
+
+        // Fill with red.
+        helper.context()->fillRect(helper.drawRect(), Color(0x7f7f0000), DeviceColorSpace);
+        clearTopLayerAlphaChannel(helper.context());
+        helper.composite();
+    }
+}
+
+// Tests scale mode with no additional copy.
+TEST(TransparencyWin, Scale)
+{
+    // Create an opaque white buffer.
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+    FloatRect fullBuffer(0, 0, 16, 16);
+    src->context()->fillRect(fullBuffer, Color::white, DeviceColorSpace);
+
+    // Scale by 2x.
+    src->context()->save();
+    TransformationMatrix scale;
+    scale.scale(2.0);
+    src->context()->concatCTM(scale);
+
+    // Start drawing a rectangle from 1->4. This should get scaled to 2->8.
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::NoLayer,
+                    TransparencyWin::ScaleTransform,
+                    IntRect(1, 1, 3, 3));
+
+        // The context should now have the identity transform and the returned
+        // rect should be scaled.
+        EXPECT_TRUE(helper.context()->getCTM().isIdentity());
+        EXPECT_EQ(2, helper.drawRect().x());
+        EXPECT_EQ(2, helper.drawRect().y());
+        EXPECT_EQ(8, helper.drawRect().right());
+        EXPECT_EQ(8, helper.drawRect().bottom());
+
+        // Set the pixel at (2, 2) to be transparent. This should be fixed when
+        // the helper goes out of scope. We don't want to call
+        // clearTopLayerAlphaChannel because that will actually clear the whole
+        // canvas (since we have no extra layer!).
+        SkBitmap& bitmap = const_cast<SkBitmap&>(helper.context()->
+            platformContext()->canvas()->getTopPlatformDevice().
+            accessBitmap(false));
+        *bitmap.getAddr32(2, 2) &= 0x00FFFFFF;
+        helper.composite();
+    }
+
+    src->context()->restore();
+
+    // Check the pixel we previously made transparent, it should have gotten
+    // fixed back up to white.
+
+    // The current version doesn't fixup transparency when there is no layer.
+    // This seems not to be necessary, so we don't bother, but if it becomes
+    // necessary, this line should be uncommented.
+    // EXPECT_EQ(Color(Color::white), getPixelAt(src->context(), 2, 2));
+}
+
+// Tests scale mode with an additional copy for transparency. This will happen
+// if we have a scaled textbox, for example. WebKit will create a new
+// transparency layer, draw the text field, then draw the text into it, then
+// composite this down with an opacity.
+TEST(TransparencyWin, ScaleTransparency)
+{
+    // Create an opaque white buffer.
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+    FloatRect fullBuffer(0, 0, 16, 16);
+    src->context()->fillRect(fullBuffer, Color::white, DeviceColorSpace);
+
+    // Make another layer (which duplicates how WebKit will make this). We fill
+    // the top half with red, and have the layer be 50% opaque.
+    src->context()->beginTransparencyLayer(0.5);
+    FloatRect topHalf(0, 0, 16, 8);
+    src->context()->fillRect(topHalf, Color(0xFFFF0000), DeviceColorSpace);
+
+    // Scale by 2x.
+    src->context()->save();
+    TransformationMatrix scale;
+    scale.scale(2.0);
+    src->context()->concatCTM(scale);
+
+    // Make a layer inset two pixels (because of scaling, this is 2->14). And
+    // will it with 50% black.
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::OpaqueCompositeLayer,
+                    TransparencyWin::ScaleTransform,
+                    IntRect(1, 1, 6, 6));
+
+        helper.context()->fillRect(helper.drawRect(), Color(0x7f000000), DeviceColorSpace);
+        clearTopLayerAlphaChannel(helper.context());
+        helper.composite();
+    }
+
+    // Finish the layer.
+    src->context()->restore();
+    src->context()->endTransparencyLayer();
+
+    Color redBackground(0xFFFF8080); // 50% red composited on white.
+    EXPECT_EQ(redBackground, getPixelAt(src->context(), 0, 0));
+    EXPECT_EQ(redBackground, getPixelAt(src->context(), 1, 1));
+
+    // Top half (minus two pixel border) should be 50% gray atop opaque
+    // red = 0xFF804141. Then that's composited with 50% transparency on solid
+    // white = 0xFFC0A1A1.
+    Color darkRed(0xFFBF8080);
+    EXPECT_EQ(darkRed, getPixelAt(src->context(), 2, 2));
+    EXPECT_EQ(darkRed, getPixelAt(src->context(), 7, 7));
+
+    // Bottom half (minus a two pixel border) should be a layer with 5% gray
+    // with another 50% opacity composited atop white.
+    Color darkWhite(0xFFBFBFBF);
+    EXPECT_EQ(darkWhite, getPixelAt(src->context(), 8, 8));
+    EXPECT_EQ(darkWhite, getPixelAt(src->context(), 13, 13));
+
+    Color white(0xFFFFFFFF); // Background in the lower-right.
+    EXPECT_EQ(white, getPixelAt(src->context(), 14, 14));
+    EXPECT_EQ(white, getPixelAt(src->context(), 15, 15));
+}
+
+TEST(TransparencyWin, Text)
+{
+    OwnPtr<ImageBuffer> src(ImageBuffer::create(IntSize(16, 16), DeviceRGB));
+
+    // Our text should end up 50% transparent blue-green.
+    Color fullResult(0x80008080);
+
+    {
+        TransparencyWin helper;
+        helper.init(src->context(),
+                    TransparencyWin::TextComposite,
+                    TransparencyWin::KeepTransform,
+                    IntRect(0, 0, 16, 16));
+        helper.setTextCompositeColor(fullResult);
+
+        // Write several different squares to simulate ClearType. These should
+        // all reduce to 2/3 coverage.
+        FloatRect pixel(0, 0, 1, 1);
+        helper.context()->fillRect(pixel, 0xFFFF0000, DeviceColorSpace);
+        pixel.move(1.0f, 0.0f);
+        helper.context()->fillRect(pixel, 0xFF00FF00, DeviceColorSpace);
+        pixel.move(1.0f, 0.0f);
+        helper.context()->fillRect(pixel, 0xFF0000FF, DeviceColorSpace);
+        pixel.move(1.0f, 0.0f);
+        helper.context()->fillRect(pixel, 0xFF008080, DeviceColorSpace);
+        pixel.move(1.0f, 0.0f);
+        helper.context()->fillRect(pixel, 0xFF800080, DeviceColorSpace);
+        pixel.move(1.0f, 0.0f);
+        helper.context()->fillRect(pixel, 0xFF808000, DeviceColorSpace);
+
+        // Try one with 100% coverage (opaque black).
+        pixel.move(1.0f, 0.0f);
+        helper.context()->fillRect(pixel, 0xFF000000, DeviceColorSpace);
+
+        // Now mess with the alpha channel.
+        clearTopLayerAlphaChannel(helper.context());
+        helper.composite();
+    }
+
+    Color oneThirdResult(0x55005555); // = fullResult * 2 / 3
+    EXPECT_EQ(oneThirdResult, getPixelAt(src->context(), 0, 0));
+    EXPECT_EQ(oneThirdResult, getPixelAt(src->context(), 1, 0));
+    EXPECT_EQ(oneThirdResult, getPixelAt(src->context(), 2, 0));
+    EXPECT_EQ(oneThirdResult, getPixelAt(src->context(), 3, 0));
+    EXPECT_EQ(oneThirdResult, getPixelAt(src->context(), 4, 0));
+    EXPECT_EQ(oneThirdResult, getPixelAt(src->context(), 5, 0));
+    EXPECT_EQ(fullResult, getPixelAt(src->context(), 6, 0));
+    EXPECT_EQ(Color::transparent, getPixelAt(src->context(), 7, 0));
+}
+
+} // namespace WebCore
diff --git a/WebKit/chromium/tests/UniscribeHelperTest.cpp b/WebKit/chromium/tests/UniscribeHelperTest.cpp
new file mode 100644
index 0000000..8aaed11
--- /dev/null
+++ b/WebKit/chromium/tests/UniscribeHelperTest.cpp
@@ -0,0 +1,165 @@
+/*
+ * Copyright (C) 2010 Google 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:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+
+#include <gtest/gtest.h>
+
+#include "PlatformString.h"
+#include "UniscribeHelper.h"
+
+using namespace WebCore;
+
+namespace {
+
+class UniscribeTest : public testing::Test {
+public:
+    UniscribeTest()
+    {
+    }
+
+    // Returns an HFONT with the given name. The caller does not have to free
+    // this, it will be automatically freed at the end of the test. Returns 0
+    // on failure. On success, the
+    HFONT MakeFont(const wchar_t* fontName, SCRIPT_CACHE** cache)
+    {
+        LOGFONT lf;
+        memset(&lf, 0, sizeof(LOGFONT));
+        lf.lfHeight = 20;
+        wcscpy_s(lf.lfFaceName, fontName);
+
+        HFONT hfont = CreateFontIndirect(&lf);
+        if (!hfont)
+            return 0;
+
+        *cache = new SCRIPT_CACHE;
+        **cache = 0;
+        createdFonts.append(std::make_pair(hfont, *cache));
+        return hfont;
+    }
+
+protected:
+    // Default font properties structure for tests to use.
+    SCRIPT_FONTPROPERTIES properties;
+
+private:
+    virtual void SetUp()
+    {
+        memset(&properties, 0, sizeof(SCRIPT_FONTPROPERTIES));
+        properties.cBytes = sizeof(SCRIPT_FONTPROPERTIES);
+        properties.wgBlank = ' ';
+        properties.wgDefault = '?'; // Used when the char is not in the font.
+        properties.wgInvalid = '#'; // Used for invalid characters.
+    }
+
+    virtual void TearDown()
+    {
+        // Free any allocated fonts.
+        for (size_t i = 0; i < createdFonts.size(); i++) {
+            DeleteObject(createdFonts[i].first);
+            ScriptFreeCache(createdFonts[i].second);
+            delete createdFonts[i].second;
+        }
+        createdFonts.clear();
+    }
+
+    // Tracks allocated fonts so we can delete them at the end of the test.
+    // The script cache pointer is heap allocated and must be freed.
+    Vector< std::pair<HFONT, SCRIPT_CACHE*> > createdFonts;
+};
+
+} // namespace
+
+// This test tests giving Uniscribe a very large buffer, which will cause a
+// failure.
+TEST_F(UniscribeTest, TooBig)
+{
+    // Make a large string with an e with a zillion combining accents.
+    String input(L"e");
+    for (int i = 0; i < 100000; i++)
+        input.append(static_cast<UChar>(0x301)); // Combining acute accent.
+
+    SCRIPT_CACHE* scriptCache;
+    HFONT hfont = MakeFont(L"Times New Roman", &scriptCache);
+    ASSERT_TRUE(hfont);
+
+    // Test a long string without the normal length protection we have. This
+    // will cause shaping to fail.
+    {
+        UniscribeHelper uniscribe(
+            input.characters(), static_cast<int>(input.length()),
+            false, hfont, scriptCache, &properties);
+        uniscribe.initWithOptionalLengthProtection(false);
+
+        // There should be one shaping entry, with nothing in it.
+        ASSERT_EQ(1, uniscribe.m_shapes.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_glyphs.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_logs.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_visualAttributes.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_advance.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_offsets.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_justify.size());
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_abc.abcA);
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_abc.abcB);
+        EXPECT_EQ(0, uniscribe.m_shapes[0].m_abc.abcC);
+
+        // The sizes of the other stuff should match the shaping entry.
+        EXPECT_EQ(1, uniscribe.m_runs.size());
+        EXPECT_EQ(1, uniscribe.m_screenOrder.size());
+
+        // Check that the various querying functions handle the empty case
+        // properly.
+        EXPECT_EQ(0, uniscribe.width());
+        EXPECT_EQ(0, uniscribe.firstGlyphForCharacter(0));
+        EXPECT_EQ(0, uniscribe.firstGlyphForCharacter(1000));
+        EXPECT_EQ(0, uniscribe.xToCharacter(0));
+        EXPECT_EQ(0, uniscribe.xToCharacter(1000));
+    }
+
+    // Now test the very large string and make sure it is handled properly by
+    // the length protection.
+    {
+        UniscribeHelper uniscribe(
+            input.characters(), static_cast<int>(input.length()),
+            false, hfont, scriptCache, &properties);
+        uniscribe.initWithOptionalLengthProtection(true);
+
+        // There should be 0 runs and shapes.
+        EXPECT_EQ(0, uniscribe.m_runs.size());
+        EXPECT_EQ(0, uniscribe.m_shapes.size());
+        EXPECT_EQ(0, uniscribe.m_screenOrder.size());
+
+        EXPECT_EQ(0, uniscribe.width());
+        EXPECT_EQ(0, uniscribe.firstGlyphForCharacter(0));
+        EXPECT_EQ(0, uniscribe.firstGlyphForCharacter(1000));
+        EXPECT_EQ(0, uniscribe.xToCharacter(0));
+        EXPECT_EQ(0, uniscribe.xToCharacter(1000));
+    }
+}

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list