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

Unified Diff: core/fxcrt/fx_xml_parser.cpp

Issue 2430743003: in the attempt to fix 627393, changed IFX_FileRead's readBlock to return the length it reads
Patch Set: fix an undefined variable Created 4 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
Index: core/fxcrt/fx_xml_parser.cpp
diff --git a/core/fxcrt/fx_xml_parser.cpp b/core/fxcrt/fx_xml_parser.cpp
index e1a209a701e55829e674f36e7f846782e5e6f134..8559b7c008d223e0ebd20af765921121de8ca473 100644
--- a/core/fxcrt/fx_xml_parser.cpp
+++ b/core/fxcrt/fx_xml_parser.cpp
@@ -153,7 +153,8 @@ bool CXML_DataStmAcc::ReadNextBlock(bool bRestart) {
if (!m_pBuffer)
m_pBuffer = FX_Alloc(uint8_t, m_dwSize);
- return m_pFileRead->ReadBlock(m_pBuffer, m_nStart, m_dwSize);
+ return m_pFileRead->ReadBlock(m_pBuffer, m_nStart, m_dwSize) == m_dwSize ||
+ m_pFileRead->IsEOF();
}
const uint8_t* CXML_DataStmAcc::GetBlockBuffer() {

Powered by Google App Engine
This is Rietveld 408576698