Remove libxml Visual Studio fix patch

This removes a libxml patch we have which now makes merge conflicts with
the latest libxml commits.

The patch is for a bug in visual studio here:
https://connect.microsoft.com/VisualStudio/feedback/details/2582138
The link doesn't work anymore but is probably fixed. Either way, there
is an automated test which will catch the relevant bug if it is still
reproduces: fast/xsl/xslt-translate.html

Bug: 599427
Change-Id: I09f9fe27dbfcf79f762406810a5ccf09d606d7de
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3428248
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Cr-Commit-Position: refs/heads/main@{#965730}
NOKEYCHECK=True
GitOrigin-RevId: 2db8d5ac301455cb6c1448d3be16ecce122f03b1
diff --git a/README.chromium b/README.chromium
index 1d1d773..7f5f636 100644
--- a/README.chromium
+++ b/README.chromium
@@ -12,7 +12,6 @@
 
 Modifications:
 
-- chromium-issue-599427.patch: workaround for VS 2015 Update 2 code-gen bug
 - chromium-issue-628581.patch: See https://crbug.com/628581#c18
 - chromium-issue-894933.patch: Use ptrdiff_t instead of unsigned long for
     pointer differences in parser.c
diff --git a/chromium/chromium-issue-599427.patch b/chromium/chromium-issue-599427.patch
deleted file mode 100644
index 20e26c5..0000000
--- a/chromium/chromium-issue-599427.patch
+++ /dev/null
@@ -1,28 +0,0 @@
---- a/xmlstring.c	2016-05-13 17:24:58.870079372 +0900
-+++ b/xmlstring.c	2016-04-25 13:58:09.879238595 +0900
-@@ -835,16 +835,20 @@
-     while ( len-- > 0) {
-         if ( !*ptr )
-             break;
--        if ( (ch = *ptr++) & 0x80)
--            while ((ch<<=1) & 0x80 ) {
--		if (*ptr == 0) break;
-+        if ( (ch = *ptr++) & 0x80) {
-+            // Workaround for an optimization bug in VS 2015 Update 2, remove
-+            // once the fix is released. crbug.com/599427
-+            // https://connect.microsoft.com/VisualStudio/feedback/details/2582138
-+            xmlChar ch2 = ch;
-+            while ((ch2<<=1) & 0x80 ) {
-                 ptr++;
--	    }
-+                if (*ptr == 0) break;
-+            }
-+        }
-     }
-     return (ptr - utf);
- }
- 
--
- /**
-  * xmlUTF8Strndup:
-  * @utf:  the input UTF8 *
diff --git a/chromium/roll.py b/chromium/roll.py
index ef23d4c..e936c25 100755
--- a/chromium/roll.py
+++ b/chromium/roll.py
@@ -66,7 +66,6 @@
 #    e. Complete the review as usual
 
 PATCHES = [
-    'chromium-issue-599427.patch',
     'chromium-issue-628581.patch',
     'libxml2-2.9.4-security-xpath-nodetab-uaf.patch',
     'chromium-issue-708434.patch',
diff --git a/src/xmlstring.c b/src/xmlstring.c
index 62d3053..e8a1e45 100644
--- a/src/xmlstring.c
+++ b/src/xmlstring.c
@@ -846,20 +846,16 @@
     while ( len-- > 0) {
         if ( !*ptr )
             break;
-        if ( (ch = *ptr++) & 0x80) {
-            // Workaround for an optimization bug in VS 2015 Update 2, remove
-            // once the fix is released. crbug.com/599427
-            // https://connect.microsoft.com/VisualStudio/feedback/details/2582138
-            xmlChar ch2 = ch;
-            while ((ch2<<=1) & 0x80 ) {
+        if ( (ch = *ptr++) & 0x80)
+            while ((ch<<=1) & 0x80 ) {
+		if (*ptr == 0) break;
                 ptr++;
-                if (*ptr == 0) break;
-            }
-        }
+	    }
     }
     return (ptr - utf);
 }
 
+
 /**
  * xmlUTF8Strndup:
  * @utf:  the input UTF8 *