Revert "blink image_decode_bench: use more Chromium stuff"

This reverts commit 1c320598.

Reason for revert: broke compilation on Win
https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8932944647631542128/+/steps/compile/0/stdout

../../third_party/blink/renderer/platform/testing/image_decode_bench.cc(43,26):  error: no matching conversion for functional-style cast from 'const char *' to 'base::FilePath'
    const auto file_path = base::FilePath(name);

Original change's description:
> blink image_decode_bench: use more Chromium stuff
> 
> This CL refactors image_decode_bench:
> - Rewrites function ReadFile() to use base::FilePath etc.
> - Configures the Logging and changes fprintf/stderr to LOG()
>  statements, and exit()s into LOG_ASSERT().
> - Names the constant kDefaultDecodeTimes.
> - s/DecodeError/PrintErrorAndExit/ to clarify what it does.
> - Changes uses of CurrentTimeTicks() with base::ElapsedTimer,
>  and uses TimeDelta ISO double to count the elapsed time.
> - Uses perf_test::PrintResult to add info about the image
>  (size and #pixels), when a new parameter (-v / -verbose)
>  is specified after the current two parameters, file and
>  #iterations; output is e.g.:
> *RESULT decode_time /usr/local/google/home/mcasas/Pictures/4.webp: 100 times 790528 pixels (1024x772)= 66.27827 ms/sample
> 
> Bug: 889898
> Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
> Change-Id: Ic0a66d2aa3d2f08b13c8af99aed31e61d7246dec
> Reviewed-on: https://chromium-review.googlesource.com/c/1225653
> Commit-Queue: Miguel Casas <mcasas@chromium.org>
> Reviewed-by: Leon Scroggins <scroggo@chromium.org>
> Reviewed-by: Noel Gordon <noel@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#598803}

TBR=scroggo@chromium.org,noel@chromium.org,mcasas@chromium.org

Change-Id: I7f18f8a464536eb5b6cee86f25930a94f357a0f1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 889898
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Reviewed-on: https://chromium-review.googlesource.com/c/1277245Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#598809}

third_party/blink/renderer/platform/BUILD.gn

@@ -1865,7 +1865,6 @@ jumbo_source_set("blink_platform_unittests_sources") {
 }
 
 executable("image_decode_bench") {
-  testonly = true
   visibility = []  # Allow re-assignment of list.
   visibility = [ "*" ]
 
@@ -1875,9 +1874,7 @@ executable("image_decode_bench") {
 
   deps = [
     ":platform",
-    "//base",
     "//mojo/core/embedder",
-    "//testing/perf",
     "//third_party/blink/renderer/platform/wtf",
   ]
 

third_party/blink/renderer/platform/testing/image_decode_bench.cc

@@ -7,7 +7,7 @@
 // clocks to measure image decode time. Basic usage:
 //
 //   % ninja -C out/Release image_decode_bench &&
-//      ./out/Release/image_decode_bench file [iterations [-v,-verbose]]
+//      ./out/Release/image_decode_bench file [iterations]
 //
 // TODO(noel): Consider adding md5 checksum support to WTF. Use it to compute
 // the decoded image frame md5 and output that value.
@@ -19,13 +19,9 @@
 #include <fstream>
 
 #include "base/command_line.h"
-#include "base/files/file_util.h"
 #include "base/memory/scoped_refptr.h"
 #include "base/message_loop/message_loop.h"
-#include "base/strings/stringprintf.h"
-#include "base/timer/elapsed_timer.h"
 #include "mojo/core/embedder/embedder.h"
-#include "testing/perf/perf_test.h"
 #include "third_party/blink/public/platform/platform.h"
 #include "third_party/blink/renderer/platform/image-decoders/image_decoder.h"
 #include "third_party/blink/renderer/platform/shared_buffer.h"
@@ -36,14 +32,34 @@ namespace blink {
 
 namespace {
 
-constexpr size_t kDefaultDecodeTimes = 1;
-
 scoped_refptr<SharedBuffer> ReadFile(const char* name) {
-  std::string file_contents;
-  const auto file_path = base::FilePath(name);
-  if (!base::ReadFileToString(file_path, &file_contents))
-    PLOG(FATAL) << "file name: " << name;
-  return SharedBuffer::Create(file_contents.data(), file_contents.size());
+  std::ifstream file(name, std::ios::in | std::ios::binary);
+  if (!file) {
+    fprintf(stderr, "Cannot open file %s\n", name);
+    exit(2);
+  }
+
+  file.seekg(0, std::ios::end);
+  std::streampos file_size = file.tellg();
+  file.seekg(0, std::ios::beg);
+
+  if (!file || file_size <= 0) {
+    fprintf(stderr, "Error seeking file %s\n", name);
+    exit(2);
+  }
+
+  if (file_size > std::numeric_limits<wtf_size_t>::max()) {
+    fprintf(stderr, "File size too large %s\n", name);
+    exit(2);
+  }
+
+  Vector<char> buffer(static_cast<wtf_size_t>(file_size));
+  if (!file.read(buffer.data(), file_size)) {
+    fprintf(stderr, "Error reading file %s\n", name);
+    exit(2);
+  }
+
+  return SharedBuffer::AdoptVector(buffer);
 }
 
 struct ImageMeta {
@@ -51,64 +67,66 @@ struct ImageMeta {
   int width;
   int height;
   int frames;
+  // Cumulative time in seconds to decode all frames.
+  double time;
 };
 
-void PrintErrorAndExit(ImageMeta* image) {
-  LOG_ASSERT(false) << "Failed to decode image " << image->name;
+void DecodeFailure(ImageMeta* image) {
+  fprintf(stderr, "Failed to decode image %s\n", image->name);
+  exit(3);
 }
 
-base::TimeDelta DecodeImageData(SharedBuffer* data, ImageMeta* image) {
-  constexpr bool data_complete = true;
+void DecodeImageData(SharedBuffer* data, ImageMeta* image) {
+  const bool data_complete = true;
   std::unique_ptr<ImageDecoder> decoder = ImageDecoder::Create(
       data, data_complete, ImageDecoder::kAlphaPremultiplied,
       ImageDecoder::kDefaultBitDepth, ColorBehavior::Ignore());
 
-  base::ElapsedTimer timer;
+  auto start = CurrentTimeTicks();
+
   bool all_data_received = true;
   decoder->SetData(data, all_data_received);
 
   size_t frame_count = decoder->FrameCount();
   for (size_t index = 0; index < frame_count; ++index) {
     if (!decoder->DecodeFrameBufferAtIndex(index))
-      PrintErrorAndExit(image);
+      DecodeFailure(image);
   }
 
-  const auto elapsed_time = timer.Elapsed();
+  image->time += (CurrentTimeTicks() - start).InSecondsF();
 
   if (!frame_count || decoder->Failed())
-    PrintErrorAndExit(image);
+    DecodeFailure(image);
 
   image->width = decoder->Size().Width();
   image->height = decoder->Size().Height();
   image->frames = frame_count;
-  return elapsed_time;
 }
 
 }  // namespace
 
-void ImageDecodeBenchMain(int argc, char* argv[]) {
+int ImageDecodeBenchMain(int argc, char* argv[]) {
   base::CommandLine::Init(argc, argv);
   const char* program = argv[0];
 
   if (argc < 2) {
-    LOG(INFO) << "Usage: " << program << " file [iterations [-v,-verbose]]";
+    fprintf(stderr, "Usage: %s file [iterations]\n", program);
     exit(1);
   }
 
   // Control bench decode iterations.
-  size_t decode_iterations = kDefaultDecodeTimes;
-  bool verbose_mode = false;
+
+  size_t decode_iterations = 1;
   if (argc >= 3) {
     char* end = nullptr;
     decode_iterations = strtol(argv[2], &end, 10);
     if (*end != '\0' || !decode_iterations) {
-      LOG_ASSERT(false) << "Second argument should be number of iterations. "
-                        << "The default is 1. You supplied " << argv[2];
+      fprintf(stderr,
+              "Second argument should be number of iterations. "
+              "The default is 1. You supplied %s\n",
+              argv[2]);
+      exit(1);
     }
-
-    const base::CommandLine* cmd_line = base::CommandLine::ForCurrentProcess();
-    DCHECK(cmd_line);
-    verbose_mode = cmd_line->HasSwitch("v") || cmd_line->HasSwitch("verbose");
   }
 
   std::unique_ptr<Platform> platform = std::make_unique<Platform>();
@@ -116,26 +134,25 @@ void ImageDecodeBenchMain(int argc, char* argv[]) {
 
   // Read entire file content into |data| (a contiguous block of memory) then
   // decode it to verify the image and record its ImageMeta data.
-  ImageMeta image = {.name = argv[1]};
+
+  ImageMeta image = {argv[1], 0, 0, 0, 0};
   scoped_refptr<SharedBuffer> data = ReadFile(argv[1]);
   DecodeImageData(data.get(), &image);
 
-  // Image decode bench for |decode_iterations|.
-  base::TimeDelta total_time;
-  for (size_t i = 0; i < decode_iterations; ++i)
-    total_time += DecodeImageData(data.get(), &image);
-
-  if (verbose_mode) {
-    perf_test::PrintResult(
-        "decode_time ", argv[1],
-        base::StringPrintf("%zu times %d pixels (%dx%d)", decode_iterations,
-                           image.width * image.height, image.width,
-                           image.height),
-        total_time.InMillisecondsF() / decode_iterations, "ms/sample", true);
-  } else {
-    printf("%f %f\n", total_time.InSecondsF(),
-           total_time.InSecondsF() / decode_iterations);
+  // Image decode bench for decode_iterations.
+
+  double total_time = 0.0;
+  for (size_t i = 0; i < decode_iterations; ++i) {
+    image.time = 0.0;
+    DecodeImageData(data.get(), &image);
+    total_time += image.time;
   }
+
+  // Results to stdout.
+
+  double average_time = total_time / decode_iterations;
+  printf("%f %f\n", total_time, average_time);
+  return 0;
 }
 
 }  // namespace blink
@@ -143,8 +160,5 @@ void ImageDecodeBenchMain(int argc, char* argv[]) {
 int main(int argc, char* argv[]) {
   base::MessageLoop message_loop;
   mojo::core::Init();
-  base::CommandLine::Init(argc, argv);
-  LOG_ASSERT(logging::InitLogging(logging::LoggingSettings()));
-
-  blink::ImageDecodeBenchMain(argc, argv);
+  return blink::ImageDecodeBenchMain(argc, argv);
 }