summaryrefslogtreecommitdiff
path: root/fpdfsdk/include/javascript
diff options
context:
space:
mode:
authorTom Sepez <tsepez@chromium.org>2015-08-31 16:54:42 -0700
committerTom Sepez <tsepez@chromium.org>2015-08-31 16:54:42 -0700
commit6e945a1ebbacde3ed2a4fc83ccb7215ccbc46931 (patch)
treef2cf6a48b3620104e54296d75b77722c1411c8e2 /fpdfsdk/include/javascript
parent6e369c2ebb4769a42736b32154c0de12b73df45d (diff)
downloadpdfium-6e945a1ebbacde3ed2a4fc83ccb7215ccbc46931.tar.xz
CJS_Context::compile unused
This, in turn, allows us to get rid of the DoJob method and its clumsy mode parameter conditional. Add explicit and consts, and remove pointless asserts along the way. R=thestig@chromium.org Review URL: https://codereview.chromium.org/1318543012 .
Diffstat (limited to 'fpdfsdk/include/javascript')
-rw-r--r--fpdfsdk/include/javascript/IJavaScript.h2
-rw-r--r--fpdfsdk/include/javascript/JS_Context.h10
2 files changed, 3 insertions, 9 deletions
diff --git a/fpdfsdk/include/javascript/IJavaScript.h b/fpdfsdk/include/javascript/IJavaScript.h
index 56922403e1..18ab34b8cb 100644
--- a/fpdfsdk/include/javascript/IJavaScript.h
+++ b/fpdfsdk/include/javascript/IJavaScript.h
@@ -17,8 +17,6 @@ class CPDFSDK_Document;
class IFXJS_Context {
public:
- virtual FX_BOOL Compile(const CFX_WideString& script,
- CFX_WideString& info) = 0;
virtual FX_BOOL RunScript(const CFX_WideString& script,
CFX_WideString& info) = 0;
diff --git a/fpdfsdk/include/javascript/JS_Context.h b/fpdfsdk/include/javascript/JS_Context.h
index 995d32219e..d5b1064cfc 100644
--- a/fpdfsdk/include/javascript/JS_Context.h
+++ b/fpdfsdk/include/javascript/JS_Context.h
@@ -16,11 +16,10 @@ class CJS_Runtime;
class CJS_Context : public IFXJS_Context {
public:
- CJS_Context(CJS_Runtime* pRuntime);
+ explicit CJS_Context(CJS_Runtime* pRuntime);
~CJS_Context() override;
// IFXJS_Context
- FX_BOOL Compile(const CFX_WideString& script, CFX_WideString& info) override;
FX_BOOL RunScript(const CFX_WideString& script,
CFX_WideString& info) override;
void OnApp_Init() override;
@@ -124,11 +123,8 @@ class CJS_Context : public IFXJS_Context {
FX_BOOL IsMsgBoxEnabled() const { return m_bMsgBoxEnable; }
CPDFDoc_Environment* GetReaderApp();
- CJS_Runtime* GetJSRuntime() { return m_pRuntime; }
-
- FX_BOOL DoJob(int nMode, const CFX_WideString& script, CFX_WideString& info);
-
- CJS_EventHandler* GetEventHandler() { return m_pEventHandler; }
+ CJS_Runtime* GetJSRuntime() const { return m_pRuntime; }
+ CJS_EventHandler* GetEventHandler() const { return m_pEventHandler; }
CPDFSDK_Document* GetReaderDocument();
private: