cordova-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From agri...@apache.org
Subject android commit: [CB-2963] Re-enable sending messages in batches.
Date Tue, 16 Apr 2013 19:05:23 GMT
Updated Branches:
  refs/heads/master 383b3dadd -> b2d61679f


[CB-2963] Re-enable sending messages in batches.

Disabled by CB-1745, which is now reverted.


Project: http://git-wip-us.apache.org/repos/asf/cordova-android/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-android/commit/b2d61679
Tree: http://git-wip-us.apache.org/repos/asf/cordova-android/tree/b2d61679
Diff: http://git-wip-us.apache.org/repos/asf/cordova-android/diff/b2d61679

Branch: refs/heads/master
Commit: b2d61679fbb544e9af8c06b0d6375131954db9e2
Parents: 383b3da
Author: Andrew Grieve <agrieve@chromium.org>
Authored: Tue Apr 16 15:04:56 2013 -0400
Committer: Andrew Grieve <agrieve@chromium.org>
Committed: Tue Apr 16 15:04:56 2013 -0400

----------------------------------------------------------------------
 .../org/apache/cordova/NativeToJsMessageQueue.java |    9 +++------
 1 files changed, 3 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-android/blob/b2d61679/framework/src/org/apache/cordova/NativeToJsMessageQueue.java
----------------------------------------------------------------------
diff --git a/framework/src/org/apache/cordova/NativeToJsMessageQueue.java b/framework/src/org/apache/cordova/NativeToJsMessageQueue.java
index ea684a4..8a13213 100755
--- a/framework/src/org/apache/cordova/NativeToJsMessageQueue.java
+++ b/framework/src/org/apache/cordova/NativeToJsMessageQueue.java
@@ -50,13 +50,10 @@ public class NativeToJsMessageQueue {
     // exec() is asynchronous. Set this to true when running bridge benchmarks.
     static final boolean DISABLE_EXEC_CHAINING = false;
     
-    // Upper limit for how much data to send to JS in one shot.
-    // TODO(agrieve): This is currently disable. It should be re-enabled once we
-    // remove support for returning values from exec() calls. This was
-    // deprecated in 2.2.0.
-    // Also, this currently only chops up on message boundaries. It may be useful
+    // Arbitrarily chosen upper limit for how much data to send to JS in one shot.
+    // This currently only chops up on message boundaries. It may be useful
     // to allow it to break up messages.
-    private static int MAX_PAYLOAD_SIZE = -1; //50 * 1024 * 10240;
+    private static int MAX_PAYLOAD_SIZE = 50 * 1024 * 10240;
     
     /**
      * The index into registeredListeners to treat as active. 


Mime
View raw message