fix: One weird Webkit issue fix at bindings parser

Author: finom

src/parsebindings/index.js

@@ -89,6 +89,11 @@ export default function parseBindings(object, givenNodes, eventOptions) {
             // fixes Firefox issue: attributes.length can be changed by processAttribute
             const attrs = attributes.length > 1 ? [...attributes] : attributes;
             nofn.forEach(attrs, (attribute) => {
+                // Sometimes Webkit returns an attribute itself when attribute.value is accessed
+                if (attribute.value && typeof attribute.value.value === 'string') {
+                    attribute = attribute.value; // eslint-disable-line no-param-reassign
+                }
+
                 if (bindingReg.test(attribute.value)) {
                     processAttribute({
                         node,

test/spec/bindings/bindings_parser_spec.js

@@ -93,6 +93,27 @@ describe('Bindings parser', () => {
         expect(node.value).toEqual(obj.x);
     });
 
+    it('should bind input value with type=text attribute (bugfix)', () => {
+        const node = parse('<input type="text" value="{{x}}">');
+        const obj = {};
+
+        parseBindings(obj, node, noDebounceFlag);
+        obj.x = 'foo';
+        expect(node.value).toEqual(obj.x);
+    });
+
+    it('should bind select value (bugfix)', () => {
+        const node = parse(`<select value="{{x}}">
+            <option selected value="bar">bar</option>
+            <option value="foo">foo</option>
+        </select>`);
+        const obj = {};
+
+        parseBindings(obj, node, noDebounceFlag);
+        obj.x = 'foo';
+        expect(node.value).toEqual(obj.x);
+    });
+
     it('should bind complex input value', () => {
         const node = parse('<input value="{{x}} {{y}}">');
         const obj = {};