Revert "Traces: add temporary debug info to LoadBinaryData"

This reverts commit 0d701c81d0.

Reason for revert: CRC is ok: http://b/296921272#comment3

Original change's description:
> Traces: add temporary debug info to LoadBinaryData
>
> Unable to repro the bug by triggering tasks on swarming, hopefully this
> eventually catches something.
>
> Bug: angleproject:8307
> Change-Id: I34dd0c8a9e82e54f3a07e2d7a249ddfaa543bae7
> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/4794110
> Reviewed-by: Cody Northrop <cnorthrop@google.com>
> Commit-Queue: Roman Lavrov <romanl@google.com>

Bug: angleproject:8307
Change-Id: I0f04baf5241e99d7c2514bcdc583e58e2fd51535
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/4834063
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Roman Lavrov <romanl@google.com>
This commit is contained in:
Roman Lavrov
2023-09-01 13:01:23 +00:00
committed by Angle LUCI CQ
parent 7df4aa1057
commit 2ef238bcb6

View File

@@ -33,23 +33,6 @@ bool LoadJSONFromFile(const std::string &fileName, rapidjson::Document *doc)
doc->ParseStream(inWrapper);
return !doc->HasParseError();
}
// https://anglebug.com/8307: temporary checks (copied from RendererVk.cpp)
uint16_t ComputeCRC16(const uint8_t *data, const size_t size)
{
constexpr uint16_t kPolynomialCRC16 = 0x8408;
uint16_t rem = 0;
for (size_t i = 0; i < size; i++)
{
rem ^= data[i];
for (int j = 0; j < 8; j++)
{
rem = (rem & 1) ? kPolynomialCRC16 ^ (rem >> 1) : rem >> 1;
}
}
return rem;
}
} // namespace
bool LoadTraceNamesFromJSON(const std::string jsonFilePath, std::vector<std::string> *namesOut)
@@ -234,10 +217,6 @@ uint8_t *TraceLibrary::LoadBinaryData(const char *fileName)
exit(1);
}
// https://anglebug.com/8307: temporary checks
std::cout << "Compressed binary data size=" << size
<< " crc16=" << ComputeCRC16(compressedData.data(), size) << "\n";
uint32_t uncompressedSize =
zlib_internal::GetGzipUncompressedSize(compressedData.data(), compressedData.size());