diff options
| author | Tom Sepez <tsepez@chromium.org> | 2015-10-09 13:14:47 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2015-10-09 13:14:47 -0700 |
| commit | 848a13b6777cbff8cc4aea3ab3d69eaa0b82ae6c (patch) | |
| tree | d2c3a5ee4f67227ac33199f6d10caaf204c92896 /fpdfsdk/src/javascript/JS_Object.h | |
| parent | f1c713663192368d26031a4caed1f9705f4510af (diff) | |
| download | pdfium-848a13b6777cbff8cc4aea3ab3d69eaa0b82ae6c.tar.xz | |
Pass IJS_Runtime, not IJS_Context, to native object constructors.
This better separates the two IJS_ classes, with the IJS_Context
taking on its proper role of describing an event. There's no need
for the event details for object creation, so this gets much
cleaner.
Move some JS error reporting code from CJS_Context to CJS_Runtime.
Make InitInstance() and ExitInstance() voids, they always return
TRUE and we never check the result anyways.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1389163007 .
Diffstat (limited to 'fpdfsdk/src/javascript/JS_Object.h')
| -rw-r--r-- | fpdfsdk/src/javascript/JS_Object.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fpdfsdk/src/javascript/JS_Object.h b/fpdfsdk/src/javascript/JS_Object.h index 563a316bd6..50751523f5 100644 --- a/fpdfsdk/src/javascript/JS_Object.h +++ b/fpdfsdk/src/javascript/JS_Object.h @@ -49,8 +49,8 @@ class CJS_Object { virtual FX_BOOL IsType(const FX_CHAR* sClassName) { return TRUE; } virtual CFX_ByteString GetClassName() { return ""; } - virtual FX_BOOL InitInstance(IJS_Context* cc) { return TRUE; } - virtual FX_BOOL ExitInstance() { return TRUE; } + virtual void InitInstance(IJS_Runtime* pIRuntime) {} + virtual void ExitInstance() {} v8::Local<v8::Object> ToV8Object() { return m_pV8Object.Get(m_pIsolate); } |