content: Move math block parsing to the callers of parseBlockContent

Prepares for #1130, this commit is almost NFC with only
difference that, in an error case we previously emitted a
`ParagraphNode` containing an `UnimplementedInlineContentNode`
(along with any adjacent nodes), now we emit a single
`UnimplementedBlockContentNode` instead.

Author: rajveermalviya

lib/model/content.dart

@@ -1471,21 +1471,6 @@ class _ZulipContentParser {
     }
 
     if (localName == 'p' && className.isEmpty) {
-      // Oddly, the way a math block gets encoded in Zulip HTML is inside a <p>.
-      if (element.nodes case [dom.Element(localName: 'span') && var child, ...]) {
-        if (child.className == 'katex-display') {
-          if (element.nodes case [_]
-                              || [_, dom.Element(localName: 'br'),
-                                     dom.Text(text: "\n")]) {
-            // This might be too specific; we'll find out when we do #190.
-            // The case with the `<br>\n` can happen when at the end of a quote;
-            // it seems like a glitch in the server's Markdown processing,
-            // so hopefully there just aren't any further such glitches.
-            return parseMathBlock(child);
-          }
-        }
-      }
-
       final parsed = parseBlockInline(element.nodes);
       return ParagraphNode(debugHtmlNode: debugHtmlNode,
         links: parsed.links,
@@ -1599,6 +1584,30 @@ class _ZulipContentParser {
     for (final node in nodes) {
       if (node is dom.Text && (node.text == '\n')) continue;
 
+      // Oddly, the way a math block gets encoded in Zulip HTML is inside a <p>.
+      if (node case dom.Element(localName: 'p', className: '', nodes: [
+            dom.Element(
+              localName: 'span',
+              className: 'katex-display') && final child, ...])) {
+        final BlockContentNode parsed;
+        if (node.nodes case [_]
+                         || [_, dom.Element(localName: 'br'),
+                                dom.Text(text: "\n")]) {
+          // This might be too specific; we'll find out when we do #190.
+          // The case with the `<br>\n` can happen when at the end of a quote;
+          // it seems like a glitch in the server's Markdown processing,
+          // so hopefully there just aren't any further such glitches.
+          parsed = parseMathBlock(child);
+        } else {
+          parsed = UnimplementedBlockContentNode(htmlNode: node);
+        }
+
+        if (currentParagraph.isNotEmpty) consumeParagraph();
+        if (imageNodes.isNotEmpty) consumeImageNodes();
+        result.add(parsed);
+        continue;
+      }
+
       if (_isPossibleInlineNode(node)) {
         if (imageNodes.isNotEmpty) {
           consumeImageNodes();
@@ -1642,6 +1651,29 @@ class _ZulipContentParser {
         continue;
       }
 
+      // Oddly, the way a math block gets encoded in Zulip HTML is inside a <p>.
+      if (node case dom.Element(localName: 'p', className: '', nodes: [
+            dom.Element(
+              localName: 'span',
+              className: 'katex-display') && final child, ...])) {
+        final BlockContentNode parsed;
+        if (node.nodes case [_]
+                         || [_, dom.Element(localName: 'br'),
+                                dom.Text(text: "\n")]) {
+          // This might be too specific; we'll find out when we do #190.
+          // The case with the `<br>\n` can happen when at the end of a quote;
+          // it seems like a glitch in the server's Markdown processing,
+          // so hopefully there just aren't any further such glitches.
+          parsed = parseMathBlock(child);
+        } else {
+          parsed = UnimplementedBlockContentNode(htmlNode: node);
+        }
+
+        if (imageNodes.isNotEmpty) consumeImageNodes();
+        result.add(parsed);
+        continue;
+      }
+
       final block = parseBlockContent(node);
       if (block is ImageNode) {
         imageNodes.add(block);