Scale focus outline thickness with zoom level.

third_party/WebKit/Source/platform/graphics/GraphicsContext.h

 /*
  * Copyright (C) 2003, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008-2009 Torch Mobile, Inc.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 286 matching lines @@
                  DrawLooperBuilder::ShadowAlphaMode =
                      DrawLooperBuilder::ShadowRespectsAlpha,
                  ShadowMode = DrawShadowAndForeground);
 
   // It is assumed that this draw looper is used only for shadows
   // (i.e. a draw looper is set if and only if there is a shadow).
   // The builder passed into this method will be destroyed.
   void setDrawLooper(std::unique_ptr<DrawLooperBuilder>);
 
   void drawFocusRing(const Vector<IntRect>&,
-                     int width,
+                     float width,
                      int offset,
                      const Color&);
-  void drawFocusRing(const Path&, int width, int offset, const Color&);
+  void drawFocusRing(const Path&, float width, int offset, const Color&);
 
   enum Edge {
     NoEdge = 0,
     TopEdge = 1 << 1,
     RightEdge = 1 << 2,
     BottomEdge = 1 << 3,
     LeftEdge = 1 << 4
   };
   typedef unsigned Edges;
   void drawInnerShadow(const FloatRoundedRect&,
@@ skipping 30 matching lines @@
   void setURLDestinationLocation(const String& name, const IntPoint&);
 
   static void adjustLineToPixelBoundaries(FloatPoint& p1,
                                           FloatPoint& p2,
                                           float strokeWidth,
                                           StrokeStyle);
 
   static int focusRingOutsetExtent(int offset, int width) {
     // Unlike normal outlines (whole width is outside of the offset), focus rings are drawn with the
     // center of the path aligned with the offset, so only half of the width is outside of the offset.
     return focusRingOffset(offset) + (focusRingWidth(width) + 1) / 2;

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

Change this call site to focusRingWidth also?

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

Done. Not sure what this callsite is for, something about layout, but now it
follows the same logic as the rest of the change.

   }
 
 #if OS(MACOSX)
   static int focusRingWidth(int width) { return width; }

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

change to float?

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

This is deleted.

 #else
   static int focusRingWidth(int width) { return 1; }

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

Doesn't this defeat the purpose of this feature?

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

I just forgot to delete this block. I moved this into the
ObjectPainter/ComputedStyle layer and GraphicsContext just uses the width it's
given. The "return 1" is now "return effectiveZoom()".

 #endif
 
 #if DCHECK_IS_ON()
   void setInDrawingRecorder(bool);
 #endif
 
   static sk_sp<SkColorFilter> WebCoreColorFilterToSkiaColorFilter(ColorFilter);
 
  private:
   const GraphicsContextState* immutableState() const { return m_paintState; }
@@ skipping 14 matching lines @@
   static SkPMColor antiColors1(int);
   static SkPMColor antiColors2(int);
   static void draw1xMarker(SkBitmap*, int);
   static void draw2xMarker(SkBitmap*, int);
 #endif
 
   void saveLayer(const SkRect* bounds, const SkPaint*);
   void restoreLayer();
 
   // Helpers for drawing a focus ring (drawFocusRing)
-  void drawFocusRingPath(const SkPath&, const Color&, int width);
-  void drawFocusRingRect(const SkRect&, const Color&, int width);
+  void drawFocusRingPath(const SkPath&, const Color&, float width);
+  void drawFocusRingRect(const SkRect&, const Color&, float width);
 
   // SkCanvas wrappers.
   void clipRRect(const SkRRect&,
                  AntiAliasingMode = NotAntiAliased,
                  SkRegion::Op = SkRegion::kIntersect_Op);
   void concat(const SkMatrix&);
 
   // Apply deferred paint state saves
   void realizePaintSave() {
     if (contextDisabled())
@@ skipping 48 matching lines @@
 
   float m_deviceScaleFactor;
 
   unsigned m_printing : 1;
   unsigned m_hasMetaData : 1;
 };
 
 }  // namespace blink
 
 #endif  // GraphicsContext_h