From 31d7a189bcbc703602da3352248616625087ef40 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Tue, 8 Mar 2016 08:42:37 -0800 Subject: CXFA_TextParseContext::m_eDisplay is too small enum FDE_CSSDISPLAY contains about 20 values, which won't fit into 4 bits. Avoid packing and burn the memory, since I worry about the m_dwMatchedDelcs geting truncated, too. R=dsinclair@chromium.org Review URL: https://codereview.chromium.org/1771193002 . --- xfa/src/fxfa/src/app/xfa_textlayout.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xfa/src/fxfa/src/app/xfa_textlayout.h b/xfa/src/fxfa/src/app/xfa_textlayout.h index 03067ced5a..d096a645a7 100644 --- a/xfa/src/fxfa/src/app/xfa_textlayout.h +++ b/xfa/src/fxfa/src/app/xfa_textlayout.h @@ -79,8 +79,8 @@ class CXFA_TextParseContext : public CFX_Target { protected: IFDE_CSSDeclaration** m_ppMatchedDecls; - FX_DWORD m_dwMatchedDecls : 28; - FDE_CSSDISPLAY m_eDisplay : 4; + FX_DWORD m_dwMatchedDecls; + FDE_CSSDISPLAY m_eDisplay; }; class CXFA_TextParser { -- cgit v1.2.3