Remove unsafe getFontMetrics methods

third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp

 /*
  * Copyright (C) 2006 Oliver Hunt <ojh16@student.canterbury.ac.nz>
  * Copyright (C) 2006 Apple Computer Inc.
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2008 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2010. 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
  * License as published by the Free Software Foundation; either
@@ skipping 133 matching lines @@
 
   return it->value.hasX() || it->value.hasY();
 }
 
 PositionWithAffinity LayoutSVGInlineText::positionForPoint(
     const LayoutPoint& point) {
   if (!hasTextBoxes() || !textLength())
     return createPositionWithAffinity(0);
 
   ASSERT(m_scalingFactor);
+
+  const SimpleFontData* fontData = m_scaledFont.primaryFont();
+  DCHECK(fontData);
   float baseline =
-      m_scaledFont.getFontMetrics().floatAscent() / m_scalingFactor;
+      fontData ? fontData->getFontMetrics().floatAscent() / m_scalingFactor : 0;
 
   LayoutBlock* containingBlock = this->containingBlock();
   ASSERT(containingBlock);
 
   // Map local point to absolute point, as the character origins stored in the
   // text fragments use absolute coordinates.
   FloatPoint absolutePoint(point);
   absolutePoint.moveBy(containingBlock->location());
 
   float closestDistance = std::numeric_limits<float>::max();
@@ skipping 114 matching lines @@
 }
 
 }  // namespace
 
 void LayoutSVGInlineText::addMetricsFromRun(const TextRun& run,
                                             bool& lastCharacterWasWhiteSpace) {
   Vector<CharacterRange> charRanges =
       scaledFont().individualCharacterRanges(run);
   synthesizeGraphemeWidths(run, charRanges);
 
+  const SimpleFontData* fontData = scaledFont().primaryFont();
+  DCHECK(fontData);
+  if (!fontData)
+    return;
+
   const float cachedFontHeight =
-      scaledFont().getFontMetrics().floatHeight() / m_scalingFactor;
+      fontData->getFontMetrics().floatHeight() / m_scalingFactor;
   const bool preserveWhiteSpace = styleRef().whiteSpace() == PRE;
   const unsigned runLength = run.length();
 
   // TODO(pdr): Character-based iteration is ambiguous and error-prone. It
   // should be unified under a single concept. See: https://crbug.com/593570
   unsigned characterIndex = 0;
   while (characterIndex < runLength) {
     bool currentCharacterIsWhiteSpace = run[characterIndex] == ' ';
     if (!preserveWhiteSpace && lastCharacterWasWhiteSpace &&
         currentCharacterIsWhiteSpace) {
@@ skipping 101 matching lines @@
 }
 
 PassRefPtr<StringImpl> LayoutSVGInlineText::originalText() const {
   RefPtr<StringImpl> result = LayoutText::originalText();
   if (!result)
     return nullptr;
   return normalizeWhitespace(result);
 }
 
 }  // namespace blink