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

Unified Diff: fpdfsdk/javascript/Annot.cpp

Issue 2453683011: Remove FX_BOOL from fpdfsdk. (Closed)
Patch Set: Regenerate patch after rebase. Created 4 years, 1 month 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
« no previous file with comments | « fpdfsdk/javascript/Annot.h ('k') | fpdfsdk/javascript/Document.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: fpdfsdk/javascript/Annot.cpp
diff --git a/fpdfsdk/javascript/Annot.cpp b/fpdfsdk/javascript/Annot.cpp
index 8e6a775dcd01cf67f98d1ae06a37368d13580208..0c16b3b05a3b828bb3fa0b8d16925f3c2555912a 100644
--- a/fpdfsdk/javascript/Annot.cpp
+++ b/fpdfsdk/javascript/Annot.cpp
@@ -37,17 +37,15 @@ Annot::Annot(CJS_Object* pJSObject) : CJS_EmbedObj(pJSObject) {}
Annot::~Annot() {}
-FX_BOOL Annot::hidden(IJS_Context* cc,
- CJS_PropValue& vp,
- CFX_WideString& sError) {
+bool Annot::hidden(IJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError) {
CPDFSDK_BAAnnot* baAnnot = ToBAAnnot(m_pAnnot.Get());
if (!baAnnot)
- return FALSE;
+ return false;
if (vp.IsGetting()) {
CPDF_Annot* pPDFAnnot = baAnnot->GetPDFAnnot();
vp << CPDF_Annot::IsAnnotationHidden(pPDFAnnot->GetAnnotDict());
- return TRUE;
+ return true;
}
bool bHidden;
@@ -66,41 +64,37 @@ FX_BOOL Annot::hidden(IJS_Context* cc,
flags |= ANNOTFLAG_PRINT;
}
baAnnot->SetFlags(flags);
- return TRUE;
+ return true;
}
-FX_BOOL Annot::name(IJS_Context* cc,
- CJS_PropValue& vp,
- CFX_WideString& sError) {
+bool Annot::name(IJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError) {
CPDFSDK_BAAnnot* baAnnot = ToBAAnnot(m_pAnnot.Get());
if (!baAnnot)
- return FALSE;
+ return false;
if (vp.IsGetting()) {
vp << baAnnot->GetAnnotName();
- return TRUE;
+ return true;
}
CFX_WideString annotName;
vp >> annotName;
baAnnot->SetAnnotName(annotName);
- return TRUE;
+ return true;
}
-FX_BOOL Annot::type(IJS_Context* cc,
- CJS_PropValue& vp,
- CFX_WideString& sError) {
+bool Annot::type(IJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError) {
if (vp.IsSetting()) {
sError = JSGetStringFromID(IDS_STRING_JSREADONLY);
- return FALSE;
+ return false;
}
CPDFSDK_BAAnnot* baAnnot = ToBAAnnot(m_pAnnot.Get());
if (!baAnnot)
- return FALSE;
+ return false;
vp << CPDF_Annot::AnnotSubtypeToString(baAnnot->GetAnnotSubtype());
- return TRUE;
+ return true;
}
void Annot::SetSDKAnnot(CPDFSDK_BAAnnot* annot) {
« no previous file with comments | « fpdfsdk/javascript/Annot.h ('k') | fpdfsdk/javascript/Document.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698