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

Unified Diff: third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc

Issue 2867293002: [LayoutNG] Compute baseline from fragment tree (Closed)
Patch Set: Cleanup Created 3 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/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc b/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc
index 3e1b38f2f9f8cb826c84f07b8f64e94709c4d53c..ef10d5879c96aa1082af089953614eb7b9479092 100644
--- a/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc
+++ b/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc
@@ -19,4 +19,11 @@ NGPhysicalLineBoxFragment::NGPhysicalLineBoxFragment(
children_.swap(children);
}
+LayoutUnit NGPhysicalLineBoxFragment::BaselinePosition(FontBaseline) const {
+ // TODO(kojii): Computing other baseline types than the used one is not
+ // implemented yet.
+ // TODO(kojii): We might need locale/script to look up OpenType BASE table.
+ return metrics_.ascent;
+}
+
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698