Revert "Sort Win SDK versions by version sorting rules"

This reverts commit 82b6a7c1.

Reason for revert: Breaks the build for Windows on Arm.

Original change's description:
> Sort Win SDK versions by version sorting rules
> 
> This should have no practical effect on current data but version
> numbers need to be sorted so that 1.12 is higher than 1.9 and
> the current code didn't do that.
> 
> Change-Id: Ie993d2c103b6f16453c8b03336484c3b05a0097b
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1605980
> Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
> Commit-Queue: Daniel Bratell <bratell@opera.com>
> Cr-Commit-Position: refs/heads/master@{#659016}

TBR=bratell@opera.com,brucedawson@chromium.org

Change-Id: I126df32184965a97fd3397db61c1ecde1c2681e1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1609787
Commit-Queue: Daniel Bratell <bratell@opera.com>
Reviewed-by: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#659052}

build/vs_toolchain.py

@@ -196,23 +196,6 @@ def _CopyRuntimeImpl(target, source, verbose=True):
     # keep it readable.
     os.chmod(target, stat.S_IWRITE | stat.S_IREAD)
 
-def _SortByHighestVersionNumberFirst(list_of_str_versions):
-  """This sorts |list_of_str_versions| according to version number rules
-  so that version "1.12" is higher than version "1.9". Does not work
-  with non-numeric versions like 1.4.a8 which will be higher than
-  1.4.a12. It does handle the versions being embedded in file paths.
-  """
-  def to_int_if_int(x):
-    try:
-      return int(x)
-    except ValueError:
-      return x
-
-  def to_number_sequence(x):
-    part_sequence = re.split(r'[\\/\.]', x)
-    return [to_int_if_int(x) for x in part_sequence]
-
-  list_of_str_versions.sort(key=to_number_sequence, reverse=True)
 
 def _CopyUCRTRuntime(target_dir, source_dir, target_cpu, dll_pattern, suffix):
   """Copy both the msvcp and vccorlib runtime DLLs, only if the target doesn't
@@ -249,7 +232,7 @@ def _CopyUCRTRuntime(target_dir, source_dir, target_cpu, dll_pattern, suffix):
     redist_dir = os.path.join(win_sdk_dir, 'Redist')
     version_dirs = glob.glob(os.path.join(redist_dir, '10.*'))
     if len(version_dirs) > 0:
-      _SortByHighestVersionNumberFirst(version_dirs)
+      version_dirs.sort(reverse=True)
       redist_dir = version_dirs[0]
     ucrt_dll_dirs = os.path.join(redist_dir, 'ucrt', 'DLLs', target_cpu)
     ucrt_files = glob.glob(os.path.join(ucrt_dll_dirs, 'api-ms-win-*.dll'))
@@ -266,7 +249,7 @@ def _CopyUCRTRuntime(target_dir, source_dir, target_cpu, dll_pattern, suffix):
       sdk_redist_root = os.path.join(win_sdk_dir, 'bin')
       sdk_bin_sub_dirs = os.listdir(sdk_redist_root)
       # Select the most recent SDK if there are multiple versions installed.
-      _SortByHighestVersionNumberFirst(sdk_bin_sub_dirs)
+      sdk_bin_sub_dirs.sort(reverse=True)
       for directory in sdk_bin_sub_dirs:
         sdk_redist_root_version = os.path.join(sdk_redist_root, directory)
         if not os.path.isdir(sdk_redist_root_version):
@@ -291,7 +274,7 @@ def FindVCComponentRoot(component):
       'VC', component, 'MSVC')
   vc_component_msvc_contents = os.listdir(vc_component_msvc_root)
   # Select the most recent toolchain if there are several.
-  _SortByHighestVersionNumberFirst(vc_component_msvc_root)
+  vc_component_msvc_contents.sort(reverse=True)
   for directory in vc_component_msvc_contents:
     if not os.path.isdir(os.path.join(vc_component_msvc_root, directory)):
       continue