Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5826)

Unified Diff: core/include/fpdfdoc/fpdf_ap.h

Issue 1296043002: Merge to XFA: Use override in more classes in core/ (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@xfa
Patch Set: Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: core/include/fpdfdoc/fpdf_ap.h
diff --git a/core/include/fpdfdoc/fpdf_ap.h b/core/include/fpdfdoc/fpdf_ap.h
index 43452fdb47b8f655c2effdfe1fab0aadf41ac371..305834e96a31d5d3474f31e88d6e70faf902c300 100644
--- a/core/include/fpdfdoc/fpdf_ap.h
+++ b/core/include/fpdfdoc/fpdf_ap.h
@@ -51,20 +51,19 @@ struct CPVT_Color {
class CPVT_Provider : public IPDF_VariableText_Provider {
public:
CPVT_Provider(IPVT_FontMap* pFontMap);
-
- virtual ~CPVT_Provider();
-
- int32_t GetCharWidth(int32_t nFontIndex, FX_WORD word, int32_t nWordStyle);
-
- int32_t GetTypeAscent(int32_t nFontIndex);
-
- int32_t GetTypeDescent(int32_t nFontIndex);
-
- int32_t GetWordFontIndex(FX_WORD word, int32_t charset, int32_t nFontIndex);
-
- FX_BOOL IsLatinWord(FX_WORD word);
-
- int32_t GetDefaultFontIndex();
+ ~CPVT_Provider() override;
+
+ // IPDF_VariableText_Provider
+ int32_t GetCharWidth(int32_t nFontIndex,
+ FX_WORD word,
+ int32_t nWordStyle) override;
+ int32_t GetTypeAscent(int32_t nFontIndex) override;
+ int32_t GetTypeDescent(int32_t nFontIndex) override;
+ int32_t GetWordFontIndex(FX_WORD word,
+ int32_t charset,
+ int32_t nFontIndex) override;
+ FX_BOOL IsLatinWord(FX_WORD word) override;
+ int32_t GetDefaultFontIndex() override;
private:
IPVT_FontMap* m_pFontMap;

Powered by Google App Engine
This is Rietveld 408576698