diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-07-25 23:25:55 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-07-25 23:25:55 +0000 |
| commit | 11a6becb837a16972ffda8f94c8fb69ae100f3f4 (patch) | |
| tree | 38f7e3508f1ef221eabd62b14665e78437da2d8f /fpdfsdk/cpdfsdk_widget.h | |
| parent | 91b8302dec04ca4ddc1f91545d192350665580cf (diff) | |
| download | pdfium-11a6becb837a16972ffda8f94c8fb69ae100f3f4.tar.xz | |
Remove some ASSERT (and cast) in favor of checked cases.
Because it is a stronger pattern at runtime.
These were found by essentially: grep -ni '\bassert\b.*type'
Change-Id: I913d77139053e8980528597a6633e1859e5204c4
Reviewed-on: https://pdfium-review.googlesource.com/38890
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_widget.h')
| -rw-r--r-- | fpdfsdk/cpdfsdk_widget.h | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fpdfsdk/cpdfsdk_widget.h b/fpdfsdk/cpdfsdk_widget.h index 97971c8342..1349981293 100644 --- a/fpdfsdk/cpdfsdk_widget.h +++ b/fpdfsdk/cpdfsdk_widget.h @@ -144,4 +144,10 @@ class CPDFSDK_Widget : public CPDFSDK_BAAnnot { #endif // PDF_ENABLE_XFA }; +inline CPDFSDK_Widget* ToCPDFSDKWidget(CPDFSDK_Annot* pAnnot) { + return pAnnot && pAnnot->GetAnnotSubtype() == CPDF_Annot::Subtype::WIDGET + ? static_cast<CPDFSDK_Widget*>(pAnnot) + : nullptr; +} + #endif // FPDFSDK_CPDFSDK_WIDGET_H_ |