Remove IPDFSDK_AnnotHandler interface.

Remove IPDFSDK_AnnotHandler interface.

It's pointless to have an abstract interface and its only concrete
implementation defined in the same header.  We can de-virtualize along
the way and get a small savings.

Rename CPDFSDK_BFAnnotHandler along the way, since its the only kind
of AnnotHandler, and I can't imagine what BF stands for.

R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/633f1e436e1cced874a44a797f02c452eb3cf958

[Tom Sepez, 2015-09-01 21:26:55]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2015-09-01 21:26:55]

Lei, more cleanup.

[Lei Zhang, 2015-09-01 21:33:30]

lgtm

https://codereview.chromium.org/1313733013/diff/40001/fpdfsdk/include/fsdk_an...
File fpdfsdk/include/fsdk_annothandler.h (right):

https://codereview.chromium.org/1313733013/diff/40001/fpdfsdk/include/fsdk_an...
fpdfsdk/include/fsdk_annothandler.h:26:
CPDFSDK_AnnotHandler(CPDFDoc_Environment* pApp)
explicit

https://codereview.chromium.org/1313733013/diff/40001/fpdfsdk/include/fsdk_an...
fpdfsdk/include/fsdk_annothandler.h:27: : m_pApp(pApp), m_pFormFiller(NULL) {}
nullptr

[Tom Sepez, 2015-09-01 22:02:02]

Committed patchset #4 (id:60001) manually as
633f1e436e1cced874a44a797f02c452eb3cf958 (presubmit successful).

[Tom Sepez, 2015-09-01 22:05:21]

On 2015/09/01 22:02:02, Tom Sepez wrote:
> Committed patchset #4 (id:60001) manually as
> 633f1e436e1cced874a44a797f02c452eb3cf958 (presubmit successful).

(Merge not possible, XFA implements two kinds of annothandler).

[Tom Sepez, 2015-09-01 22:05:30]

https://codereview.chromium.org/1313733013/diff/40001/fpdfsdk/include/fsdk_an...
File fpdfsdk/include/fsdk_annothandler.h (right):

https://codereview.chromium.org/1313733013/diff/40001/fpdfsdk/include/fsdk_an...
fpdfsdk/include/fsdk_annothandler.h:26:
CPDFSDK_AnnotHandler(CPDFDoc_Environment* pApp)
On 2015/09/01 21:33:30, Lei Zhang wrote:
> explicit

Done.

https://codereview.chromium.org/1313733013/diff/40001/fpdfsdk/include/fsdk_an...
fpdfsdk/include/fsdk_annothandler.h:27: : m_pApp(pApp), m_pFormFiller(NULL) {}
On 2015/09/01 21:33:30, Lei Zhang wrote:
> nullptr

Done.