Chromium Code Reviews| Index: fpdfsdk/src/fsdk_annothandler.cpp |
| diff --git a/fpdfsdk/src/fsdk_annothandler.cpp b/fpdfsdk/src/fsdk_annothandler.cpp |
| index bc0f047897b8a5488d7ef58e87baf356199da3ab..15375c86ec92f80c6a09b8e2960e673b15e52e74 100644 |
| --- a/fpdfsdk/src/fsdk_annothandler.cpp |
| +++ b/fpdfsdk/src/fsdk_annothandler.cpp |
| @@ -92,11 +92,11 @@ void CPDFSDK_AnnotHandlerMgr::Annot_OnCreate(CPDFSDK_Annot* pAnnot) |
| CPDF_Annot* pPDFAnnot = pAnnot->GetPDFAnnot(); |
| ASSERT(pPDFAnnot != NULL); |
|
Tom Sepez
2015/06/05 21:15:30
nit: these two asserts are pointless as we'll segv
Lei Zhang
2015/06/05 22:25:47
Ok, removed a whole bunch of them.
|
| - ASSERT(pPDFAnnot->m_pAnnotDict != NULL); |
| + ASSERT(pPDFAnnot->GetAnnotDict() != NULL); |
| CPDFSDK_DateTime curTime; |
| - pPDFAnnot->m_pAnnotDict->SetAtString("M", curTime.ToPDFDateTimeString()); |
| - pPDFAnnot->m_pAnnotDict->SetAtNumber("F", (int)0); |
| + pPDFAnnot->GetAnnotDict()->SetAtString("M", curTime.ToPDFDateTimeString()); |
| + pPDFAnnot->GetAnnotDict()->SetAtNumber("F", 0); |
| if (IPDFSDK_AnnotHandler* pAnnotHandler = GetAnnotHandler(pAnnot)) |
| { |
| @@ -402,7 +402,7 @@ CPDFSDK_Annot* CPDFSDK_BFAnnotHandler::NewAnnot(CPDF_Annot* pAnnot, CPDFSDK_Pag |
| ASSERT(pInterForm != NULL); |
| CPDFSDK_Widget* pWidget = NULL; |
| - if (CPDF_FormControl* pCtrl = CPDFSDK_Widget::GetFormControl(pInterForm->GetInterForm(), pAnnot->m_pAnnotDict)) |
| + if (CPDF_FormControl* pCtrl = CPDFSDK_Widget::GetFormControl(pInterForm->GetInterForm(), pAnnot->GetAnnotDict())) |
| { |
| pWidget = new CPDFSDK_Widget(pAnnot, pPage, pInterForm); |
| pInterForm->AddMap(pCtrl, pWidget); |