From 0f6b51c0fdd14f5762bf3c7412ac59c825443cc3 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Fri, 14 Aug 2015 14:23:57 -0700 Subject: Use override in more classes in fpdfsdk/ R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1287193005 . --- fpdfsdk/include/javascript/IJavaScript.h | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'fpdfsdk/include/javascript/IJavaScript.h') diff --git a/fpdfsdk/include/javascript/IJavaScript.h b/fpdfsdk/include/javascript/IJavaScript.h index 53a4225aed..56922403e1 100644 --- a/fpdfsdk/include/javascript/IJavaScript.h +++ b/fpdfsdk/include/javascript/IJavaScript.h @@ -17,7 +17,6 @@ class CPDFSDK_Document; class IFXJS_Context { public: - virtual ~IFXJS_Context() {} virtual FX_BOOL Compile(const CFX_WideString& script, CFX_WideString& info) = 0; virtual FX_BOOL RunScript(const CFX_WideString& script, @@ -128,6 +127,9 @@ class IFXJS_Context { virtual void OnExternal_Exec() = 0; virtual void EnableMessageBox(FX_BOOL bEnable) = 0; + + protected: + virtual ~IFXJS_Context() {} }; class IFXJS_Runtime { @@ -140,7 +142,7 @@ class IFXJS_Runtime { virtual CPDFSDK_Document* GetReaderDocument() = 0; protected: - ~IFXJS_Runtime() {} + virtual ~IFXJS_Runtime() {} }; class CPDFDoc_Environment; -- cgit v1.2.3