Use PaintOperation instead of bFillOrStroke in CPVT_GenerateAP.

core/fpdfdoc/include/cpdf_defaultappearance.h

Index: core/fpdfdoc/include/cpdf_defaultappearance.h
diff --git a/core/fpdfdoc/include/cpdf_defaultappearance.h b/core/fpdfdoc/include/cpdf_defaultappearance.h
index c45d4eeffc3b314152994fa85f4b44958b39af86..3da0aa487d6692d51e52f387bf25c30348f8ab81 100644
--- a/core/fpdfdoc/include/cpdf_defaultappearance.h
+++ b/core/fpdfdoc/include/cpdf_defaultappearance.h
@@ -14,6 +14,7 @@
 #include "core/fxge/include/fx_dib.h"
 
 enum class BorderStyle { SOLID, DASH, BEVELED, INSET, UNDERLINE };
+enum class PaintOperation { kStroke, kFill };

[Lei Zhang, 2016/08/02 18:52:51]

Sorry to give you slightly bad advice. Google style guide prefers kFoo, but
Chromium style guide prefers FOO. Dan also voted for the latter, and BorderStyle
above is also SHOUTY_CASE. So can we do one more round of search+replace and git
cl format?

[jaepark, 2016/08/02 19:23:26]

Done.

 
 class CPDF_DefaultAppearance {
  public:
@@ -30,14 +31,15 @@ class CPDF_DefaultAppearance {
   CFX_ByteString GetFontString();
   void GetFont(CFX_ByteString& csFontNameTag, FX_FLOAT& fFontSize);
 
-  FX_BOOL HasColor(FX_BOOL bStrokingOperation = FALSE);
-  CFX_ByteString GetColorString(FX_BOOL bStrokingOperation = FALSE);
+  FX_BOOL HasColor(PaintOperation nOperation = PaintOperation::kFill);
+  CFX_ByteString GetColorString(
+      PaintOperation nOperation = PaintOperation::kFill);
   void GetColor(int& iColorType,
                 FX_FLOAT fc[4],
-                FX_BOOL bStrokingOperation = FALSE);
+                PaintOperation nOperation = PaintOperation::kFill);
   void GetColor(FX_ARGB& color,
                 int& iColorType,
-                FX_BOOL bStrokingOperation = FALSE);
+                PaintOperation nOperation = PaintOperation::kFill);
 
   FX_BOOL HasTextMatrix();
   CFX_ByteString GetTextMatrixString();