Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(999)

Unified Diff: third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainterTest.cpp

Issue 2151203002: Revert of Calculate correct cull rect for SVG inline text boxes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainter.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainterTest.cpp
diff --git a/third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainterTest.cpp b/third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainterTest.cpp
deleted file mode 100644
index 4c2f4d07127d3a1cb87e7f544503c9710982f62c..0000000000000000000000000000000000000000
--- a/third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainterTest.cpp
+++ /dev/null
@@ -1,123 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "core/paint/SVGInlineTextBoxPainter.h"
-
-#include "core/dom/Document.h"
-#include "core/dom/Range.h"
-#include "core/editing/DOMSelection.h"
-#include "core/frame/LocalDOMWindow.h"
-#include "core/layout/LayoutTestHelper.h"
-#include "core/layout/line/InlineTextBox.h"
-#include "core/layout/svg/LayoutSVGInlineText.h"
-#include "core/layout/svg/LayoutSVGText.h"
-#include "core/paint/PaintLayer.h"
-#include "platform/graphics/GraphicsLayer.h"
-#include "platform/graphics/paint/DisplayItemList.h"
-#include "platform/graphics/paint/DrawingDisplayItem.h"
-#include "platform/graphics/paint/PaintController.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace blink {
-namespace {
-
-class SVGInlineTextBoxPainterTest : public RenderingTest {
-public:
- const DrawingDisplayItem* getDrawingForSVGTextById(const char* elementName)
- {
- // Look up the inline text box that serves as the display item client for the painted text.
- LayoutSVGText* targetSVGText = toLayoutSVGText(
- document().getElementById(AtomicString(elementName))->layoutObject());
- LayoutSVGInlineText* targetInlineText = targetSVGText->descendantTextNodes()[0];
- const DisplayItemClient* targetClient = static_cast<const DisplayItemClient*>(targetInlineText->firstTextBox());
-
- // Find the appropriate drawing in the display item list.
- const DisplayItemList& displayItemList = rootPaintController().getDisplayItemList();
- for (size_t i = 0; i < displayItemList.size(); i++) {
- if (displayItemList[i].client() == *targetClient)
- return static_cast<const DrawingDisplayItem*>(&displayItemList[i]);
- }
-
- return nullptr;
- }
-
-
- void selectAllText()
- {
- Range* range = document().createRange();
- range->selectNode(document().documentElement());
- LocalDOMWindow* window = document().domWindow();
- DOMSelection* selection = window->getSelection();
- selection->removeAllRanges();
- selection->addRange(range);
- }
-
-private:
- PaintController& rootPaintController()
- {
- return document().view()->layoutView()->layer()->graphicsLayerBacking()->getPaintController();
- }
-
- void SetUp() override
- {
- RenderingTest::SetUp();
- enableCompositing();
- }
-};
-
-static void assertTextDrawingEquals(const DrawingDisplayItem* drawingDisplayItem, const char* str)
-{
- ASSERT_EQ(str, static_cast<const InlineTextBox*>(&drawingDisplayItem->client())->text());
-}
-
-static void assertCullRectEquals(const DrawingDisplayItem* drawingDisplayItem, const IntRect& expectedRect)
-{
- ASSERT_EQ(expectedRect, IntRect(drawingDisplayItem->picture()->cullRect()));
-}
-
-TEST_F(SVGInlineTextBoxPainterTest, TextCullRect_DefaultWritingMode)
-{
- setBodyInnerHTML(
- "<svg width='400px' height='400px' font-family='Arial' font-size='30'>"
- "<text id='target' x='50' y='30'>x</text>"
- "</svg>");
- document().view()->updateAllLifecyclePhases();
-
- const DrawingDisplayItem* drawingDisplayItem = getDrawingForSVGTextById("target");
- assertTextDrawingEquals(drawingDisplayItem, "x");
- assertCullRectEquals(drawingDisplayItem, IntRect(50, 3, 15, 33));
-
- selectAllText();
- document().view()->updateAllLifecyclePhases();
-
- drawingDisplayItem = getDrawingForSVGTextById("target");
- assertTextDrawingEquals(drawingDisplayItem, "x");
- assertCullRectEquals(drawingDisplayItem, IntRect(50, 3, 15, 33));
-}
-
-TEST_F(SVGInlineTextBoxPainterTest, TextCullRect_WritingModeTopToBottom)
-{
- setBodyInnerHTML(
- "<svg width='400px' height='400px' font-family='Arial' font-size='30'>"
- "<text id='target' x='50' y='30' writing-mode='tb'>x</text>"
- "</svg>");
- document().view()->updateAllLifecyclePhases();
-
- const DrawingDisplayItem* drawingDisplayItem = getDrawingForSVGTextById("target");
- assertTextDrawingEquals(drawingDisplayItem, "x");
- assertCullRectEquals(drawingDisplayItem, IntRect(33, 30, 34, 15));
-
- selectAllText();
- document().view()->updateAllLifecyclePhases();
-
- // The selection rect is one pixel taller due to sub-pixel difference
- // between the text bounds and selection bounds in combination with use of
- // enclosingIntRect() in SVGInlineTextBox::localSelectionRect().
- drawingDisplayItem = getDrawingForSVGTextById("target");
- assertTextDrawingEquals(drawingDisplayItem, "x");
- assertCullRectEquals(drawingDisplayItem, IntRect(33, 30, 34, 16));
-}
-
-} // namespace
-} // namespace blink
« no previous file with comments | « third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainter.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698