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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: fpdfsdk/src/javascript/JS_Context.cpp

Issue 1318543012: CJS_Context::compile unused (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Remove pointless asserts. 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
« no previous file with comments | « fpdfsdk/include/jsapi/fxjs_v8.h ('k') | fpdfsdk/src/jsapi/fxjs_v8.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: fpdfsdk/src/javascript/JS_Context.cpp
diff --git a/fpdfsdk/src/javascript/JS_Context.cpp b/fpdfsdk/src/javascript/JS_Context.cpp
index 3fedd13b21d6b68373f97559a9d8dcd3a7601b1e..1e42b50138794efd1cc8a86f73a8e6edcbec30cd 100644
--- a/fpdfsdk/src/javascript/JS_Context.cpp
+++ b/fpdfsdk/src/javascript/JS_Context.cpp
@@ -4,12 +4,11 @@
// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
-#include "../../include/javascript/JavaScript.h"
#include "../../include/javascript/IJavaScript.h"
-//#include "../../include/javascript/JS_ResMgr.h"
#include "../../include/javascript/JS_Context.h"
#include "../../include/javascript/JS_EventHandler.h"
#include "../../include/javascript/JS_Runtime.h"
+#include "../../include/javascript/JavaScript.h"
#include "../../include/javascript/resource.h"
/* -------------------------- CJS_Context -------------------------- */
@@ -24,8 +23,6 @@ CJS_Context::~CJS_Context() {
}
CPDFSDK_Document* CJS_Context::GetReaderDocument() {
- ASSERT(m_pRuntime != NULL);
-
return m_pRuntime->GetReaderDocument();
}
@@ -35,20 +32,20 @@ CPDFDoc_Environment* CJS_Context::GetReaderApp() {
return m_pRuntime->GetReaderApp();
}
-FX_BOOL CJS_Context::DoJob(int nMode,
- const CFX_WideString& script,
- CFX_WideString& info) {
+FX_BOOL CJS_Context::RunScript(const CFX_WideString& script,
+ CFX_WideString& info) {
+ v8::Isolate::Scope isolate_scope(m_pRuntime->GetIsolate());
+ v8::HandleScope handle_scope(m_pRuntime->GetIsolate());
+ v8::Local<v8::Context> context = m_pRuntime->NewJSContext();
+ v8::Context::Scope context_scope(context);
+
if (m_bBusy) {
info = JSGetStringFromID(this, IDS_STRING_JSBUSY);
return FALSE;
}
-
m_bBusy = TRUE;
- ASSERT(m_pRuntime != NULL);
- ASSERT(m_pEventHandler != NULL);
ASSERT(m_pEventHandler->IsValid());
-
if (!m_pRuntime->AddEventToLoop(m_pEventHandler->TargetName(),
m_pEventHandler->EventType())) {
info = JSGetStringFromID(this, IDS_STRING_JSEVENT);
@@ -57,27 +54,15 @@ FX_BOOL CJS_Context::DoJob(int nMode,
FXJSErr error = {NULL, NULL, 0};
int nRet = 0;
-
if (script.GetLength() > 0) {
- if (nMode == 0) {
- nRet = JS_Execute(*m_pRuntime, this, script.c_str(), script.GetLength(),
- &error);
- } else {
- nRet = JS_Parse(*m_pRuntime, this, script.c_str(), script.GetLength(),
+ nRet = JS_Execute(*m_pRuntime, this, script.c_str(), script.GetLength(),
&error);
- }
}
if (nRet < 0) {
CFX_WideString sLine;
sLine.Format(L"[ Line: %05d { %s } ] : %s", error.linnum - 1, error.srcline,
error.message);
-
- // TRACE(L"/* -------------- JS Error --------------
- //*/\n");
- // TRACE(sLine);
- // TRACE(L"\n");
- // CFX_ByteString sTemp = CFX_ByteString::FromUnicode(error.message);
info += sLine;
} else {
info = JSGetStringFromID(this, IDS_STRING_RUN);
@@ -92,107 +77,72 @@ FX_BOOL CJS_Context::DoJob(int nMode,
return nRet >= 0;
}
-FX_BOOL CJS_Context::RunScript(const CFX_WideString& script,
- CFX_WideString& info) {
- v8::Isolate::Scope isolate_scope(m_pRuntime->GetIsolate());
- v8::HandleScope handle_scope(m_pRuntime->GetIsolate());
- v8::Local<v8::Context> context = m_pRuntime->NewJSContext();
- v8::Context::Scope context_scope(context);
-
- return DoJob(0, script, info);
-}
-
-FX_BOOL CJS_Context::Compile(const CFX_WideString& script,
- CFX_WideString& info) {
- v8::Isolate::Scope isolate_scope(m_pRuntime->GetIsolate());
- v8::HandleScope handle_scope(m_pRuntime->GetIsolate());
- v8::Local<v8::Context> context = m_pRuntime->NewJSContext();
- v8::Context::Scope context_scope(context);
-
- return DoJob(1, script, info);
-}
-
void CJS_Context::OnApp_Init() {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnApp_Init();
}
void CJS_Context::OnDoc_Open(CPDFSDK_Document* pDoc,
const CFX_WideString& strTargetName) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnDoc_Open(pDoc, strTargetName);
}
void CJS_Context::OnDoc_WillPrint(CPDFSDK_Document* pDoc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnDoc_WillPrint(pDoc);
}
void CJS_Context::OnDoc_DidPrint(CPDFSDK_Document* pDoc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnDoc_DidPrint(pDoc);
}
void CJS_Context::OnDoc_WillSave(CPDFSDK_Document* pDoc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnDoc_WillSave(pDoc);
}
void CJS_Context::OnDoc_DidSave(CPDFSDK_Document* pDoc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnDoc_DidSave(pDoc);
}
void CJS_Context::OnDoc_WillClose(CPDFSDK_Document* pDoc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnDoc_WillClose(pDoc);
}
void CJS_Context::OnPage_Open(CPDFSDK_Document* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnPage_Open(pTarget);
}
void CJS_Context::OnPage_Close(CPDFSDK_Document* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnPage_Close(pTarget);
}
void CJS_Context::OnPage_InView(CPDFSDK_Document* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnPage_InView(pTarget);
}
void CJS_Context::OnPage_OutView(CPDFSDK_Document* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnPage_OutView(pTarget);
}
void CJS_Context::OnField_MouseDown(FX_BOOL bModifier,
FX_BOOL bShift,
CPDF_FormField* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_MouseDown(bModifier, bShift, pTarget);
}
void CJS_Context::OnField_MouseEnter(FX_BOOL bModifier,
FX_BOOL bShift,
CPDF_FormField* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_MouseEnter(bModifier, bShift, pTarget);
}
void CJS_Context::OnField_MouseExit(FX_BOOL bModifier,
FX_BOOL bShift,
CPDF_FormField* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_MouseExit(bModifier, bShift, pTarget);
}
void CJS_Context::OnField_MouseUp(FX_BOOL bModifier,
FX_BOOL bShift,
CPDF_FormField* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_MouseUp(bModifier, bShift, pTarget);
}
@@ -200,7 +150,6 @@ void CJS_Context::OnField_Focus(FX_BOOL bModifier,
FX_BOOL bShift,
CPDF_FormField* pTarget,
const CFX_WideString& Value) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_Focus(bModifier, bShift, pTarget, Value);
}
@@ -208,7 +157,6 @@ void CJS_Context::OnField_Blur(FX_BOOL bModifier,
FX_BOOL bShift,
CPDF_FormField* pTarget,
const CFX_WideString& Value) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_Blur(bModifier, bShift, pTarget, Value);
}
@@ -216,7 +164,6 @@ void CJS_Context::OnField_Calculate(CPDF_FormField* pSource,
CPDF_FormField* pTarget,
CFX_WideString& Value,
FX_BOOL& bRc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_Calculate(pSource, pTarget, Value, bRc);
}
@@ -251,7 +198,6 @@ void CJS_Context::OnField_Validate(CFX_WideString& strChange,
CPDF_FormField* pTarget,
CFX_WideString& Value,
FX_BOOL& bRc) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnField_Validate(strChange, strChangeEx, bKeyDown, bModifier,
bShift, pTarget, Value, bRc);
}
@@ -259,100 +205,84 @@ void CJS_Context::OnField_Validate(CFX_WideString& strChange,
void CJS_Context::OnScreen_Focus(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_Focus(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_Blur(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_Blur(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_Open(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_Open(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_Close(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_Close(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_MouseDown(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_MouseDown(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_MouseUp(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_MouseUp(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_MouseEnter(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_MouseEnter(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_MouseExit(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_MouseExit(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_InView(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_InView(bModifier, bShift, pScreen);
}
void CJS_Context::OnScreen_OutView(FX_BOOL bModifier,
FX_BOOL bShift,
CPDFSDK_Annot* pScreen) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnScreen_OutView(bModifier, bShift, pScreen);
}
void CJS_Context::OnBookmark_MouseUp(CPDF_Bookmark* pBookMark) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnBookmark_MouseUp(pBookMark);
}
void CJS_Context::OnLink_MouseUp(CPDFSDK_Document* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnLink_MouseUp(pTarget);
}
void CJS_Context::OnConsole_Exec() {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnConsole_Exec();
}
void CJS_Context::OnExternal_Exec() {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnExternal_Exec();
}
void CJS_Context::OnBatchExec(CPDFSDK_Document* pTarget) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnBatchExec(pTarget);
}
void CJS_Context::OnMenu_Exec(CPDFSDK_Document* pTarget,
const CFX_WideString& strTargetName) {
- ASSERT(m_pEventHandler != NULL);
m_pEventHandler->OnMenu_Exec(pTarget, strTargetName);
}
« no previous file with comments | « fpdfsdk/include/jsapi/fxjs_v8.h ('k') | fpdfsdk/src/jsapi/fxjs_v8.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698