[GitHub] incubator-weex issue #1365: [WEEX-528][iOS] fix textaera set padding-right i...

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1365
  





  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex issue #1364: [WEEX-504][iOS] set :active,placeho...

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1364
  






  

  
  Messages

  
  
  :book:
  has no jsfm file changed.

  

  :book:
  jsfm test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #1365: [WEEX-528][iOS] fix textaera set padding-...

2018-07-23 Thread VIceguest
GitHub user VIceguest opened a pull request:

https://github.com/apache/incubator-weex/pull/1365

[WEEX-528][iOS] fix textaera set padding-right invalid

http://dotwe.org/vue/7c839c2484f247095e27f0276acaabc8
placeholder在设置padding-right时显示也异常

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/VIceguest/incubator-weex 
0.18.0-hotfix-textarea-padding-right

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1365.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1365


commit ce111effe2704bfc3406229d11b74621a34025db
Author: jiangbin 
Date:   2018-07-24T05:51:09Z

[WEEX-528][iOS] fix textaera set padding-right invalid




---


[GitHub] incubator-weex pull request #1364: [WEEX-504][iOS] set :active,...

2018-07-23 Thread VIceguest
GitHub user VIceguest opened a pull request:

https://github.com/apache/incubator-weex/pull/1364

[WEEX-504][iOS]  set :active,placeholder display when activ…

textarea 设置 :active伪类,placeholder不消失,与当前文字叠加
显示

demo参考http://dotwe.org/vue/c45ae83c092e7ca988ffc6952077d10a

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/VIceguest/incubator-weex 
0.18.0-hotfix-textarea-placeholder

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1364.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1364


commit 344557be8efacf3661b734315b2dc59e164d1e16
Author: jiangbin 
Date:   2018-07-24T05:48:36Z

[WEEX-504][iOS]  set :active,placeholder display when active 
regardless of have text




---


[GitHub] incubator-weex issue #1363: [weex-529] [Android] Allow register custom gestu...

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1363
  






  

  
  Messages

  
  
  :book:
  has no jsfm file changed.

  

  :book:
  jsfm test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #1363: [weex-529] [Android] Allow register custo...

2018-07-23 Thread Rowandjj
GitHub user Rowandjj opened a pull request:

https://github.com/apache/incubator-weex/pull/1363

[weex-529] [Android] Allow register custom gesture listeners 

https://issues.apache.org/jira/browse/WEEX-529

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Rowandjj/incubator-weex 
android-feature-gesture-v2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1363.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1363


commit c735686ab29b44d40bf827a60a1eb72ae42b254d
Author: chuyi 
Date:   2018-07-23T06:20:13Z

[WEEX-529] android WXGesture class allow add custom gesture listener




---


[GitHub] incubator-weex pull request #1362: [weex-529] [Android] Allow register custo...

2018-07-23 Thread Rowandjj
Github user Rowandjj closed the pull request at:

https://github.com/apache/incubator-weex/pull/1362


---


[GitHub] incubator-weex pull request #1328: [WEEX-504][iOS] set :active,...

2018-07-23 Thread VIceguest
Github user VIceguest closed the pull request at:

https://github.com/apache/incubator-weex/pull/1328


---


[GitHub] incubator-weex pull request #1355: [WEEX-528][iOS] fix textaera set padding-...

2018-07-23 Thread VIceguest
Github user VIceguest closed the pull request at:

https://github.com/apache/incubator-weex/pull/1355


---


[GitHub] incubator-weex issue #1362: [weex-529] [Android] Allow register custom gestu...

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1362
  





  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #1362: [weex-529] [Android] Allow register custo...

2018-07-23 Thread Rowandjj
GitHub user Rowandjj opened a pull request:

https://github.com/apache/incubator-weex/pull/1362

[weex-529] [Android] Allow register custom gesture listeners

https://issues.apache.org/jira/browse/WEEX-529



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Rowandjj/incubator-weex 
android-feature-gesture

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1362.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1362


