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

Unified Diff: xfa/src/fwl/src/basewidget/fwl_editimp.cpp

Issue 1422113003: Fix XFA compilation noise, part 1. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@xfa
Patch Set: Nits. Created 5 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 | « xfa/src/fwl/src/basewidget/fwl_datetimepickerimp.cpp ('k') | xfa/src/fwl/src/basewidget/fwl_listboximp.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/src/fwl/src/basewidget/fwl_editimp.cpp
diff --git a/xfa/src/fwl/src/basewidget/fwl_editimp.cpp b/xfa/src/fwl/src/basewidget/fwl_editimp.cpp
index 93061f35cbcdf87d1b54ccaab846cbac608b47d4..ca273070ab97560982768557b0bfe73693884ca6 100644
--- a/xfa/src/fwl/src/basewidget/fwl_editimp.cpp
+++ b/xfa/src/fwl/src/basewidget/fwl_editimp.cpp
@@ -169,20 +169,20 @@ CFWL_EditImp::CFWL_EditImp(IFWL_Widget* pOuter)
m_bLButtonDown(FALSE),
m_nSelStart(0),
m_nLimit(-1),
+ m_fSpaceAbove(0),
+ m_fSpaceBelow(0),
m_fFontSize(0),
+ m_bSetRange(FALSE),
+ m_iMin(-1),
+ m_iMax(0xFFFFFFF),
m_pVertScrollBar(NULL),
m_pHorzScrollBar(NULL),
m_pCaret(NULL),
- m_iMin(-1),
- m_iMax(0xFFFFFFF),
- m_bSetRange(FALSE),
m_pTextField(NULL),
m_backColor(0),
m_updateBackColor(FALSE),
m_iCurRecord(-1),
- m_iMaxRecord(128),
- m_fSpaceAbove(0),
- m_fSpaceBelow(0) {
+ m_iMaxRecord(128) {
m_rtClient.Reset();
m_rtEngine.Reset();
m_rtStatic.Reset();
@@ -197,20 +197,20 @@ CFWL_EditImp::CFWL_EditImp(const CFWL_WidgetImpProperties& properties,
m_bLButtonDown(FALSE),
m_nSelStart(0),
m_nLimit(-1),
+ m_fSpaceAbove(0),
+ m_fSpaceBelow(0),
m_fFontSize(0),
+ m_bSetRange(FALSE),
+ m_iMin(-1),
+ m_iMax(0xFFFFFFF),
m_pVertScrollBar(NULL),
m_pHorzScrollBar(NULL),
m_pCaret(NULL),
- m_iMin(-1),
- m_iMax(0xFFFFFFF),
- m_bSetRange(FALSE),
m_pTextField(NULL),
m_backColor(0),
m_updateBackColor(FALSE),
m_iCurRecord(-1),
- m_iMaxRecord(128),
- m_fSpaceAbove(0),
- m_fSpaceBelow(0) {
+ m_iMaxRecord(128) {
m_rtClient.Reset();
m_rtEngine.Reset();
m_rtStatic.Reset();
@@ -464,7 +464,6 @@ void CFWL_EditImp::DrawSpellCheck(CFX_Graphics* pGraphics,
}
FX_ARGB cr = 0xFFFF0000;
CFX_Color crLine(cr);
- FX_FLOAT fWidth = 1.0f;
CFWL_EvtEdtCheckWord checkWordEvent;
checkWordEvent.m_pSrcTarget = m_pInterface;
CFX_ByteString sLatinWord;
@@ -1019,7 +1018,6 @@ void CFWL_EditImp::DrawTextBk(CFX_Graphics* pGraphics,
}
CFX_RectF rtScorll;
m_pHorzScrollBar->GetWidgetRect(rtScorll);
- FX_FLOAT fStaticWidth = rtScorll.height;
CFX_RectF rtStatic;
rtStatic.Set(m_rtClient.right() - rtScorll.height,
m_rtClient.bottom() - rtScorll.height, rtScorll.height,
« no previous file with comments | « xfa/src/fwl/src/basewidget/fwl_datetimepickerimp.cpp ('k') | xfa/src/fwl/src/basewidget/fwl_listboximp.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698