Scale focus outline thickness with zoom level.

third_party/WebKit/Source/core/paint/ObjectPainter.cpp

 // Copyright 2014 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/ObjectPainter.h"
 
 #include "core/layout/LayoutBlock.h"
 #include "core/layout/LayoutInline.h"
 #include "core/layout/LayoutObject.h"
 #include "core/layout/LayoutTheme.h"
@@ skipping 199 matching lines @@
   path.lineTo(quad[3]);
   SkPaint paint(context.fillPaint());
   paint.setAntiAlias(antialias);
   paint.setColor(color.rgb());
 
   context.drawPath(path, paint);
 }
 
 }  // namespace
 
+float ObjectPainter::getOutlineStrokeWidth(const ComputedStyle& style) {
+#if OS(MACOSX)
+  return style.outlineWidth();

[chrishtr, 2016/10/07 21:14:21]

Why this special case?

Also, it apears there is the opposite heuristic in GraphicsContext.h?

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/graph...

[Bret, 2016/10/08 01:32:40]

This is the same case, just moved around. See my other comment.

+#else
+  // Draw an outline with thickness in proportion to the zoom level, but never
+  // less than 1 pixel so that it remains visible.
+  return std::max(style.effectiveZoom(), 1.f);
+#endif
+}
+
 void ObjectPainter::paintOutline(const PaintInfo& paintInfo,
                                  const LayoutPoint& paintOffset) {
   ASSERT(shouldPaintSelfOutline(paintInfo.phase));
 
   const ComputedStyle& styleToUse = m_layoutObject.styleRef();
   if (!styleToUse.hasOutline() ||
       styleToUse.visibility() != EVisibility::Visible)
     return;
 
   // Only paint the focus ring by hand if the theme isn't able to draw the focus
@@ skipping 33 matching lines @@
   IntRect unitedOutlineRect = unionRectEvenIfEmpty(pixelSnappedOutlineRects);
   IntRect bounds = unitedOutlineRect;
   bounds.inflate(m_layoutObject.styleRef().outlineOutsetExtent());
   LayoutObjectDrawingRecorder recorder(paintInfo.context, m_layoutObject,
                                        paintInfo.phase, bounds);
 
   Color color =
       m_layoutObject.resolveColor(styleToUse, CSSPropertyOutlineColor);
   if (styleToUse.outlineStyleIsAuto()) {
     paintInfo.context.drawFocusRing(pixelSnappedOutlineRects,
-                                    styleToUse.outlineWidth(),
+                                    getOutlineStrokeWidth(styleToUse),
                                     styleToUse.outlineOffset(), color);
     return;
   }
 
   if (unitedOutlineRect == pixelSnappedOutlineRects[0]) {
     paintSingleRectangleOutline(paintInfo, unitedOutlineRect, styleToUse,
                                 color);
     return;
   }
   paintComplexOutline(paintInfo.context, pixelSnappedOutlineRects, styleToUse,
@@ skipping 403 matching lines @@
   m_layoutObject.paint(info, paintOffset);
   info.phase = PaintPhaseFloat;
   m_layoutObject.paint(info, paintOffset);
   info.phase = PaintPhaseForeground;
   m_layoutObject.paint(info, paintOffset);
   info.phase = PaintPhaseOutline;
   m_layoutObject.paint(info, paintOffset);
 }
 
 }  // namespace blink