pdfium.git - Unnamed repository; edit this file 'description' to name the repository.

Age	Commit message (Collapse)	Author
2016-05-20	Clean up XFA code which causes warnings	weili
	This is part of efforts to bring XFA to chromium_code standard. The warnings are from unreachable code, or using potentially uninitialized variables, or using assignment within a condition. This change list only contains easy to fix cases. More cleanups will follow. BUG=pdfium:29 Review-Url: https://codereview.chromium.org/1998873002
2016-05-18	Remove Release() from CXFA_ChecksumContext	tsepez
	Review-Url: https://codereview.chromium.org/1991913002
2016-05-16	Revert "Replace Release() { delete this; } in fde_xml_imp.h"	Tom Sepez
	This reverts commit fa34e805fd03ba81bcfe1148cf96b24fe63b39a0. Reason for revert: broke asan tests. TBR=thestig@chromium.org Review URL: https://codereview.chromium.org/1982843002 .
2016-05-16	Replace Release() { delete this; } in fde_xml_imp.h	tsepez
	Review-Url: https://codereview.chromium.org/1981003002
2016-05-13	Make CFX_WideString(const CFX_WideString&) explicit.	tsepez
	BUG= Review-Url: https://codereview.chromium.org/1979723003
2016-04-28	Do not check pointers before deleting them.	thestig
	XFA edition. Review-Url: https://codereview.chromium.org/1925363002
2016-04-27	Standardize on ASSERT.	dsinclair
	There are currently three ways to assert in the code (ASSERT, FXSYS_assert and assert). This CL standardizes on ASSERT. The benefit of ASSERT is that it can be overridden if the platform requies and we can pickup the Chromium version if it has already been defined in the build. This does change behaviour. Currently FXSYS_assert is always defined but ASSERT is only defined in debug builds. So, the FXSYS_assert's would fire in Release builds. That will no longer happen. BUG=pdfium:219 Review-Url: https://codereview.chromium.org/1914303003
2016-04-19	XFA unused function cleanup	dsinclair
	This CL removes a series of unused XFA methods. Review URL: https://codereview.chromium.org/1899103002
2016-04-14	Simplify XFA event handling.	thestig
	- Avoid casting to void* - Don't bother firing events that no one listens for Review URL: https://codereview.chromium.org/1890563003
2016-04-14	Rename CFX_ByteTextBuf::GetByteString() to AsStringC().	tsepez
	Do the same for CFX_WideTextBuf as well. The name is confusing because these return CFX_ByteStringC, not CFX_ByteString. The AsStringC naming gives parallelism with the string types themselves, and may help to indicate that the result is only good for the lifetime of the object being operated upon. Review URL: https://codereview.chromium.org/1886263003
2016-04-14	Replace calls to deprecated CFX_{Wide,Byte}String::Empty()	tsepez
	Use the more standard name "clear()" instead. Review URL: https://codereview.chromium.org/1888103002
2016-04-13	Remove implicit cast from CFX_WideString to (const wchar_t*)	tsepez
	BUG= Review URL: https://codereview.chromium.org/1882043004
2016-04-08	Rename both As{Byte,Wide}StringC() helpers to AsStringC().	tsepez
	The naming is redundant given the base type, and will stand in the way of consolidating Byte and Wide code. BUG= Review URL: https://codereview.chromium.org/1862123003
2016-04-06	Move remaining xfa/include/fxfa files	dsinclair
	This CL moves the remaining files in xfa/include/fxfa to xfa/fxfa/include. Review URL: https://codereview.chromium.org/1864973005
2016-04-05	Make down-conversion explicit from CFX_Widetring to CFX_WideStringC.	tsepez
	Companion to https://codereview.chromium.org/1853233002 BUG= Review URL: https://codereview.chromium.org/1857073002
2016-04-04	Replace FDE_XMLSYNTAX defines with enum classes.	dsinclair
	This CL replaces FDE_XMLSYNTAXMODE_ with FDE_XmlSyntaxState:: and FDE_XMLSYNTAXSTATUS_ with FDE_XmlSyntaxResult::. The various variables also have their names updated to make them clearer. Review URL: https://codereview.chromium.org/1856003002
2016-04-01	Remove CFX_{Byte,Wide}String::Equal in favor of "==".	tsepez
	Makes the code slightly cleaner. Review URL: https://codereview.chromium.org/1846083002
2016-03-31	Remove IXFA_* interfaces.	dsinclair
	This CL removes the IXFA_* interfaces which are: - Implemented once. - Not implemented by an fpdfsdk class. This requires making a few classes visible to fpdfsdk so we can have the correct instances available instead of the IXFA types. Review URL: https://codereview.chromium.org/1846993002
2016-03-29	Remove IFDE_XML* classes.	dsinclair
	The CFDE_XML* classes did not inhert from the IFDE_XML variants but we casted to them anyway. This CL removes the IFDE_XML* variants and we just use the CPDF_XML* classes directly. BUG=pdfium:357 Review URL: https://codereview.chromium.org/1836353002
2016-03-25	Remove FX_DWORD from XFA.	tsepez
	Review URL: https://codereview.chromium.org/1830323006
2016-03-21	Remove FX_WORD in favor of uint16_t.	Tom Sepez
	It isn't buying us anthing, and it looks strange in a struct when other uint types are already present. R=dsinclair@chromium.org Review URL: https://codereview.chromium.org/1821043003 .
2016-03-14	Move xfa/src up to xfa/.	Dan Sinclair
	This CL moves the xfa/src files up to the xfa/ directory and fixes the includes, include guards, and build files. R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1803723002 .