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

Unified Diff: third_party/WebKit/Source/core/page/SpatialNavigation.cpp

Issue 2640143005: Support subpixel layout for borders. (Closed)
Patch Set: Rebaselined tests. Created 3 years, 10 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/page/SpatialNavigation.cpp
diff --git a/third_party/WebKit/Source/core/page/SpatialNavigation.cpp b/third_party/WebKit/Source/core/page/SpatialNavigation.cpp
index 95db920d452304e9feb116c80118cf89c8429a38..d3782c1ad726872177f7c57ae9d145c61796c2e1 100644
--- a/third_party/WebKit/Source/core/page/SpatialNavigation.cpp
+++ b/third_party/WebKit/Source/core/page/SpatialNavigation.cpp
@@ -419,12 +419,12 @@ LayoutRect nodeRectInAbsoluteCoordinates(Node* node, bool ignoreBorder) {
if (ignoreBorder) {
rect.move(node->layoutObject()->style()->borderLeftWidth(),
node->layoutObject()->style()->borderTopWidth());
- rect.setWidth(rect.width() -
- node->layoutObject()->style()->borderLeftWidth() -
- node->layoutObject()->style()->borderRightWidth());
- rect.setHeight(rect.height() -
- node->layoutObject()->style()->borderTopWidth() -
- node->layoutObject()->style()->borderBottomWidth());
+ rect.setWidth(LayoutUnit(
+ rect.width() - node->layoutObject()->style()->borderLeftWidth() -
+ node->layoutObject()->style()->borderRightWidth()));
+ rect.setHeight(LayoutUnit(
+ rect.height() - node->layoutObject()->style()->borderTopWidth() -
+ node->layoutObject()->style()->borderBottomWidth()));
}
return rect;
}
« no previous file with comments | « third_party/WebKit/Source/core/layout/line/InlineFlowBox.h ('k') | third_party/WebKit/Source/core/paint/BoxBorderPainter.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698