fix bug

Author: fb55

packages/parse5/lib/parser/index.test.ts

@@ -172,4 +172,51 @@ describe('parser', () => {
             expect((b.childNodes[0] as TextNode).value).toBe('should be outside');
         });
     });
+
+    describe('Customizable Select', () => {
+        it('should NOT enable selectedcontent for select multiple', () => {
+            const html = `
+                <select multiple>
+                    <button>
+                        <selectedcontent></selectedcontent>
+                    </button>
+                    <option selected>foo</option>
+                </select>
+            `;
+
+            const doc = parse(html);
+            const htmlEl = doc.childNodes.find((n) => n.nodeName === 'html') as Element;
+            const body = htmlEl.childNodes.find((n) => n.nodeName === 'body') as Element;
+            const select = body.childNodes.find((n) => (n as Element).tagName === 'select') as Element;
+            const button = select.childNodes.find((n) => (n as Element).tagName === 'button') as Element;
+            const selectedcontent = button.childNodes.find(
+                (n) => (n as Element).tagName === 'selectedcontent',
+            ) as Element;
+
+            expect(selectedcontent.childNodes.length).toBe(0);
+        });
+
+        it('should enable selectedcontent for select without multiple', () => {
+            const html = `
+                <select>
+                    <button>
+                        <selectedcontent></selectedcontent>
+                    </button>
+                    <option selected>foo</option>
+                </select>
+            `;
+
+            const doc = parse(html);
+            const htmlEl = doc.childNodes.find((n) => n.nodeName === 'html') as Element;
+            const body = htmlEl.childNodes.find((n) => n.nodeName === 'body') as Element;
+            const select = body.childNodes.find((n) => (n as Element).tagName === 'select') as Element;
+            const button = select.childNodes.find((n) => (n as Element).tagName === 'button') as Element;
+            const selectedcontent = button.childNodes.find(
+                (n) => (n as Element).tagName === 'selectedcontent',
+            ) as Element;
+
+            expect(selectedcontent.childNodes.length).toBeGreaterThan(0);
+            expect((selectedcontent.childNodes[0] as TextNode).value).toBe('foo');
+        });
+    });
 });

packages/parse5/lib/parser/index.ts

@@ -2292,7 +2292,7 @@ function selectStartTagInBody<T extends TreeAdapterTypeMap>(p: Parser<T>, token:
     p.framesetOk = false;
 
     // Initialize selectedcontent state for this select
-    p.selectedcontentSelect = p.openElements.current;
+    p.selectedcontentSelect = getTokenAttr(token, 'multiple') === null ? p.openElements.current : null;
     p.enabledSelectedcontent = null;
     p.firstOptionInSelect = null;
     p.selectedOptionInSelect = null;