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

Unified Diff: third_party/WebKit/LayoutTests/paint/invalidation/ruby-text-flipped-blocks-expected.txt

Issue 2150143002: Don't over-flip when mapping visual rects for Ruby text. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rework tests to use Ahem. 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
Index: third_party/WebKit/LayoutTests/paint/invalidation/ruby-text-flipped-blocks-expected.txt
diff --git a/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/clip-with-layout-delta-expected.txt b/third_party/WebKit/LayoutTests/paint/invalidation/ruby-text-flipped-blocks-expected.txt
similarity index 56%
copy from third_party/WebKit/LayoutTests/platform/mac/fast/repaint/clip-with-layout-delta-expected.txt
copy to third_party/WebKit/LayoutTests/paint/invalidation/ruby-text-flipped-blocks-expected.txt
index ba6178ad89d95e87ed0c0a9fc28d3f5892b3b26c..9c0309723194a8d71ea14dd2da73625c10cad4a3 100644
--- a/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/clip-with-layout-delta-expected.txt
+++ b/third_party/WebKit/LayoutTests/paint/invalidation/ruby-text-flipped-blocks-expected.txt
@@ -9,34 +9,29 @@
"drawsContent": true,
"paintInvalidations": [
{
- "object": "LayoutBlockFlow (relative positioned) DIV",
- "rect": [108, 8, 100, 100],
- "reason": "background obscuration change"
+ "object": "LayoutBlockFlow DIV",
+ "rect": [8, 8, 15, 10],
+ "reason": "forced by layout"
},
{
- "object": "LayoutBlockFlow SPAN id='t2' class='blue'",
- "rect": [108, 8, 100, 100],
+ "object": "LayoutRubyBase (anonymous)",
+ "rect": [8, 8, 10, 10],
"reason": "style change"
},
{
- "object": "LayoutBlockFlow DIV",
- "rect": [8, 8, 100, 100],
- "reason": "background obscuration change"
- },
- {
- "object": "LayoutBlockFlow SPAN id='t1'",
- "rect": [8, 8, 100, 100],
- "reason": "style change"
+ "object": "LayoutRubyRun (anonymous)",
+ "rect": [8, 8, 10, 10],
+ "reason": "forced by layout"
},
{
"object": "LayoutText #text",
- "rect": [108, 8, 4, 18],
- "reason": "forced by layout"
+ "rect": [8, 8, 10, 10],
+ "reason": "style change"
},
{
"object": "LayoutText #text",
- "rect": [8, 8, 4, 18],
- "reason": "forced by layout"
+ "rect": [18, 10, 5, 6],
+ "reason": "style change"
}
]
}
@@ -44,30 +39,26 @@
"objectPaintInvalidations": [
{
"object": "LayoutBlockFlow DIV",
- "reason": "background obscuration change"
- },
- {
- "object": "LayoutBlockFlow SPAN id='t1'",
- "reason": "style change"
+ "reason": "forced by layout"
},
{
"object": "RootInlineBox",
- "reason": "style change"
+ "reason": "forced by layout"
},
{
- "object": "LayoutText #text",
+ "object": "LayoutRubyRun (anonymous)",
"reason": "forced by layout"
},
{
- "object": "InlineTextBox '\u00A0'",
- "reason": "forced by layout"
+ "object": "LayoutText #text",
+ "reason": "style change"
},
{
- "object": "LayoutBlockFlow (relative positioned) DIV",
- "reason": "background obscuration change"
+ "object": "InlineTextBox 'c'",
+ "reason": "style change"
},
{
- "object": "LayoutBlockFlow SPAN id='t2' class='blue'",
+ "object": "LayoutRubyBase (anonymous)",
"reason": "style change"
},
{
@@ -76,11 +67,11 @@
},
{
"object": "LayoutText #text",
- "reason": "forced by layout"
+ "reason": "style change"
},
{
- "object": "InlineTextBox '\u00A0'",
- "reason": "forced by layout"
+ "object": "InlineTextBox 'b'",
+ "reason": "style change"
}
]
}

Powered by Google App Engine
This is Rietveld 408576698