From 2185fbd591767afdfb0b1a335c05796cce179fe0 Mon Sep 17 00:00:00 2001
From: MrRaindrop <tekkahs@gmail.com>
Date: Wed, 31 Jan 2018 12:11:22 +0800
Subject: [PATCH] fix web component.

---
 examples/components/web.vue           |  5 +-
 package.json                          |  2 +-
 packages/weex-vue-render/package.json |  2 +-
 src/components/web/index.js           | 78 ++++++++++++++++++++++-----
 4 files changed, 70 insertions(+), 17 deletions(-)

diff --git a/examples/components/web.vue b/examples/components/web.vue
index 5c58491..e4e3228 100644
--- a/examples/components/web.vue
+++ b/examples/components/web.vue
@@ -12,7 +12,9 @@
         @click.native="refresh"></button>
     </div>
     <web class="content" ref="webview" :src="src"
-      @pagestart="startload" @pagefinish="finishload" @error="failload"></web>
+      @pagestart="startload"
+      @pagefinish="finishload"
+      @error="failload"></web>
   </div>
 </template>
 
@@ -20,6 +22,7 @@
   var webview = weex.requireModule('webview');
   const src = 'http://invalid.src/for/test'
   const newSrc = 'http://m.taobao.com'
+  // const newSrc = window.location.href.replace(/\?[^?]*$/, '')  // for test.
   module.exports = {
     data () {
       return {
diff --git a/package.json b/package.json
index bcc7bfd..e38e73f 100755
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "weex-vue-render",
   "description": "Web renderer for weex project written in Vue DSL.",
-  "version": "1.0.15",
+  "version": "1.0.16",
   "license": "Apache-2.0",
   "main": "dist/index.common.js",
   "keywords": [
diff --git a/packages/weex-vue-render/package.json b/packages/weex-vue-render/package.json
index 9f2d28e..89e2b91 100644
--- a/packages/weex-vue-render/package.json
+++ b/packages/weex-vue-render/package.json
@@ -1,6 +1,6 @@
 {
   "name": "weex-vue-render",
-  "version": "1.0.15",
+  "version": "1.0.16",
   "description": "Web renderer for weex project written in Vue DSL.",
   "license": "Apache-2.0",
   "main": "dist/index.common.js",
diff --git a/src/components/web/index.js b/src/components/web/index.js
index e52bdde..b85cdcb 100644
--- a/src/components/web/index.js
+++ b/src/components/web/index.js
@@ -23,41 +23,77 @@ function getWeb (weex) {
   const { dispatchNativeEvent } = weex.utils
 
   return {
+    data () {
+      return {
+        currentSrc: ''
+      }
+    },
     name: 'weex-web',
     props: {
       src: String
     },
+    watch: {
+      src (newVal) {
+        this.currentSrc = newVal
+      }
+    },
     methods: {
-      // TODO: check cross-origin
       goBack () {
-        if (this.$el) {
-          this.$el.contentWindow.history.back()
+        const el = this.$el
+        if (el) {
+          const win = el.contentWindow
+          try {
+            win.history.back()
+            this.currentSrc = win.location.href
+          }
+          catch (err) {
+            dispatchNativeEvent(el, 'error', err)
+          }
         }
       },
       goForward () {
-        if (this.$el) {
-          this.$el.contentWindow.history.forward()
+        const el = this.$el
+        if (el) {
+          const win = el.contentWindow
+          try {
+            win.history.forward()
+            this.currentSrc = win.location.href
+          }
+          catch (err) {
+            dispatchNativeEvent(el, 'error', err)
+          }
         }
       },
       reload () {
-        if (this.$el) {
-          this.$el.contentWindow.history.reload()
+        const el = this.$el
+        if (el) {
+          try {
+            el.contentWindow.location.reload()
+            dispatchNativeEvent(el, 'pagestart', { url: this.currentSrc })
+          }
+          catch (err) {
+            dispatchNativeEvent(el, 'error', err)
+          }
         }
       }
     },
 
+    created () {
+      this.currentSrc = this.src
+    },
+
     mounted () {
       const el = this.$el
-      this._prevSrc = this.src
+      this._prevSrc = this.currentSrc
       if (el) {
-        dispatchNativeEvent(el, 'pagestart', { url: this.src })
+        dispatchNativeEvent(el, 'pagestart', { url: this.currentSrc })
       }
     },
 
     updated () {
-      if (this.src !== this._prevSrc) {
-        this._prevSrc = this.src
-        dispatchNativeEvent(this.$el, 'pagestart', { url: this.src })
+      if (this.currentSrc !== this._prevSrc) {
+        this._prevSrc = this.currentSrc
+        dispatchNativeEvent(this.$el, 'pagestart', { url: this.currentSrc })
       }
     },
 
@@ -65,11 +101,25 @@ function getWeb (weex) {
       return createElement('iframe', {
         attrs: {
           'weex-type': 'web',
-          src: this.src
+          src: this.currentSrc
         },
         on: {
           load: event => {
-            dispatchNativeEvent(event.target, 'pagefinish', { url: this.src })
+            this.$nextTick(function () {
+              const el = this.$el
+              try {
+                const html = el.contentWindow.document.documentElement
+                if (html) {
+                  dispatchNativeEvent(el, 'pagefinish', { url: this.currentSrc })
+                }
+                else {
+                  dispatchNativeEvent(el, 'error', new Error('[vue-render]:found no page content.'))
+                }
+              }
+              catch (err) {
+                dispatchNativeEvent(el, 'error', err)
+              }
+            })
           }
         },
         staticClass: 'weex-web weex-el',