commit c735686ab29b44d40bf827a60a1eb72ae42b254d
Author: chuyi 
Date:   2018-07-23T06:20:13Z

[WEEX-529] android WXGesture class allow add custom gesture listener

commit 82bb8965b5f310accc9376c02d32e6cd93889b55
Author: miomin 
Date:   2018-07-23T07:30:42Z

* [android] Update libweexcore.so




---


[GitHub] incubator-weex issue #1361: * [WEEX-534][Android] Add JS Error Log in Graphi...

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1361
  





  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #1361: * [WEEX-534][Android] Add JS Error Log in...

2018-07-23 Thread YorkShen
GitHub user YorkShen opened a pull request:

https://github.com/apache/incubator-weex/pull/1361

* [WEEX-534][Android] Add JS Error Log in GraphicActionAddElement

https://issues.apache.org/jira/projects/WEEX/issues/WEEX-534

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/YorkShen/incubator-weex 
android_addElementException

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1361.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1361


commit 40e105cc7b2acec16d9ee72cb85a305b57191f6a
Author: YorkShen 
Date:   2018-07-23T14:32:19Z

* [Android] Add JS Error Log in GraphicActionAddElement

Set component.view.ContentDescription to ref if there is no 
contentDescription.




---


incubator-weex git commit: [WEEX-527][Android] Sandbox support to debug

2018-07-23 Thread kyork
Repository: incubator-weex
Updated Branches:
  refs/heads/master 291e48b3e -> a120d35e9


[WEEX-527][Android] Sandbox support to debug


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/a120d35e
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/a120d35e
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/a120d35e

Branch: refs/heads/master
Commit: a120d35e9f5dd5bf7448d5e729997bdfcebac973
Parents: 291e48b
Author: miomin 
Authored: Fri Jul 20 17:32:34 2018 +0800
Committer: YorkShen 
Committed: Mon Jul 23 20:27:54 2018 +0800

--
 .../src/main/java/com/taobao/weex/WXEnvironment.java|  4 ++--
 .../java/com/taobao/weex/bridge/WXBridgeManager.java| 12 ++--
 2 files changed, 8 insertions(+), 8 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/a120d35e/android/sdk/src/main/java/com/taobao/weex/WXEnvironment.java
