CPDF_ModuleMgr::m_SecurityHandlerMap only used by dead code.
authorTom Sepez <tsepez@chromium.org>
Fri, 14 Aug 2015 19:39:55 +0000 (12:39 -0700)
committerTom Sepez <tsepez@chromium.org>
Fri, 14 Aug 2015 19:39:55 +0000 (12:39 -0700)
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1289693008 .

core/include/fpdfapi/fpdf_module.h
core/src/fpdfapi/fpdf_basic_module.cpp

index bfa051e..2f88644 100644 (file)
@@ -50,17 +50,9 @@ class CPDF_ModuleMgr {
   CCodec_ModuleMgr* GetCodecModule() { return m_pCodecModule; }
 
   void InitPageModule();
-
   void InitRenderModule();
 
-  void SetDownloadCallback(FX_BOOL (*callback)(const FX_CHAR* module_name));
-
-  FX_BOOL DownloadModule(const FX_CHAR* module_name);
-
-  void NotifyModuleAvailable(const FX_CHAR* module_name);
-
   IPDF_RenderModule* GetRenderModule() const { return m_pRenderModule.get(); }
-
   IPDF_PageModule* GetPageModule() const { return m_pPageModule.get(); }
 
   void LoadEmbeddedGB1CMaps();
@@ -75,13 +67,6 @@ class CPDF_ModuleMgr {
   ICodec_IccModule* GetIccModule();
   ICodec_FlateModule* GetFlateModule();
 
-  void RegisterSecurityHandler(
-      const FX_CHAR* name,
-      CPDF_SecurityHandler* (*CreateHandler)(void* param),
-      void* param);
-
-  CPDF_SecurityHandler* CreateSecurityHandler(const FX_CHAR* name);
-
   void SetPrivateData(void* module_id,
                       void* pData,
                       PD_CALLBACK_FREEDATA callback);
@@ -93,14 +78,9 @@ class CPDF_ModuleMgr {
   ~CPDF_ModuleMgr();
 
   CCodec_ModuleMgr* m_pCodecModule;
-
   nonstd::unique_ptr<IPDF_RenderModule> m_pRenderModule;
   nonstd::unique_ptr<IPDF_PageModule> m_pPageModule;
-
   FX_BOOL (*m_pDownloadCallback)(const FX_CHAR* module_name);
-
-  CFX_MapByteStringToPtr m_SecurityHandlerMap;
-
   CFX_PrivateData m_privateData;
 };
 
index 595dadb..198f131 100644 (file)
@@ -36,34 +36,6 @@ CPDF_ModuleMgr::CPDF_ModuleMgr() : m_pCodecModule(nullptr) {}
 
 CPDF_ModuleMgr::~CPDF_ModuleMgr() {}
 
-void CPDF_ModuleMgr::SetDownloadCallback(
-    FX_BOOL (*callback)(const FX_CHAR* module_name)) {
-  m_pDownloadCallback = callback;
-}
-FX_BOOL CPDF_ModuleMgr::DownloadModule(const FX_CHAR* module_name) {
-  if (m_pDownloadCallback == NULL) {
-    return FALSE;
-  }
-  return m_pDownloadCallback(module_name);
-}
-void CPDF_ModuleMgr::NotifyModuleAvailable(const FX_CHAR* module_name) {
-  if (FXSYS_strcmp(module_name, kAddinNameCJK) == 0) {
-    m_pPageModule->NotifyCJKAvailable();
-  }
-}
-void CPDF_ModuleMgr::RegisterSecurityHandler(
-    const FX_CHAR* filter,
-    CPDF_SecurityHandler* (*CreateHandler)(void* param),
-    void* param) {
-  if (CreateHandler == NULL) {
-    m_SecurityHandlerMap.RemoveKey(filter);
-  } else {
-    m_SecurityHandlerMap.SetAt(filter, (void*)CreateHandler);
-  }
-  if (param) {
-    m_SecurityHandlerMap.SetAt(FX_BSTRC("_param_") + filter, param);
-  }
-}
 void CPDF_ModuleMgr::SetPrivateData(void* module_id,
                                     void* pData,
                                     PD_CALLBACK_FREEDATA callback) {
@@ -72,19 +44,6 @@ void CPDF_ModuleMgr::SetPrivateData(void* module_id,
 void* CPDF_ModuleMgr::GetPrivateData(void* module_id) {
   return m_privateData.GetPrivateData(module_id);
 }
-CPDF_SecurityHandler* CPDF_ModuleMgr::CreateSecurityHandler(
-    const FX_CHAR* filter) {
-  CPDF_SecurityHandler* (*CreateHandler)(void*) = NULL;
-  if (!m_SecurityHandlerMap.Lookup(filter, (void*&)CreateHandler)) {
-    return NULL;
-  }
-  if (CreateHandler == NULL) {
-    return NULL;
-  }
-  void* param = NULL;
-  m_SecurityHandlerMap.Lookup(FX_BSTRC("_param_") + filter, param);
-  return CreateHandler(param);
-}
 ICodec_FaxModule* CPDF_ModuleMgr::GetFaxModule() {
   return m_pCodecModule ? m_pCodecModule->GetFaxModule() : NULL;
 }