diff options
| author | Tom Sepez <tsepez@chromium.org> | 2017-02-16 17:01:00 -0800 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-02-21 15:18:27 +0000 |
| commit | b1670b5cca9a59dfb612ef9eb891a70dd716bf9c (patch) | |
| tree | e624b1ca93721894ff29a14bb52443bf4f952cf6 /fpdfsdk/javascript/cjs_runtime.h | |
| parent | 026f3d306d339585612d63a9f6ac5ed8839a3179 (diff) | |
| download | pdfium-b1670b5cca9a59dfb612ef9eb891a70dd716bf9c.tar.xz | |
Pass CJS_Runtime to JS callbacks.
This is much more convenient, since only a fraction of them
need an IJS_EventContext, which can be obtained from the
CJS_Runtime.
Make GetCurrentEventContext() specific to CJS_Runtime, and
return the concrete type. This saves a lot of casting.
Change-Id: If79a3bcbf44de513f3caace153099234cc313d47
Reviewed-on: https://pdfium-review.googlesource.com/2793
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk/javascript/cjs_runtime.h')
| -rw-r--r-- | fpdfsdk/javascript/cjs_runtime.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fpdfsdk/javascript/cjs_runtime.h b/fpdfsdk/javascript/cjs_runtime.h index 96b171b41e..039e24b63e 100644 --- a/fpdfsdk/javascript/cjs_runtime.h +++ b/fpdfsdk/javascript/cjs_runtime.h @@ -27,7 +27,6 @@ class CJS_Runtime : public IJS_Runtime, public: using FieldEvent = std::pair<CFX_WideString, JS_EVENT_T>; - static CJS_Runtime* FromEventContext(const IJS_EventContext* cc); static CJS_Runtime* CurrentRuntimeFromIsolate(v8::Isolate* pIsolate); explicit CJS_Runtime(CPDFSDK_FormFillEnvironment* pFormFillEnv); @@ -36,11 +35,12 @@ class CJS_Runtime : public IJS_Runtime, // IJS_Runtime IJS_EventContext* NewEventContext() override; void ReleaseEventContext(IJS_EventContext* pContext) override; - IJS_EventContext* GetCurrentEventContext() override; CPDFSDK_FormFillEnvironment* GetFormFillEnv() const override; int ExecuteScript(const CFX_WideString& script, CFX_WideString* info) override; + CJS_EventContext* GetCurrentEventContext() const; + // Returns true if the event isn't already found in the set. bool AddEventToSet(const FieldEvent& event); void RemoveEventFromSet(const FieldEvent& event); |