Simplify test by calling strlen() instead of std::move().

(The motivation here is to fix a compile error on the libc++ bot -- looks like
std::size() is a C++17 thing, and libc++ only enables these functions with
/std:c++17 while the MSVC STL allows it even in C++14 language mode. But using
strlen() is simpler too.)

Bug: 801780
Change-Id: Ic955935253e61d3a3947a314a2bdc3e0a9354016
Reviewed-on: https://chromium-review.googlesource.com/1182081Reviewed-by: Greg Thompson <grt@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585089}

base/win/pe_image_unittest.cc

@@ -4,7 +4,6 @@
 
 // This file contains unit tests for PEImage.
 #include <algorithm>
-#include <iterator>
 #include <vector>
 
 #include "base/files/file_path.h"
@@ -214,7 +213,7 @@ TEST(PEImageTest, GetDebugId) {
   LPCSTR pdb_file = nullptr;
   size_t pdb_file_length = 0;
   EXPECT_TRUE(pe.GetDebugId(&guid, &age, &pdb_file, &pdb_file_length));
-  EXPECT_EQ(pdb_file_length, std::size(kPdbFileName) - 1);
+  EXPECT_EQ(pdb_file_length, strlen(kPdbFileName));
   EXPECT_STREQ(pdb_file, kPdbFileName);
 
   // Should be valid to call without parameters.