Remove unsafe getFontMetrics methods

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

 /*
  * Copyright (C) Research In Motion Limited 2010-2012. 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
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 448 matching lines @@
   }
   return glyphExtents;
 }
 
 static inline FloatRect calculateGlyphBoundaries(
     const QueryData* queryData,
     const SVGTextFragment& fragment,
     int startPosition) {
   const float scalingFactor = queryData->textLineLayout.scalingFactor();
   ASSERT(scalingFactor);
+  const SimpleFontData* fontData =
+      queryData->textLineLayout.scaledFont().primaryFont();
+  DCHECK(fontData);
+  if (!fontData)
+    return FloatRect();
+
   const float baseline =
-      queryData->textLineLayout.scaledFont().getFontMetrics().floatAscent() /
-      scalingFactor;
-
+      fontData->getFontMetrics().floatAscent() / scalingFactor;
   float glyphOffsetInDirection =
       calculateGlyphRange(queryData, fragment, 0, startPosition);
   FloatPoint glyphPosition = logicalGlyphPositionToPhysical(
       queryData, fragment, glyphOffsetInDirection);
   glyphPosition.move(0, -baseline);
 
   // Use the SVGTextMetrics computed by SVGTextMetricsBuilder.
   const MetricsList& metricsList = queryData->textLineLayout.metricsList();
   auto metrics = findMetricsForCharacter(metricsList, fragment, startPosition);
 
@@ skipping 76 matching lines @@
   return offset;
 }
 
 static bool characterNumberAtPositionCallback(QueryData* queryData,
                                               const SVGTextFragment& fragment) {
   CharacterNumberAtPositionData* data =
       static_cast<CharacterNumberAtPositionData*>(queryData);
 
   const float scalingFactor = data->textLineLayout.scalingFactor();
   ASSERT(scalingFactor);
+
+  const SimpleFontData* fontData =
+      data->textLineLayout.scaledFont().primaryFont();
+  DCHECK(fontData);
+  if (!fontData)
+    return false;
+
   const float baseline =
-      data->textLineLayout.scaledFont().getFontMetrics().floatAscent() /
-      scalingFactor;
+      fontData->getFontMetrics().floatAscent() / scalingFactor;
 
   // Test the query point against the bounds of the entire fragment first.
   if (!fragment.boundingBox(baseline).contains(data->position))
     return false;
 
   AffineTransform fragmentTransform = fragment.buildFragmentTransform(
       SVGTextFragment::TransformIgnoringTextLength);
 
   // Iterate through the glyphs in this fragment, and check if their extents
   // contain the query point.
@@ skipping 25 matching lines @@
 }
 
 int SVGTextQuery::characterNumberAtPosition(const FloatPoint& position) const {
   CharacterNumberAtPositionData data(position);
   spatialQuery(m_queryRootLayoutObject, &data,
                characterNumberAtPositionCallback);
   return data.characterNumberWithin(m_queryRootLayoutObject);
 }
 
 }  // namespace blink