diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-08-07 23:24:09 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-07 23:24:09 +0000 |
| commit | 69e19e2c0b6753a69470840226456858cdef5773 (patch) | |
| tree | d7e2890b20c6a9fe683bcd5903b8cd957d1ef2f2 /core/fpdfapi | |
| parent | e4d2cffd31a1f705f04d11223444455aa35923a6 (diff) | |
| download | pdfium-69e19e2c0b6753a69470840226456858cdef5773.tar.xz | |
Make CPDF_Font::GetFontBBox() return the bounding box.
Instead of using an out parameter. Fix nits in the only caller as well.
Change-Id: I04eb92705b2fbda001191c36da972eee499acbc7
Reviewed-on: https://pdfium-review.googlesource.com/39490
Reviewed-by: Nicolás Peña Moreno <npm@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdfapi')
| -rw-r--r-- | core/fpdfapi/font/cpdf_font.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/fpdfapi/font/cpdf_font.h b/core/fpdfapi/font/cpdf_font.h index d79b13fb6b..f246293c3e 100644 --- a/core/fpdfapi/font/cpdf_font.h +++ b/core/fpdfapi/font/cpdf_font.h @@ -72,7 +72,7 @@ class CPDF_Font { FXFT_Face GetFace() const { return m_Font.GetFace(); } void AppendChar(ByteString* str, uint32_t charcode) const; - void GetFontBBox(FX_RECT& rect) const { rect = m_FontBBox; } + const FX_RECT& GetFontBBox() const { return m_FontBBox; } int GetTypeAscent() const { return m_Ascent; } int GetTypeDescent() const { return m_Descent; } uint32_t GetStringWidth(const ByteStringView& pString); |