[Windows] Remove NOTREACHED from IsReparsePoint.

This CL removes the NOTREACHED check from the function as it doesn't
provide any value and just causes unexpected crashes in the browser
process during debug builds.

Bug: 1029073
Change-Id: I14de816b9fd4415238fa4244316baa4c06fcdc15
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1992167
Reviewed-by: Will Harris <wfh@chromium.org>
Commit-Queue: James Forshaw <forshaw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#729609}

sandbox/win/src/win_utils.cc

@@ -234,7 +234,6 @@ DWORD IsReparsePoint(const std::wstring& full_path) {
             (path.rfind(L'\\') == kNTDotPrefixLen - 1)) {
           break;
         }
-        NOTREACHED_NT();
         return error;
       }
     } else if (FILE_ATTRIBUTE_REPARSE_POINT & attributes) {