fix: do not cache scoped slots when mixed with normal slots

Author: yyx990803

src/core/vdom/helpers/normalize-scoped-slots.js

@@ -15,8 +15,14 @@ export function normalizeScopedSlots (
   } else if (slots._normalized) {
     // fast path 1: child component re-render only, parent did not change
     return slots._normalized
-  } else if (slots.$stable && prevSlots && prevSlots !== emptyObject) {
-    // fast path 2: stable scoped slots, only need to normalize once
+  } else if (
+    slots.$stable &&
+    prevSlots &&
+    prevSlots !== emptyObject &&
+    Object.keys(normalSlots).length === 0
+  ) {
+    // fast path 2: stable scoped slots w/ no normal slots to proxy,
+    // only need to normalize once
     return prevSlots
   } else {
     res = {}

test/unit/features/component/component-scoped-slot.spec.js

@@ -1103,4 +1103,29 @@ describe('Component scoped slot', () => {
     }).$mount()
     expect(vm.$el.textContent).toBe('hello')
   })
+
+  it('should not cache scoped slot normalzation when there are a mix of normal and scoped slots', done => {
+    const foo = {
+      template: `<div><slot name="foo" /> <slot name="bar" /></div>`
+    }
+
+    const vm = new Vue({
+      data: {
+        msg: 'foo'
+      },
+      template: `
+        <foo>
+          <div slot="foo">{{ msg }}</div>
+          <template #bar><div>bar</div></template>
+        </foo>
+      `,
+      components: { foo }
+    }).$mount()
+
+    expect(vm.$el.textContent).toBe(`foo bar`)
+    vm.msg = 'baz'
+    waitForUpdate(() => {
+      expect(vm.$el.textContent).toBe(`baz bar`)
+    }).then(done)
+  })
 })