diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-09-01 13:53:19 -0700 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-09-01 13:53:19 -0700 |
commit | 23213b2750e33dfe96b091b78ec8b10fe4d73a67 (patch) | |
tree | 00b51c5973a0e6b214532834f839d5e61c90768e /fpdfsdk/src | |
parent | 6e945a1ebbacde3ed2a4fc83ccb7215ccbc46931 (diff) | |
download | pdfium-23213b2750e33dfe96b091b78ec8b10fe4d73a67.tar.xz |
CPDFSDK_MediaActionHandler is unused.
It's never instantiated, and any pointers to it are always NULL.
Also kill an unused arg to CPDFSDK_ActionHandler ctor along the way.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1329583004 .
Diffstat (limited to 'fpdfsdk/src')
-rw-r--r-- | fpdfsdk/src/fsdk_actionhandler.cpp | 42 | ||||
-rw-r--r-- | fpdfsdk/src/fsdk_mgr.cpp | 2 |
2 files changed, 3 insertions, 41 deletions
diff --git a/fpdfsdk/src/fsdk_actionhandler.cpp b/fpdfsdk/src/fsdk_actionhandler.cpp index a16c21a321..a2a9eff817 100644 --- a/fpdfsdk/src/fsdk_actionhandler.cpp +++ b/fpdfsdk/src/fsdk_actionhandler.cpp @@ -9,27 +9,16 @@ #include "../include/fsdk_actionhandler.h" #include "../include/javascript/IJavaScript.h" -/* -------------------------- CBA_ActionHandler -------------------------- */ - -CPDFSDK_ActionHandler::CPDFSDK_ActionHandler(CPDFDoc_Environment* pEvi) - : m_pFormActionHandler(new CPDFSDK_FormActionHandler), - m_pMediaActionHandler(NULL) {} - -void CPDFSDK_ActionHandler::SetMediaActionHandler( - CPDFSDK_MediaActionHandler* pHandler) { - ASSERT(pHandler != NULL); - ASSERT(m_pMediaActionHandler == NULL); - m_pMediaActionHandler = pHandler; +CPDFSDK_ActionHandler::CPDFSDK_ActionHandler() + : m_pFormActionHandler(new CPDFSDK_FormActionHandler) { } -// document open FX_BOOL CPDFSDK_ActionHandler::DoAction_DocOpen(const CPDF_Action& action, CPDFSDK_Document* pDocument) { CFX_PtrList list; return ExecuteDocumentOpenAction(action, pDocument, list); } -// document open FX_BOOL CPDFSDK_ActionHandler::DoAction_JavaScript( const CPDF_Action& JsAction, CFX_WideString csJSName, @@ -453,14 +442,8 @@ void CPDFSDK_ActionHandler::DoAction_NoJs(const CPDF_Action& action, DoAction_URI(pDocument, action); break; case CPDF_Action::Sound: - if (m_pMediaActionHandler) { - m_pMediaActionHandler->DoAction_Sound(action, pDocument); - } break; case CPDF_Action::Movie: - if (m_pMediaActionHandler) { - m_pMediaActionHandler->DoAction_Movie(action, pDocument); - } break; case CPDF_Action::Hide: if (m_pFormActionHandler) { @@ -492,9 +475,6 @@ void CPDFSDK_ActionHandler::DoAction_NoJs(const CPDF_Action& action, DoAction_SetOCGState(pDocument, action); break; case CPDF_Action::Rendition: - if (m_pMediaActionHandler) { - m_pMediaActionHandler->DoAction_Rendition(action, pDocument); - } break; case CPDF_Action::Trans: break; @@ -766,21 +746,3 @@ FX_BOOL CPDFSDK_FormActionHandler::DoAction_ImportData( return FALSE; } - -FX_BOOL CPDFSDK_MediaActionHandler::DoAction_Rendition( - const CPDF_Action& action, - CPDFSDK_Document* pDocument) { - return FALSE; -} - -FX_BOOL CPDFSDK_MediaActionHandler::DoAction_Sound( - const CPDF_Action& action, - CPDFSDK_Document* pDocument) { - return FALSE; -} - -FX_BOOL CPDFSDK_MediaActionHandler::DoAction_Movie( - const CPDF_Action& action, - CPDFSDK_Document* pDocument) { - return FALSE; -} diff --git a/fpdfsdk/src/fsdk_mgr.cpp b/fpdfsdk/src/fsdk_mgr.cpp index fa270cc289..3eca559285 100644 --- a/fpdfsdk/src/fsdk_mgr.cpp +++ b/fpdfsdk/src/fsdk_mgr.cpp @@ -396,7 +396,7 @@ CPDFSDK_AnnotHandlerMgr* CPDFDoc_Environment::GetAnnotHandlerMgr() { CPDFSDK_ActionHandler* CPDFDoc_Environment::GetActionHander() { if (!m_pActionHandler) - m_pActionHandler = new CPDFSDK_ActionHandler(this); + m_pActionHandler = new CPDFSDK_ActionHandler(); return m_pActionHandler; } |