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

Side by Side Diff: xfa/fxfa/app/xfa_ffwidget.cpp

Issue 2263923003: Make CFX_Color constructor explicit (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: address comment Created 4 years, 3 months 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 unified diff | Download patch
« no previous file with comments | « xfa/fwl/theme/cfwl_widgettp.cpp ('k') | xfa/fxgraphics/cfx_color.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #include "xfa/fxfa/include/xfa_ffwidget.h" 7 #include "xfa/fxfa/include/xfa_ffwidget.h"
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <memory> 10 #include <memory>
(...skipping 1572 matching lines...) Expand 10 before | Expand all | Expand 10 after
1583 iHatch = FX_HATCHSTYLE_Horizontal; 1583 iHatch = FX_HATCHSTYLE_Horizontal;
1584 break; 1584 break;
1585 case XFA_ATTRIBUTEENUM_Vertical: 1585 case XFA_ATTRIBUTEENUM_Vertical:
1586 iHatch = FX_HATCHSTYLE_Vertical; 1586 iHatch = FX_HATCHSTYLE_Vertical;
1587 break; 1587 break;
1588 default: 1588 default:
1589 break; 1589 break;
1590 } 1590 }
1591 1591
1592 CFX_Pattern pattern(iHatch, crEnd, crStart); 1592 CFX_Pattern pattern(iHatch, crEnd, crStart);
1593 CFX_Color cr(&pattern); 1593 CFX_Color cr(&pattern, 0x0);
1594 pGS->SetFillColor(&cr); 1594 pGS->SetFillColor(&cr);
1595 pGS->FillPath(&fillPath, FXFILL_WINDING, pMatrix); 1595 pGS->FillPath(&fillPath, FXFILL_WINDING, pMatrix);
1596 } 1596 }
1597 static void XFA_BOX_Fill_Linear(CXFA_Box box, 1597 static void XFA_BOX_Fill_Linear(CXFA_Box box,
1598 CFX_Graphics* pGS, 1598 CFX_Graphics* pGS,
1599 CFX_Path& fillPath, 1599 CFX_Path& fillPath,
1600 CFX_RectF rtFill, 1600 CFX_RectF rtFill,
1601 CFX_Matrix* pMatrix) { 1601 CFX_Matrix* pMatrix) {
1602 CXFA_Fill fill = box.GetFill(); 1602 CXFA_Fill fill = box.GetFill();
1603 FX_ARGB crStart = fill.GetColor(); 1603 FX_ARGB crStart = fill.GetColor();
(...skipping 440 matching lines...) Expand 10 before | Expand all | Expand 10 after
2044 } 2044 }
2045 XFA_BOX_Fill(box, strokes, pGS, rtWidget, pMatrix, dwFlags); 2045 XFA_BOX_Fill(box, strokes, pGS, rtWidget, pMatrix, dwFlags);
2046 XFA_BOX_Stroke(box, strokes, pGS, rtWidget, pMatrix, dwFlags); 2046 XFA_BOX_Stroke(box, strokes, pGS, rtWidget, pMatrix, dwFlags);
2047 } 2047 }
2048 2048
2049 CXFA_CalcData::CXFA_CalcData() : m_iRefCount(0) {} 2049 CXFA_CalcData::CXFA_CalcData() : m_iRefCount(0) {}
2050 2050
2051 CXFA_CalcData::~CXFA_CalcData() { 2051 CXFA_CalcData::~CXFA_CalcData() {
2052 m_Globals.RemoveAll(); 2052 m_Globals.RemoveAll();
2053 } 2053 }
OLDNEW
« no previous file with comments | « xfa/fwl/theme/cfwl_widgettp.cpp ('k') | xfa/fxgraphics/cfx_color.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698