Fixed escaping during autolink

Implementing the HTML escaping code manually is necessary to avoid the
following exception during testing:

Method escapeHtml in android.text.Html not mocked

app/src/main/java/eu/faircode/email/HtmlHelper.java

@@ -127,15 +127,22 @@ public class HtmlHelper {
 
     static String autolink(String text) {
         Matcher matcher = pattern.matcher(text);
-        StringBuffer sb = new StringBuffer();
+        StringBuilder sb = new StringBuilder();
+        int end = 0;
         while (matcher.find()) {
-            String ref = matcher.group();
+            sb.append(html(text.substring(end, matcher.start())));
-            matcher.appendReplacement(sb, String.format("<a href=\"%s\">%s</a>", ref, ref));
+            String ref = html(matcher.group());
+            sb.append(String.format("<a href=\"%s\">%s</a>", ref, ref));
+            end = matcher.end();
         }
-        matcher.appendTail(sb);
+        sb.append(text.substring(end));
         return sb.toString();
     }
 
+    private static String html(String plain) {
+        return plain.replace("&", "&amp;").replace("<", "&lt;");
+    }
+
     static Drawable decodeImage(String source, Context context, long id, boolean show) {
         int px = Helper.dp2pixels(context, 48);
 

app/src/test/java/eu/faircode/email/HtmlHelperTest.java

@@ -28,10 +28,10 @@ class HtmlHelperTest {
         testAutolink(
                 "https://example.org/search?q=%C3%A4&hl=nl",
 
-                // TODO: Strictly speaking, the & should be encoded as &.
+                // The & should be encoded as &, even though
-                // Most browsers can deal with this situation though.
+                // most browsers can deal with this situation.
-                "<a href=\"https://example.org/search?q=%C3%A4&hl=nl\">" +
+                "<a href=\"https://example.org/search?q=%C3%A4&amp;hl=nl\">" +
-                        "https://example.org/search?q=%C3%A4&hl=nl</a>"
+                        "https://example.org/search?q=%C3%A4&amp;hl=nl</a>"
         );
 
         testAutolink(
@@ -43,8 +43,8 @@ class HtmlHelperTest {
         testAutolink(
                 "Go to <http://example.org/>.",
 
-                // FIXME: The < must be encoded as &lt;.
+                // The < must be encoded as &lt;, to avoid confusion.
-                "Go to <<a href=\"http://example.org/\">http://example.org/</a>>."
+                "Go to &lt;<a href=\"http://example.org/\">http://example.org/</a>>."
         );
 
         testAutolink(