From d983b09c3ae29a97cba8e9ec9c6351545f6087ee Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 14 Dec 2015 16:58:33 -0800 Subject: Merge to XFA: Remove FX_BSTRC. R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1521563002 . (cherry picked from commit 1956a174020686f91cd3b34294e91f4560fe45aa) Review URL: https://codereview.chromium.org/1526823002 . --- core/include/fpdfapi/fpdf_parser.h | 9 ++++----- core/include/fpdfdoc/fpdf_doc.h | 26 ++++++++++++-------------- core/include/fxcrt/fx_string.h | 1 - 3 files changed, 16 insertions(+), 20 deletions(-) (limited to 'core/include') diff --git a/core/include/fpdfapi/fpdf_parser.h b/core/include/fpdfapi/fpdf_parser.h index ad85de4c9a..a266b20ba2 100644 --- a/core/include/fpdfapi/fpdf_parser.h +++ b/core/include/fpdfapi/fpdf_parser.h @@ -313,11 +313,10 @@ class CPDF_SyntaxParser { void GetNextWord(); - FX_BOOL IsWholeWord(FX_FILESIZE startpos, - FX_FILESIZE limit, - const uint8_t* tag, - FX_DWORD taglen, - FX_BOOL checkKeyword); + bool IsWholeWord(FX_FILESIZE startpos, + FX_FILESIZE limit, + const CFX_ByteStringC& tag, + FX_BOOL checkKeyword); CFX_ByteString ReadString(); diff --git a/core/include/fpdfdoc/fpdf_doc.h b/core/include/fpdfdoc/fpdf_doc.h index 07f735e21b..52be0dba0f 100644 --- a/core/include/fpdfdoc/fpdf_doc.h +++ b/core/include/fpdfdoc/fpdf_doc.h @@ -1099,42 +1099,40 @@ class CPDF_ApSettings { int GetRotation() const; FX_ARGB GetBorderColor(int& iColorType) const { - return GetColor(iColorType, FX_BSTRC("BC")); + return GetColor(iColorType, "BC"); } FX_FLOAT GetOriginalBorderColor(int index) const { - return GetOriginalColor(index, FX_BSTRC("BC")); + return GetOriginalColor(index, "BC"); } void GetOriginalBorderColor(int& iColorType, FX_FLOAT fc[4]) const { - GetOriginalColor(iColorType, fc, FX_BSTRC("BC")); + GetOriginalColor(iColorType, fc, "BC"); } FX_ARGB GetBackgroundColor(int& iColorType) const { - return GetColor(iColorType, FX_BSTRC("BG")); + return GetColor(iColorType, "BG"); } FX_FLOAT GetOriginalBackgroundColor(int index) const { - return GetOriginalColor(index, FX_BSTRC("BG")); + return GetOriginalColor(index, "BG"); } void GetOriginalBackgroundColor(int& iColorType, FX_FLOAT fc[4]) const { - GetOriginalColor(iColorType, fc, FX_BSTRC("BG")); + GetOriginalColor(iColorType, fc, "BG"); } - CFX_WideString GetNormalCaption() const { return GetCaption(FX_BSTRC("CA")); } + CFX_WideString GetNormalCaption() const { return GetCaption("CA"); } - CFX_WideString GetRolloverCaption() const { - return GetCaption(FX_BSTRC("RC")); - } + CFX_WideString GetRolloverCaption() const { return GetCaption("RC"); } - CFX_WideString GetDownCaption() const { return GetCaption(FX_BSTRC("AC")); } + CFX_WideString GetDownCaption() const { return GetCaption("AC"); } - CPDF_Stream* GetNormalIcon() const { return GetIcon(FX_BSTRC("I")); } + CPDF_Stream* GetNormalIcon() const { return GetIcon("I"); } - CPDF_Stream* GetRolloverIcon() const { return GetIcon(FX_BSTRC("RI")); } + CPDF_Stream* GetRolloverIcon() const { return GetIcon("RI"); } - CPDF_Stream* GetDownIcon() const { return GetIcon(FX_BSTRC("IX")); } + CPDF_Stream* GetDownIcon() const { return GetIcon("IX"); } CPDF_IconFit GetIconFit() const; diff --git a/core/include/fxcrt/fx_string.h b/core/include/fxcrt/fx_string.h index f493c53ed9..1c1e237076 100644 --- a/core/include/fxcrt/fx_string.h +++ b/core/include/fxcrt/fx_string.h @@ -135,7 +135,6 @@ inline bool operator==(const char* lhs, const CFX_ByteStringC& rhs) { inline bool operator!=(const char* lhs, const CFX_ByteStringC& rhs) { return rhs != lhs; } -#define FX_BSTRC(str) CFX_ByteStringC(str, sizeof str - 1) #define FXBSTR_ID(c1, c2, c3, c4) ((c1 << 24) | (c2 << 16) | (c3 << 8) | (c4)) // A mutable string with shared buffers using copy-on-write semantics that -- cgit v1.2.3