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 .

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;
 }