Re-enable MSVC warning 4800 for compiling with chromium_code

core/src/fpdfapi/fpdf_parser/cpdf_hint_tables.cpp

 // Copyright 2016 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "core/src/fpdfapi/fpdf_parser/cpdf_hint_tables.h"
 
 #include "core/include/fxcrt/fx_safe_types.h"
 #include "core/include/fpdfapi/cpdf_array.h"
@@ skipping 249 matching lines @@
   if (nFirstPageObjNum < 0)
     return FALSE;
 
   FX_DWORD dwPrevObjLen = 0;
   FX_DWORD dwCurObjLen = 0;
   FX_SAFE_DWORD required_bits = dwSharedObjTotal;
   required_bits *= dwDeltaGroupLen;
   if (!CanReadFromBitStream(hStream, required_bits))
     return FALSE;
 
-  for (int i = 0; i < dwSharedObjTotal; ++i) {
+  for (size_t i = 0; i < dwSharedObjTotal; ++i) {

[Tom Sepez, 2016/03/11 00:30:28]

nit: how about FX_DWORD, to match the type of dwSharedObjTotal?

[Wei Li, 2016/03/11 04:11:36]

Done.

     dwPrevObjLen = dwCurObjLen;
     FX_SAFE_DWORD safeObjLen = hStream->GetBits(dwDeltaGroupLen);
     safeObjLen += dwGroupLeastLen;
     if (!safeObjLen.IsValid())
       return FALSE;
 
     dwCurObjLen = safeObjLen.ValueOrDie();
     if (i < m_nFirstPageSharedObjs) {
       m_dwSharedObjNumArray.Add(nFirstPageObjNum + i);
       if (i == 0)
@@ skipping 179 matching lines @@
   CPDF_Array* pRange = m_pLinearizedDict->GetArrayBy("H");
   if (!pRange)
     return -1;
 
   CPDF_Object* pStreamLen = pRange->GetElementValue(1);
   if (!pStreamLen)
     return -1;
 
   return pStreamLen->GetInteger();
 }