Set caret width based on device scale factor

third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 1160 matching lines @@
         x += textIndentOffset;
       break;
     case AlignCenter:
       x = (x + maxX) / 2;
       if (currentStyle.isLeftToRightDirection())
         x += textIndentOffset / 2;
       else
         x -= textIndentOffset / 2;
       break;
     case AlignRight:
-      x = maxX - caretWidth();
+      x = maxX - caretWidth(frameView()->getHostWindow());

[oshima, 2017/02/10 19:55:16]

can you just get once in this function?

[malaykeshav, 2017/02/10 21:00:54]

Done

       if (!currentStyle.isLeftToRightDirection())
         x -= textIndentOffset;
       break;
   }
-  x = std::min(x, (maxX - caretWidth()).clampNegativeToZero());
+  x = std::min(
+      x,
+      (maxX - caretWidth(frameView()->getHostWindow())).clampNegativeToZero());
 
   const Font& font = style()->font();
   const SimpleFontData* fontData = font.primaryFont();
   LayoutUnit height;
   // crbug.com/595692 This check should not be needed but sometimes
   // primaryFont is null.
   if (fontData)
     height = LayoutUnit(fontData->getFontMetrics().height());
   LayoutUnit verticalSpace =
       lineHeight(true, currentStyle.isHorizontalWritingMode() ? HorizontalLine
                                                               : VerticalLine,
                  PositionOfInteriorLineBoxes) -
       height;
   LayoutUnit y = paddingTop() + borderTop() + (verticalSpace / 2);
   return currentStyle.isHorizontalWritingMode()
-             ? LayoutRect(x, y, caretWidth(), height)
-             : LayoutRect(y, x, height, caretWidth());
+             ? LayoutRect(x, y, caretWidth(frameView()->getHostWindow()),
+                          height)
+             : LayoutRect(y, x, height,
+                          caretWidth(frameView()->getHostWindow()));
 }
 
 const LayoutObject* LayoutBoxModelObject::pushMappingToContainer(
     const LayoutBoxModelObject* ancestorToStopAt,
     LayoutGeometryMap& geometryMap) const {
   ASSERT(ancestorToStopAt != this);
 
   AncestorSkipInfo skipInfo(ancestorToStopAt);
   LayoutObject* container = this->container(&skipInfo);
   if (!container)
@@ skipping 133 matching lines @@
   if (rootElementStyle->hasBackground())
     return false;
 
   if (node() != document().firstBodyElement())
     return false;
 
   return true;
 }
 
 }  // namespace blink