feat(weex): WIP mark recycle list child component root

Author: yyx990803

Diff for: src/compiler/helpers.js

@@ -26,6 +26,12 @@ export function addAttr (el: ASTElement, name: string, value: any) {
   el.plain = false
 }
 
+// add a raw attr (use this in preTransforms)
+export function addRawAttr (el: ASTElement, name: string, value: any) {
+  el.attrsMap[name] = value
+  el.attrsList.push({ name, value })
+}
+
 export function addDirective (
   el: ASTElement,
   name: string,

Diff for: src/core/vdom/create-component.js

@@ -4,7 +4,6 @@ import VNode from './vnode'
 import { resolveConstructorOptions } from 'core/instance/init'
 import { queueActivatedComponent } from 'core/observer/scheduler'
 import { createFunctionalComponent } from './create-functional-component'
-import { renderRecyclableComponentTemplate } from 'weex/runtime/recycle-list/render-component-template'
 
 import {
   warn,
@@ -28,6 +27,11 @@ import {
   deactivateChildComponent
 } from '../instance/lifecycle'
 
+import {
+  isRecyclableComponent,
+  renderRecyclableComponentTemplate
+} from 'weex/runtime/recycle-list/render-component-template'
+
 // hooks to be invoked on component VNodes during patch
 const componentVNodeHooks = {
   init (
@@ -196,7 +200,7 @@ export function createComponent (
   // Weex specific: invoke recycle-list optimized @render function for
   // extracting cell-slot template.
   // https://github.com/Hanks10100/weex-native-directive/tree/master/component
-  if (__WEEX__ && data.attrs && ('@inRecycleList' in data.attrs)) {
+  if (__WEEX__ && isRecyclableComponent(vnode)) {
     return renderRecyclableComponentTemplate(vnode)
   }
 

Diff for: src/platforms/weex/compiler/modules/recycle-list/component-root.js

@@ -0,0 +1,14 @@
+/* @flow */
+
+import { addRawAttr } from 'compiler/helpers'
+
+// mark component root nodes as
+export function preTransformComponentRoot (
+  el: ASTElement,
+  options: WeexCompilerOptions
+) {
+  if (!el.parent) {
+    // component root
+    addRawAttr(el, '$isComponentRoot', true)
+  }
+}

Diff for: src/platforms/weex/compiler/modules/recycle-list/component.js

@@ -1,13 +1,16 @@
 /* @flow */
 
-import { addAttr } from 'compiler/helpers'
+import { addRawAttr } from 'compiler/helpers'
 import { RECYCLE_LIST_MARKER } from 'weex/util/index'
 
 // mark components as inside recycle-list so that we know we need to invoke
 // their special @render function instead of render in create-component.js
-export function postTransformComponent (el: ASTElement, options: WeexCompilerOptions) {
+export function preTransformComponent (
+  el: ASTElement,
+  options: WeexCompilerOptions
+) {
   // $flow-disable-line (we know isReservedTag is there)
   if (!options.isReservedTag(el.tag) && el.tag !== 'cell-slot') {
-    addAttr(el, RECYCLE_LIST_MARKER, true)
+    addRawAttr(el, RECYCLE_LIST_MARKER, 'true')
   }
 }

Diff for: src/platforms/weex/compiler/modules/recycle-list/index.js

@@ -1,6 +1,7 @@
 /* @flow */
 
-import { postTransformComponent } from './component'
+import { preTransformComponent } from './component'
+import { preTransformComponentRoot } from './component-root'
 import { postTransformText } from './text'
 import { preTransformVBind } from './v-bind'
 import { preTransformVIf } from './v-if'
@@ -19,6 +20,8 @@ function preTransformNode (el: ASTElement, options: WeexCompilerOptions) {
     currentRecycleList = el
   }
   if (shouldCompile(el, options)) {
+    preTransformComponent(el, options)
+    preTransformComponentRoot(el, options)
     preTransformVBind(el, options)
     preTransformVIf(el, options) // also v-else-if and v-else
     preTransformVFor(el, options)
@@ -33,7 +36,6 @@ function transformNode (el: ASTElement, options: WeexCompilerOptions) {
 
 function postTransformNode (el: ASTElement, options: WeexCompilerOptions) {
   if (shouldCompile(el, options)) {
-    postTransformComponent(el, options)
     // <text>: transform children text into value attr
     if (el.tag === 'text') {
       postTransformText(el, options)

Diff for: src/platforms/weex/compiler/modules/recycle-list/v-bind.js

@@ -2,7 +2,7 @@
 
 import { camelize } from 'shared/util'
 import { bindRE } from 'compiler/parser/index'
-import { getAndRemoveAttr } from 'compiler/helpers'
+import { getAndRemoveAttr, addRawAttr } from 'compiler/helpers'
 
 function parseAttrName (name: string): string {
   return camelize(name.replace(bindRE, ''))
@@ -16,10 +16,7 @@ export function preTransformVBind (el: ASTElement, options: WeexCompilerOptions)
         '@binding': getAndRemoveAttr(el, attr)
       }
       delete el.attrsMap[attr]
-      el.attrsMap[name] = value
-      el.attrsList.push({ name, value })
-      // addAttr(el, name, value)
-      // el.hasBindings = false
+      addRawAttr(el, name, value)
     }
   }
 }

Diff for: src/platforms/weex/compiler/modules/recycle-list/v-for.js

@@ -1,7 +1,7 @@
 /* @flow */
 
 import { forAliasRE, forIteratorRE, stripParensRE } from 'compiler/parser/index'
-import { getAndRemoveAttr } from 'compiler/helpers'
+import { getAndRemoveAttr, addRawAttr } from 'compiler/helpers'
 
 export function preTransformVFor (el: ASTElement, options: WeexCompilerOptions) {
   const exp = getAndRemoveAttr(el, 'v-for')
@@ -26,8 +26,7 @@ export function preTransformVFor (el: ASTElement, options: WeexCompilerOptions)
       }
     }
     delete el.attrsMap['v-for']
-    el.attrsMap['[[repeat]]'] = desc
-    el.attrsList.push({ name: '[[repeat]]', value: desc })
+    addRawAttr(el, '[[repeat]]', desc)
   } else if (process.env.NODE_ENV !== 'production' && options.warn) {
     options.warn(`Invalid v-for expression: ${exp}`)
   }

Diff for: src/platforms/weex/compiler/modules/recycle-list/v-if.js

@@ -1,6 +1,6 @@
 /* @flow */
 
-import { getAndRemoveAttr } from 'compiler/helpers'
+import { getAndRemoveAttr, addRawAttr } from 'compiler/helpers'
 
 function hasConditionDirective (el: ASTElement): boolean {
   for (const attr in el.attrsMap) {
@@ -42,7 +42,6 @@ export function preTransformVIf (el: ASTElement, options: WeexCompilerOptions) {
         return
       }
     }
-    el.attrsMap['[[match]]'] = exp
-    el.attrsList.push({ name: '[[match]]', value: exp })
+    addRawAttr(el, '[[match]]', exp)
   }
 }

Diff for: src/platforms/weex/util/index.js

@@ -4,7 +4,7 @@ declare var document: Object;
 import { makeMap } from 'shared/util'
 import { warn } from 'core/util/index'
 
-export const RECYCLE_LIST_MARKER = '@inRecycleList'
+export const RECYCLE_LIST_MARKER = '$inRecycleList'
 
 export const isReservedTag = makeMap(
   'template,script,style,element,content,slot,link,meta,svg,view,' +