fix(weex): use document instead of renderer to create Element

Author: Hanks10100

build/config.js

@@ -17,7 +17,7 @@ const banner =
 
 const weexFactoryPlugin = {
   intro () {
-    return 'module.exports = function weexFactory (exports, renderer) {'
+    return 'module.exports = function weexFactory (exports, document) {'
   },
   outro () {
     return '}'

src/platforms/weex/entry-framework.js

@@ -1,32 +1,19 @@
-import TextNode from 'weex/runtime/text-node'
-
 // this will be preserved during build
 const VueFactory = require('./factory')
 
 const instances = {}
 
-const renderer = {
-  TextNode
-}
-
 /**
- * Prepare framework config, basically about the virtual-DOM and JS bridge.
- * @param {object} cfg
+ * Prepare framework config.
+ * Nothing need to do actually, just an interface provided to weex runtime.
  */
-export function init (cfg) {
-  renderer.Document = cfg.Document
-  renderer.Element = cfg.Element
-  renderer.Comment = cfg.Comment
-}
+export function init () {}
 
 /**
  * Reset framework config and clear all registrations.
  */
 export function reset () {
   clear(instances)
-  delete renderer.Document
-  delete renderer.Element
-  delete renderer.Comment
 }
 
 /**
@@ -182,7 +169,7 @@ export function receiveTasks (id, tasks) {
  */
 function createVueModuleInstance (instanceId, weex) {
   const exports = {}
-  VueFactory(exports, renderer)
+  VueFactory(exports, weex.document)
   const Vue = exports.Vue
 
   const instance = instances[instanceId]

src/platforms/weex/runtime/node-ops.js

@@ -1,22 +1,24 @@
-/* globals renderer */
-// renderer is injected by weex factory wrapper
+/* globals document */
+// document is injected by weex factory wrapper
+
+import TextNode from 'weex/runtime/text-node'
 
 export const namespaceMap = {}
 
 export function createElement (tagName) {
-  return new renderer.Element(tagName)
+  return document.createElement(tagName)
 }
 
 export function createElementNS (namespace, tagName) {
-  return new renderer.Element(namespace + ':' + tagName)
+  return document.createElement(namespace + ':' + tagName)
 }
 
 export function createTextNode (text) {
-  return new renderer.TextNode(text)
+  return new TextNode(text)
 }
 
 export function createComment (text) {
-  return new renderer.Comment(text)
+  return document.createComment(text)
 }
 
 export function insertBefore (node, target, before) {

src/platforms/weex/util/index.js

@@ -1,4 +1,4 @@
-/* globals renderer */
+/* globals document */
 
 import { makeMap } from 'shared/util'
 
@@ -34,8 +34,8 @@ export function getTagNamespace () { /* console.log('getTagNamespace') */ }
 export function isUnknownElement () { /* console.log('isUnknownElement') */ }
 
 export function query (el, document) {
-  // renderer is injected by weex factory wrapper
-  const placeholder = new renderer.Comment('root')
+  // document is injected by weex factory wrapper
+  const placeholder = document.createComment('root')
   placeholder.hasAttribute = placeholder.removeAttribute = function () {} // hack for patch
   document.documentElement.appendChild(placeholder)
   return placeholder