Add notes and fix some small problems

Author: VisualSJ

extensions/ccui/uiwidgets/UIVideoPlayer.js

@@ -202,6 +202,10 @@ ccui.VideoPlayer = ccui.Widget.extend({
 
 });
 
+/**
+ * The VideoPlayer support list of events
+ * @type {{PLAYING: string, PAUSED: string, STOPPED: string, COMPLETED: string}}
+ */
 ccui.VideoPlayer.EventType = {
     PLAYING: "ui_video_play",
     PAUSED: "ui_video_pause",
@@ -215,7 +219,7 @@ ccui.VideoPlayer.EventType = {
      * @devicePixelRatio Whether you need to consider devicePixelRatio calculated position
      * @event To get the data using events
      */
-    video.polyfill = {
+    video._polyfill = {
         devicePixelRatio: false,
         event: "canplay",
         canPlayType: []
@@ -224,14 +228,14 @@ ccui.VideoPlayer.EventType = {
     (function(){
         var dom = document.createElement("video");
         if(dom.canPlayType("video/ogg"))
-            video.polyfill.canPlayType.push(".ogg");
+            video._polyfill.canPlayType.push(".ogg");
         if(dom.canPlayType("video/mp4"))
-            video.polyfill.canPlayType.push(".mp4");
+            video._polyfill.canPlayType.push(".mp4");
     })();
 
     if(cc.sys.OS_IOS === cc.sys.os){
-        video.polyfill.devicePixelRatio = true;
-        video.polyfill.event = "progress";
+        video._polyfill.devicePixelRatio = true;
+        video._polyfill.event = "progress";
     }
 
 })(ccui.VideoPlayer);
@@ -402,4 +406,4 @@ ccui.VideoPlayer.EventType = {
         }
     };
 
-})(ccui.VideoPlayer.polyfill);
+})(ccui.VideoPlayer._polyfill);

extensions/ccui/uiwidgets/UIWebView.js

@@ -83,7 +83,7 @@ ccui.WebView = ccui.Widget.extend({
      * go back
      */
     goBack: function(){
-        if(ccui.WebView.polyfill.closeHistory)
+        if(ccui.WebView._polyfill.closeHistory)
             return cc.log("The current browser does not support the GoBack");
         var iframe = this._renderCmd._iframe;
         if(iframe){
@@ -97,7 +97,7 @@ ccui.WebView = ccui.Widget.extend({
      * go forward
      */
     goForward: function(){
-        if(ccui.WebView.polyfill.closeHistory)
+        if(ccui.WebView._polyfill.closeHistory)
             return cc.log("The current browser does not support the GoForward");
         var iframe = this._renderCmd._iframe;
         if(iframe){
@@ -116,6 +116,7 @@ ccui.WebView = ccui.Widget.extend({
         if(iframe){
             var win = iframe.contentWindow;
             try{
+                cc.eventManager.dispatchCustomEvent(ccui.WebView.EventType.JS_EVALUATED);
                 win.eval(str);
             }catch(err){
                 console.error(err);
@@ -161,16 +162,6 @@ ccui.WebView = ccui.Widget.extend({
         }
     },
 
-    //setOnShouldStartLoading: function(callback){},
-    //setOnDidFinishLoading: function(){},
-    //setOnDidFailLoading: function(){},
-    //setOnJSCallback: function(){},
-
-    //getOnShouldStartLoading: function(){},
-    //getOnDidFinishLoading: function(){},
-    //getOnDidFailLoading: function(){},
-    //getOnJSCallback: function(){},
-
     _createRenderCmd: function(){
         return new ccui.WebView.RenderCmd(this);
     },
@@ -206,18 +197,19 @@ ccui.WebView = ccui.Widget.extend({
 ccui.WebView.EventType = {
     LOADING: "ui_webview_loading",
     LOADED: "ui_webview_load",
-    ERROR: "ui_webview_error"
+    ERROR: "ui_webview_error",
+    JS_EVALUATED: "ui_webview_js"
 };
 
 (function(){
 
-    var polyfill = ccui.WebView.polyfill = {
+    var polyfill = ccui.WebView._polyfill = {
         devicePixelRatio: false,
         enableDiv: false
     };
 
     if(cc.sys.os === cc.sys.OS_IOS)
-        ccui.WebView.polyfill.enableDiv = true;
+        polyfill.enableDiv = true;
 
     if(cc.sys.isMobile){
         if(cc.sys.browserType === cc.sys.BROWSER_TYPE_FIREFOX){
@@ -383,4 +375,4 @@ ccui.WebView.EventType = {
         }
     };
 
-})(ccui.WebView.polyfill);
+})(ccui.WebView._polyfill);