| Index: xfa/fde/css/fde_cssdatatable.cpp
|
| diff --git a/xfa/fde/css/fde_cssdatatable.cpp b/xfa/fde/css/fde_cssdatatable.cpp
|
| index 1af0059cb65942eb73a9a16d35196313648d8a7e..ac8c4669323ede9c6e6370390162779f57c2fd74 100644
|
| --- a/xfa/fde/css/fde_cssdatatable.cpp
|
| +++ b/xfa/fde/css/fde_cssdatatable.cpp
|
| @@ -37,13 +37,12 @@ CFX_FloatRect FDE_CSSBoundaryToRect(IFDE_CSSBoundaryStyle* pBoundStyle,
|
| FX_BOOL bPadding,
|
| FX_BOOL bBorder,
|
| FX_BOOL bMargin) {
|
| - ASSERT(pBoundStyle != NULL);
|
| FX_FLOAT fResult;
|
| const FDE_CSSRECT* pRect;
|
| CFX_FloatRect rect(0, 0, 0, 0);
|
| if (bPadding) {
|
| pRect = pBoundStyle->GetPaddingWidth();
|
| - if (pRect != NULL) {
|
| + if (pRect) {
|
| if (FDE_CSSLengthToFloat(pRect->left, fContainerWidth, fResult)) {
|
| rect.left += fResult;
|
| }
|
| @@ -60,7 +59,7 @@ CFX_FloatRect FDE_CSSBoundaryToRect(IFDE_CSSBoundaryStyle* pBoundStyle,
|
| }
|
| if (bBorder) {
|
| pRect = pBoundStyle->GetBorderWidth();
|
| - if (pRect != NULL) {
|
| + if (pRect) {
|
| if (FDE_CSSLengthToFloat(pRect->left, fContainerWidth, fResult)) {
|
| rect.left += fResult;
|
| }
|
| @@ -77,7 +76,7 @@ CFX_FloatRect FDE_CSSBoundaryToRect(IFDE_CSSBoundaryStyle* pBoundStyle,
|
| }
|
| if (bMargin) {
|
| pRect = pBoundStyle->GetMarginWidth();
|
| - if (pRect != NULL) {
|
| + if (pRect) {
|
| if (FDE_CSSLengthToFloat(pRect->left, fContainerWidth, fResult)) {
|
| rect.left += fResult;
|
| }
|
| @@ -95,7 +94,6 @@ CFX_FloatRect FDE_CSSBoundaryToRect(IFDE_CSSBoundaryStyle* pBoundStyle,
|
| return rect;
|
| }
|
| uint32_t FDE_CSSFontStyleToFDE(IFDE_CSSFontStyle* pFontStyle) {
|
| - ASSERT(pFontStyle != NULL);
|
| uint32_t dwFontStyle = FX_FONTSTYLE_Normal;
|
| if (pFontStyle->GetFontStyle() == FDE_CSSFONTSTYLE_Italic) {
|
| dwFontStyle |= FX_FONTSTYLE_Italic;
|
| @@ -562,7 +560,7 @@ static const FDE_CSSPERSUDOTABLE g_FDE_CSSPersudoType[] = {
|
| };
|
| FDE_LPCCSSPERSUDOTABLE FDE_GetCSSPersudoByEnum(FDE_CSSPERSUDO ePersudo) {
|
| return (ePersudo < FDE_CSSPERSUDO_NONE) ? (g_FDE_CSSPersudoType + ePersudo)
|
| - : NULL;
|
| + : nullptr;
|
| }
|
| const FDE_CSSPROPERTYTABLE* FDE_GetCSSPropertyByName(
|
| const CFX_WideStringC& wsName) {
|
| @@ -582,10 +580,11 @@ const FDE_CSSPROPERTYTABLE* FDE_GetCSSPropertyByName(
|
| iEnd = iMid - 1;
|
| }
|
| } while (iStart <= iEnd);
|
| - return NULL;
|
| + return nullptr;
|
| }
|
| const FDE_CSSPROPERTYTABLE* FDE_GetCSSPropertyByEnum(FDE_CSSPROPERTY eName) {
|
| - return (eName < FDE_CSSPROPERTY_MAX) ? (g_FDE_CSSProperties + eName) : NULL;
|
| + return (eName < FDE_CSSPROPERTY_MAX) ? (g_FDE_CSSProperties + eName)
|
| + : nullptr;
|
| }
|
| const FDE_CSSPROPERTYVALUETABLE* FDE_GetCSSPropertyValueByName(
|
| const CFX_WideStringC& wsName) {
|
| @@ -605,12 +604,12 @@ const FDE_CSSPROPERTYVALUETABLE* FDE_GetCSSPropertyValueByName(
|
| iEnd = iMid - 1;
|
| }
|
| } while (iStart <= iEnd);
|
| - return NULL;
|
| + return nullptr;
|
| }
|
| const FDE_CSSPROPERTYVALUETABLE* FDE_GetCSSPropertyValueByEnum(
|
| FDE_CSSPROPERTYVALUE eName) {
|
| return (eName < FDE_CSSPROPERTYVALUE_MAX) ? (g_FDE_CSSPropertyValues + eName)
|
| - : NULL;
|
| + : nullptr;
|
| }
|
| FDE_LPCCSSMEDIATYPETABLE FDE_GetCSSMediaTypeByName(
|
| const CFX_WideStringC& wsName) {
|
| @@ -631,7 +630,7 @@ FDE_LPCCSSMEDIATYPETABLE FDE_GetCSSMediaTypeByName(
|
| iEnd = iMid - 1;
|
| }
|
| } while (iStart <= iEnd);
|
| - return NULL;
|
| + return nullptr;
|
| }
|
| FDE_LPCCSSLENGTHUNITTABLE FDE_GetCSSLengthUnitByName(
|
| const CFX_WideStringC& wsName) {
|
| @@ -652,7 +651,7 @@ FDE_LPCCSSLENGTHUNITTABLE FDE_GetCSSLengthUnitByName(
|
| iEnd = iMid - 1;
|
| }
|
| } while (iStart <= iEnd);
|
| - return NULL;
|
| + return nullptr;
|
| }
|
| FDE_LPCCSSCOLORTABLE FDE_GetCSSColorByName(const CFX_WideStringC& wsName) {
|
| ASSERT(!wsName.IsEmpty());
|
| @@ -671,7 +670,7 @@ FDE_LPCCSSCOLORTABLE FDE_GetCSSColorByName(const CFX_WideStringC& wsName) {
|
| iEnd = iMid - 1;
|
| }
|
| } while (iStart <= iEnd);
|
| - return NULL;
|
| + return nullptr;
|
| }
|
|
|
| FX_BOOL FDE_ParseCSSNumber(const FX_WCHAR* pszValue,
|
| @@ -702,7 +701,7 @@ FX_BOOL FDE_ParseCSSString(const FX_WCHAR* pszValue,
|
| int32_t iValueLen,
|
| int32_t& iOffset,
|
| int32_t& iLength) {
|
| - ASSERT(pszValue != NULL && iValueLen > 0);
|
| + ASSERT(pszValue && iValueLen > 0);
|
| iOffset = 0;
|
| iLength = iValueLen;
|
| if (iValueLen >= 2) {
|
| @@ -718,7 +717,7 @@ FX_BOOL FDE_ParseCSSURI(const FX_WCHAR* pszValue,
|
| int32_t iValueLen,
|
| int32_t& iOffset,
|
| int32_t& iLength) {
|
| - ASSERT(pszValue != NULL && iValueLen > 0);
|
| + ASSERT(pszValue && iValueLen > 0);
|
| if (iValueLen < 6 || pszValue[iValueLen - 1] != ')' ||
|
| FX_wcsnicmp(L"url(", pszValue, 4)) {
|
| return FALSE;
|
|
|