--
diff --git a/android/sdk/src/main/java/com/taobao/weex/WXEnvironment.java 
b/android/sdk/src/main/java/com/taobao/weex/WXEnvironment.java
index 86d70c4..cdb3aeb 100644
--- a/android/sdk/src/main/java/com/taobao/weex/WXEnvironment.java
+++ b/android/sdk/src/main/java/com/taobao/weex/WXEnvironment.java
@@ -79,7 +79,7 @@ public class WXEnvironment {
* Debug model
*/
   public static boolean sDebugMode = false;
-  public static final boolean sForceEnableDevTool = false;
+  public static final boolean sForceEnableDevTool = true;
   public static String sDebugWsUrl = "";
   public static boolean sDebugServerConnectable = false;
   public static boolean sRemoteDebugMode = false;
@@ -255,7 +255,7 @@ public class WXEnvironment {
*/
   e.printStackTrace();
 }
-return false;
+return true;
   }
 
   public static boolean isPerf() {

http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/a120d35e/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java
--
diff --git 
a/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java 
b/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java
index 02ae49c..c0bf2c5 100644
--- a/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java
+++ b/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java
@@ -224,7 +224,7 @@ public class WXBridgeManager implements Callback, 
BactchExecutor {
 setJSFrameworkInit(false);
 WXModuleManager.resetAllModuleState();
 String jsf = "";
-if (!isSandBoxContext || WXEnvironment.sDebugServerConnectable) {
+if (!isSandBoxContext) {
   jsf = WXFileUtils.loadAsset("main.js", 
WXEnvironment.getApplication());
 } else {
   jsf = WXFileUtils.loadAsset("weex-main-jsfm.js", 
WXEnvironment.getApplication());
@@ -240,7 +240,7 @@ public class WXBridgeManager implements Callback, 
BactchExecutor {
 setJSFrameworkInit(false);
 WXModuleManager.resetAllModuleState();
 String jsf = "";
-if (!isSandBoxContext || WXEnvironment.sDebugServerConnectable) {
+if (!isSandBoxContext) {
   jsf = WXFileUtils.loadAsset("main.js", 
WXEnvironment.getApplication());
 } else {
   jsf = WXFileUtils.loadAsset("weex-main-jsfm.js", 
WXEnvironment.getApplication());
@@ -1291,12 +1291,12 @@ public class WXBridgeManager implements Callback, 
BactchExecutor {
 
 // if { "framework": "Vue" } or  { "framework": "Rax" } will use 
invokeCreateInstanceContext
 // others will use invokeExecJS
-if (!isSandBoxContext || WXEnvironment.sDebugServerConnectable) {
+if (!isSandBoxContext) {
   invokeExecJS(instance.getInstanceId(), null, METHOD_CREATE_INSTANCE, 
args, false);
   return;
 }
 if (type == BundType.Vue || type == BundType.Rax) {
-  invokeCreateInstanceContext(instance.getInstanceId(), null, 
METHOD_CREATE_INSTANCE, args, false);
+  invokeCreateInstanceContext(instance.getInstanceId(), null, 
"createInstanceContext", args, false);
   return;
 } else {
   invokeExecJS(instance.getInstanceId(), null, METHOD_CREATE_INSTANCE, 
args, false);
@@ -1315,7 +1315,7 @@ public class WXBridgeManager implements Callback, 
BactchExecutor {
   }
 
   public WXJSObject optionObjConvert(boolean useSandBox, BundType type, 
WXJSObject opt) {
-if (!useSandBox || type == BundType.Others || 
WXEnvironment.sDebugServerConnectable) {
+if (!useSandBox || type == BundType.Others) {
   return opt;
 }
 try {
@@ -1604,7 +1604,7 @@ public class WXBridgeManager implements Callback, 
BactchExecutor {
 // if 

[GitHub] incubator-weex issue #1360: * [WEEX-533][Android] Fix animation memory leak

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1360
  






  

  
  Messages

  
  
  :book:
  has no jsfm file changed.

  

  :book:
  jsfm test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


incubator-weex git commit: * [Android] Fix animation memory leak

2018-07-23 Thread kyork
Repository: incubator-weex
Updated Branches:
  refs/heads/master 41038abc5 -> 291e48b3e


* [Android] Fix animation memory leak


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/291e48b3
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/291e48b3
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/291e48b3

Branch: refs/heads/master
Commit: 291e48b3e304305f5d71c2df484f952101450249
Parents: 41038ab
Author: YorkShen 
Authored: Mon Jul 23 20:18:56 2018 +0800
Committer: YorkShen 
Committed: Mon Jul 23 20:18:56 2018 +0800

--
 android/sdk/src/main/java/com/taobao/weex/WXSDKEngine.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/291e48b3/android/sdk/src/main/java/com/taobao/weex/WXSDKEngine.java
--
diff --git a/android/sdk/src/main/java/com/taobao/weex/WXSDKEngine.java 
b/android/sdk/src/main/java/com/taobao/weex/WXSDKEngine.java
index b7f7df6..c8c4d78 100644
--- a/android/sdk/src/main/java/com/taobao/weex/WXSDKEngine.java
+++ b/android/sdk/src/main/java/com/taobao/weex/WXSDKEngine.java
@@ -327,7 +327,7 @@ public class WXSDKEngine implements Serializable {
 
   registerModule("modal", WXModalUIModule.class, false);
   registerModule("instanceWrap", WXInstanceWrap.class, false);
-  registerModule("animation", WXAnimationModule.class, true);
+  registerModule("animation", WXAnimationModule.class, false);
   registerModule("webview", WXWebViewModule.class, true);
   registerModule("navigator", WXNavigatorModule.class);
   registerModule("stream", WXStreamModule.class);



[GitHub] incubator-weex pull request #1360: * [WEEX-533][Android] Fix animation memor...

2018-07-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-weex/pull/1360


---


[GitHub] incubator-weex pull request #1360: * [WEEX-533][Android] Fix animation memor...

2018-07-23 Thread YorkShen
GitHub user YorkShen opened a pull request:

https://github.com/apache/incubator-weex/pull/1360

* [WEEX-533][Android] Fix animation memory leak

https://issues.apache.org/jira/browse/WEEX-533

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/YorkShen/incubator-weex 
bugfix_animation_memoryleak

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1360.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1360


commit 291e48b3e304305f5d71c2df484f952101450249
Author: YorkShen 
Date:   2018-07-23T12:18:56Z

* [Android] Fix animation memory leak




---


[GitHub] incubator-weex pull request #1359: [WEEX-532][Android] Fix Iconfont issue

2018-07-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-weex/pull/1359


---


incubator-weex git commit: * [android] Fix Iconfont issue

2018-07-23 Thread kyork
Repository: incubator-weex
Updated Branches:
  refs/heads/master 907315429 -> 41038abc5


* [android] Fix Iconfont issue

http://rax.alibaba-inc.com/playground/28149825-d0c2-490a-b048-17832cf31886

This closes #1359


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/41038abc
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/41038abc
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/41038abc

Branch: refs/heads/master
Commit: 41038abc572d17af4e51f05f148ba8647aaa148f
Parents: 9073154
Author: YorkShen 
Authored: Mon Jul 23 18:01:43 2018 +0800
Committer: YorkShen 
Committed: Mon Jul 23 20:03:36 2018 +0800

--
 android/sdk/src/main/java/com/taobao/weex/utils/FontDO.java | 9 +++--
 1 file changed, 7 insertions(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/41038abc/android/sdk/src/main/java/com/taobao/weex/utils/FontDO.java
--
diff --git a/android/sdk/src/main/java/com/taobao/weex/utils/FontDO.java 
b/android/sdk/src/main/java/com/taobao/weex/utils/FontDO.java
index ca81fa8..cc5d23f 100644
--- a/android/sdk/src/main/java/com/taobao/weex/utils/FontDO.java
+++ b/android/sdk/src/main/java/com/taobao/weex/utils/FontDO.java
@@ -100,7 +100,12 @@ public class FontDO {
   String base64Data = data[1];
   if (!TextUtils.isEmpty(base64Data)) {
 String md5 = WXFileUtils.md5(base64Data);
-File tmpFile = new File(new 
File(WXEnvironment.getApplication().getCacheDir(), "font-family"), md5);
+File cacheDir = new 
File(WXEnvironment.getApplication().getCacheDir(),
+"font-family");
+if (!cacheDir.exists()) {
+  cacheDir.mkdirs();
+}
+File tmpFile = new File(cacheDir, md5);
 if(!tmpFile.exists()){
   tmpFile.createNewFile();
   WXFileUtils.saveFile(tmpFile.getPath(), 
Base64.decode(base64Data, Base64.DEFAULT), WXEnvironment.getApplication());
@@ -118,7 +123,7 @@ public class FontDO {
 mState = STATE_INIT;
   } catch (Exception e) {
 mType = STATE_INVALID;
-WXLogUtils.e("TypefaceUtil", "URI.create(mUrl) failed mUrl: " + mUrl);
+WXLogUtils.e("TypefaceUtil", "URI.create(mUrl) failed mUrl: " + mUrl+ 
"\n"+ WXLogUtils.getStackTrace(e));
   }
 } else {
   mUrl = src;



[GitHub] incubator-weex pull request #1357: [WEEX-524][iOS] Try to fix a crash operat...

2018-07-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-weex/pull/1357


---


[1/2] incubator-weex git commit: Try to fix a crash operating mutable array in multi-thread

2018-07-23 Thread cxfeng
Repository: incubator-weex
Updated Branches:
  refs/heads/master e36eec286 -> 907315429


Try to fix a crash operating mutable array in multi-thread


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/1172a3b6
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/1172a3b6
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/1172a3b6

Branch: refs/heads/master
Commit: 1172a3b62269b80f909ad17a2b07cbfe2300d7dd
Parents: e36eec2
Author: xuyouyang 
Authored: Mon Jul 23 12:06:42 2018 +0800
Committer: xuyouyang 
Committed: Mon Jul 23 12:06:42 2018 +0800

--
 .../WeexSDK/Sources/Bridge/WXBridgeContext.m| 67 +++-
 1 file changed, 37 insertions(+), 30 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/1172a3b6/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
--
diff --git a/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m 
b/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
index 6fab4ab..cfb0e7b 100644
--- a/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
+++ b/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
@@ -422,12 +422,13 @@ _Pragma("clang diagnostic pop") \
 
 - (WXSDKInstance *)topInstance
 {
-if (self.insStack.count > 0) {
-WXSDKInstance *topInstance = [WXSDKManager 
instanceForID:[self.insStack firstObject]];
-return topInstance;
-}
-
-return nil;
+   WXSDKInstance *topInstance = nil;
+   @synchronized(self) {
+   if (self.insStack.count > 0) {
+   topInstance = [WXSDKManager 
instanceForID:[self.insStack firstObject]];
+   }
+   }
+return topInstance;
 }
 
 - (NSMutableDictionary *)sendQueue
@@ -493,14 +494,16 @@ _Pragma("clang diagnostic pop") \
 {
 WXAssertBridgeThread();
 WXAssertParam(instanceIdString);
-
-if (![self.insStack containsObject:instanceIdString]) {
-if ([options[@"RENDER_IN_ORDER"] boolValue]) {
-[self.insStack addObject:instanceIdString];
-} else {
-[self.insStack insertObject:instanceIdString atIndex:0];
-}
-}
+   
+   @synchronized(self) {
+   if (![self.insStack containsObject:instanceIdString]) {
+   if ([options[@"RENDER_IN_ORDER"] boolValue]) {
+   [self.insStack addObject:instanceIdString];
+   } else {
+   [self.insStack insertObject:instanceIdString 
atIndex:0];
+   }
+   }
+   }
 
 //create a sendQueue bind to the current instance
 NSMutableArray *sendQueue = [NSMutableArray array];
@@ -658,9 +661,11 @@ _Pragma("clang diagnostic pop") \
 WXAssertParam(instance);
 
 //remove instance from stack
-if([self.insStack containsObject:instance]){
-[self.insStack removeObject:instance];
-}
+   @synchronized(self) {
+   if([self.insStack containsObject:instance]){
+   [self.insStack removeObject:instance];
+   }
+   }
 
 if(_jsBridge && [_jsBridge respondsToSelector:@selector(removeTimers:)]){
 [_jsBridge removeTimers:instance];
@@ -896,19 +901,21 @@ _Pragma("clang diagnostic pop") \
 BOOL hasTask = NO;
 NSMutableArray *tasks = [NSMutableArray array];
 NSString *execIns = nil;
-
-for (NSString *instance in self.insStack) {
-NSMutableArray *sendQueue = self.sendQueue[instance];
-if(sendQueue.count > 0){
-hasTask = YES;
-for(WXCallJSMethod *method in sendQueue){
-[tasks addObject:[method callJSTask]];
-}
-[sendQueue removeAllObjects];
-execIns = instance;
-break;
-}
-}
+   
+   @synchronized(self) {
+   for (NSString *instance in self.insStack) {
+   NSMutableArray *sendQueue = self.sendQueue[instance];
+   if(sendQueue.count > 0){
+   hasTask = YES;
+   for(WXCallJSMethod *method in sendQueue){
+   [tasks addObject:[method callJSTask]];
+   }
+   [sendQueue removeAllObjects];
+   execIns = instance;
+   break;
+   }
+   }
+   }
 
 if ([tasks count] > 0 && execIns) {
 WXSDKInstance * execInstance = [WXSDKManager instanceForID:execIns];



[2/2] incubator-weex git commit: Try to fix a crash operating mutable array in multi-thread

2018-07-23 Thread cxfeng
Try to fix a crash operating mutable array in multi-thread


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/90731542
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/90731542
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/90731542

Branch: refs/heads/master
Commit: 907315429de7183bd64e39627f5c65e61b24d2f4
Parents: 1172a3b
Author: xuyouyang 
Authored: Mon Jul 23 14:19:48 2018 +0800
Committer: xuyouyang 
Committed: Mon Jul 23 14:19:48 2018 +0800

--
 ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m | 8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/90731542/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
--
diff --git a/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m 
b/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
index cfb0e7b..cbc7354 100644
--- a/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
+++ b/ios/sdk/WeexSDK/Sources/Bridge/WXBridgeContext.m
@@ -65,7 +65,7 @@ _Pragma("clang diagnostic pop") \
 //store the methods which will be executed from native to js
 @property (nonatomic, strong) NSMutableDictionary   *sendQueue;
 //the instance stack
-@property (nonatomic, strong) WXThreadSafeMutableArray*insStack;
+@property (nonatomic, strong) NSMutableArray*insStack;
 //identify if the JSFramework has been loaded
 @property (nonatomic) BOOL frameworkLoadFinished;
 //store some methods temporarily before JSFramework is loaded
@@ -415,7 +415,7 @@ _Pragma("clang diagnostic pop") \
 {
 if (_insStack) return _insStack;
 
-_insStack = [WXThreadSafeMutableArray array];
+_insStack = [NSMutableArray array];
 
 return _insStack;
 }
@@ -662,9 +662,7 @@ _Pragma("clang diagnostic pop") \
 
 //remove instance from stack
@synchronized(self) {
-   if([self.insStack containsObject:instance]){
-   [self.insStack removeObject:instance];
-   }
+   [self.insStack removeObject:instance];
}
 
 if(_jsBridge && [_jsBridge respondsToSelector:@selector(removeTimers:)]){



[GitHub] incubator-weex issue #1359: [WEEX-532][Android] Fix Iconfont issue

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1359
  






  

  
  Messages

  
  
  :book:
  has no jsfm file changed.

  

  :book:
  jsfm test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #1359: [WEEX-532][Android] Fix Iconfont issue

2018-07-23 Thread YorkShen
GitHub user YorkShen opened a pull request:

https://github.com/apache/incubator-weex/pull/1359

[WEEX-532][Android] Fix Iconfont issue

https://issues.apache.org/jira/browse/WEEX-532

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/YorkShen/incubator-weex android_iconfont

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1359.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1359


commit 8e47317e84f79e11f7ec588836109f8590efd568
Author: YorkShen 
Date:   2018-07-23T10:01:43Z

* [android] Fix Iconfont issue

http://rax.alibaba-inc.com/playground/28149825-d0c2-490a-b048-17832cf31886




---


[GitHub] incubator-weex issue #1358: [WEEX-461][jsfm] Remove useless trace function i...

2018-07-23 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/1358
  





  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #1358: [WEEX-461][jsfm] Remove useless trace fun...

2018-07-23 Thread Hanks10100
GitHub user Hanks10100 opened a pull request:

https://github.com/apache/incubator-weex/pull/1358

[WEEX-461][jsfm] Remove useless trace function in js framework

Stop tracking the DSL type in js framework.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Hanks10100/incubator-weex jsfm-feature-notrace

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/1358.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1358


commit 0b2657c1061b8751b66ac2105a9685edc18eefb3
Author: Hanks 
Date:   2018-07-23T08:06:38Z

[WEEX-461][jsfm] remove useless trace function in js framework




---