Show DragImage when the dragged element or its parent has transform

Source/web/tests/WebFrameTest.cpp

 /*
  * 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
@@ skipping 39 matching lines @@
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/html/HTMLFormElement.h"
 #include "core/loader/FrameLoadRequest.h"
 #include "core/page/EventHandler.h"
 #include "core/rendering/HitTestResult.h"
 #include "core/rendering/RenderView.h"
 #include "core/rendering/TextAutosizer.h"
 #include "core/rendering/compositing/RenderLayerCompositor.h"
+#include "platform/DragImage.h"
 #include "platform/UserGestureIndicator.h"
 #include "platform/geometry/FloatRect.h"
 #include "platform/network/ResourceError.h"
 #include "platform/scroll/ScrollbarTheme.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebFloatRect.h"
 #include "public/platform/WebThread.h"
 #include "public/platform/WebURL.h"
 #include "public/platform/WebURLResponse.h"
 #include "public/platform/WebUnitTestSupport.h"
@@ skipping 5372 matching lines @@
 {
     registerMockedHttpURLLoad("link-manifest-change.html");
 
     ManifestChangeWebFrameClient webFrameClient;
     FrameTestHelpers::WebViewHelper webViewHelper;
     webViewHelper.initializeAndLoad(m_baseURL + "link-manifest-change.html", true, &webFrameClient);
 
     EXPECT_EQ(14, webFrameClient.manifestChangeCount());
 }
 
+TEST_F(WebFrameTest, NodeImageTest)
+{
+    registerMockedHttpURLLoad("nodeimage.html");
+
+    FixedLayoutTestWebViewClient client;
+    client.m_screenInfo.deviceScaleFactor = 1;
+    int viewportWidth = 100;
+    int viewportHeight = 100;
+
+    FrameTestHelpers::WebViewHelper webViewHelper;
+    webViewHelper.initializeAndLoad(m_baseURL + "nodeimage.html", true, 0, &client);
+    webViewHelper.webView()->settings()->setViewportEnabled(true);
+    webViewHelper.webView()->resize(WebSize(viewportWidth, viewportHeight));
+    webViewHelper.webView()->layout();
+
+
+    RefPtr<WebCore::LocalFrame> frame = webViewHelper.webViewImpl()->page()->mainFrame();
+
+    // Sub Test 1
+    {

[esprehn, 2014/06/04 18:13:45]

I think it's preferable to split the test into two separate TEST_F and have a
helper function that does the setup above.

[spartha, 2014/06/05 07:48:55]

Done.

+        SkBitmap bitmap;
+        ASSERT_TRUE(bitmap.allocN32Pixels(40, 40));
+        SkCanvas canvas(bitmap);
+        canvas.drawColor(SK_ColorGREEN);
+        WebCore::Element* element = frame->document()->getElementById("case-css-transform");
+        OwnPtr<WebCore::DragImage> dragImage = frame->nodeImage(*element);
+        EXPECT_TRUE(dragImage);
+        EXPECT_EQ(40, dragImage->size().width());
+        EXPECT_EQ(40, dragImage->size().height());
+        const SkBitmap& dragBitmap = dragImage->bitmap();
+        SkAutoLockPixels lockPixel(dragBitmap);
+        EXPECT_EQ(0, memcmp(bitmap.getPixels(), dragBitmap.getPixels(), bitmap.getSize()));
+    }
+
+    // Sub Test 2
+    {
+        SkBitmap bitmap;
+        SkPaint paint;
+        ASSERT_TRUE(bitmap.allocN32Pixels(20, 40));
+        SkCanvas canvas(bitmap);
+        canvas.drawColor(SK_ColorGREEN);
+        WebCore::Element* element = frame->document()->getElementById("case-css-3dtransform");
+        OwnPtr<WebCore::DragImage> dragImage = frame->nodeImage(*element);
+        EXPECT_TRUE(dragImage);
+        EXPECT_EQ(20, dragImage->size().width());
+        EXPECT_EQ(40, dragImage->size().height());
+        const SkBitmap& dragBitmap = dragImage->bitmap();
+        SkAutoLockPixels lockPixel(dragBitmap);
+        EXPECT_EQ(0, memcmp(bitmap.getPixels(), dragBitmap.getPixels(), bitmap.getSize()));
+    }
+
+}
+
 } // namespace