Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(650)

Unified Diff: public/fpdf_formfill.h

Issue 1758553003: Trigger page view event when re-layout is finished (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: public/fpdf_formfill.h
diff --git a/public/fpdf_formfill.h b/public/fpdf_formfill.h
index 31117538e2416a6b7e4eb7c5019680fa9341a662..6b181a2a7cd63e4f9c9eba52bfa5f744060ee0e8 100644
--- a/public/fpdf_formfill.h
+++ b/public/fpdf_formfill.h
@@ -820,20 +820,29 @@ typedef struct _FPDF_FORMFILLINFO {
/**
* Method: FFI_PageEvent
- * This method fires when pages have been added or deleted.
+ * This method fires when pages have been added to or deleted from the XFA
+ * document.
* Interface Version:
* 2
* Implementation Required:
* yes
* Parameters:
* pThis - Pointer to the interface structure itself.
- * page_index - 0-based page number.
+ * page_count - The number of pages to be added to or deleted from the
+ *document.
Tom Sepez 2016/03/17 16:16:43 nit: indentation.
Jim Wang 2016/03/18 01:12:25 Done.
* event_type - See FXFA_PAGEVIEWEVENT_* above.
* Return value:
* None.
+ * Comments:
+ * The pages to be added or deleted always start from the last page
+ *of document. This means that if parameter page_count is 2 and
Tom Sepez 2016/03/17 16:16:43 nit: indentation (also line 840 and 842).
Jim Wang 2016/03/18 01:12:25 Done.
+ * event type is FXFA_PAGEVIEWEVENT_POSTADDED, 2 new pages have been
+ *appended to the tail of document; If page_count is 2 and
+ * event type is FXFA_PAGEVIEWEVENT_POSTREMOVED, the last 2 pages
+ *have been deleted.
**/
void (*FFI_PageEvent)(struct _FPDF_FORMFILLINFO* pThis,
- int page_index,
+ int page_count,
FPDF_DWORD event_type);
/**

Powered by Google App Engine
This is Rietveld 408576698