[Impeller] Remove unused bounds method from typographer interface

Author: bdero

impeller/typographer/backends/skia/typeface_skia.cc

@@ -28,17 +28,6 @@ bool TypefaceSkia::IsEqual(const Typeface& other) const {
   return sk_other->typeface_ == typeface_;
 }
 
-Rect TypefaceSkia::GetBoundingBox() const {
-  if (!IsValid()) {
-    return {};
-  }
-
-  const auto bounds = typeface_->getBounds();
-
-  return Rect::MakeLTRB(bounds.left(), bounds.top(), bounds.right(),
-                        bounds.bottom());
-}
-
 const sk_sp<SkTypeface>& TypefaceSkia::GetSkiaTypeface() const {
   return typeface_;
 }

impeller/typographer/backends/skia/typeface_skia.h

@@ -28,9 +28,6 @@ class TypefaceSkia final : public Typeface,
   // |Comparable<Typeface>|
   bool IsEqual(const Typeface& other) const override;
 
-  // |Typeface|
-  Rect GetBoundingBox() const override;
-
   const sk_sp<SkTypeface>& GetSkiaTypeface() const;
 
  private:

impeller/typographer/typeface.h

@@ -24,15 +24,6 @@ class Typeface : public Comparable<Typeface> {
 
   virtual bool IsValid() const = 0;
 
-  //----------------------------------------------------------------------------
-  /// @brief      Get the union of the bounding boxes of all glyphs in the
-  ///             typeface. This box is unit-scaled and conservatively large to
-  ///             cover all glyphs.
-  ///
-  /// @return     The conservative unit-scaled bounding box.
-  ///
-  virtual Rect GetBoundingBox() const = 0;
-
  private:
   FML_DISALLOW_COPY_AND_ASSIGN(Typeface);
 };